Re: [dpdk-dev] [PATCH v5 1/3] lib/lpm: integrate RCU QSBR

2020-07-04 Thread Ruifeng Wang
Horman > ; dev ; Ananyev, Konstantin > ; Honnappa Nagarahalli > ; nd > Subject: Re: [dpdk-dev] [PATCH v5 1/3] lib/lpm: integrate RCU QSBR > > On Mon, Jun 29, 2020 at 10:03 AM Ruifeng Wang > wrote: > > > > Currently, the tbl8 group is freed even though the readers

Re: [dpdk-dev] [PATCH v5 1/3] lib/lpm: integrate RCU QSBR

2020-07-03 Thread David Marchand
Hello Ruifeng, On Mon, Jun 29, 2020 at 1:56 PM David Marchand wrote: > > diff --git a/lib/librte_lpm/rte_lpm.c b/lib/librte_lpm/rte_lpm.c > > index 38ab512a4..41e9c49b8 100644 > > --- a/lib/librte_lpm/rte_lpm.c > > +++ b/lib/librte_lpm/rte_lpm.c > > @@ -1,5 +1,6 @@ > > /* SPDX-License-Identifier

Re: [dpdk-dev] [PATCH v5 1/3] lib/lpm: integrate RCU QSBR

2020-06-30 Thread Kinsella, Ray
On 29/06/2020 13:55, Bruce Richardson wrote: > On Mon, Jun 29, 2020 at 01:56:07PM +0200, David Marchand wrote: >> On Mon, Jun 29, 2020 at 10:03 AM Ruifeng Wang wrote: >>> >>> Currently, the tbl8 group is freed even though the readers might be >>> using the tbl8 group entries. The freed tbl8 gro

Re: [dpdk-dev] [PATCH v5 1/3] lib/lpm: integrate RCU QSBR

2020-06-30 Thread Kinsella, Ray
On 29/06/2020 09:02, Ruifeng Wang wrote: > Currently, the tbl8 group is freed even though the readers might be > using the tbl8 group entries. The freed tbl8 group can be reallocated > quickly. This results in incorrect lookup results. > > RCU QSBR process is integrated for safe tbl8 group recl

Re: [dpdk-dev] [PATCH v5 1/3] lib/lpm: integrate RCU QSBR

2020-06-29 Thread Bruce Richardson
On Mon, Jun 29, 2020 at 01:56:07PM +0200, David Marchand wrote: > On Mon, Jun 29, 2020 at 10:03 AM Ruifeng Wang wrote: > > > > Currently, the tbl8 group is freed even though the readers might be > > using the tbl8 group entries. The freed tbl8 group can be reallocated > > quickly. This results in

Re: [dpdk-dev] [PATCH v5 1/3] lib/lpm: integrate RCU QSBR

2020-06-29 Thread David Marchand
On Mon, Jun 29, 2020 at 10:03 AM Ruifeng Wang wrote: > > Currently, the tbl8 group is freed even though the readers might be > using the tbl8 group entries. The freed tbl8 group can be reallocated > quickly. This results in incorrect lookup results. > > RCU QSBR process is integrated for safe tbl8

[dpdk-dev] [PATCH v5 1/3] lib/lpm: integrate RCU QSBR

2020-06-29 Thread Ruifeng Wang
Currently, the tbl8 group is freed even though the readers might be using the tbl8 group entries. The freed tbl8 group can be reallocated quickly. This results in incorrect lookup results. RCU QSBR process is integrated for safe tbl8 group reclaim. Refer to RCU documentation to understand various