[dpdk-dev] [PATCH v2 3/5] test/hash: add lock free reader writer functional tests
Wang, Yipeng1
yipeng1.wang at intel.com
Wed Feb 5 20:34:41 CET 2020
>-----Original Message-----
>From: David Marchand [mailto:david.marchand at redhat.com]
>Sent: Wednesday, February 5, 2020 8:42 AM
>To: Honnappa Nagarahalli <Honnappa.Nagarahalli at arm.com>
>Cc: Amit Gupta <agupta3 at marvell.com>; Wang, Yipeng1 <yipeng1.wang at intel.com>; Gobriel, Sameh <sameh.gobriel at intel.com>;
>thomas at monjalon.net; dev <dev at dpdk.org>; nd <nd at arm.com>
>Subject: Re: [PATCH v2 3/5] test/hash: add lock free reader writer functional tests
>
>On Wed, Feb 5, 2020 at 5:22 PM Honnappa Nagarahalli
><Honnappa.Nagarahalli at arm.com> wrote:
>>
>> >
>> > On Mon, Feb 3, 2020 at 8:49 PM Honnappa Nagarahalli
>> > <honnappa.nagarahalli at arm.com> wrote:
>> > >
>> > > Add lock-free reader writer concurrency functional tests.
>> > > These tests will provide the same coverage that non lock-free APIs
>> > > have.
>> > >
>> > > Signed-off-by: Honnappa Nagarahalli <honnappa.nagarahalli at arm.com>
>> > > ---
>> > > app/test/test_hash_readwrite.c | 58
>> > > +++++++++++++++++++++-------------
>> > > 1 file changed, 36 insertions(+), 22 deletions(-)
>> > >
>> > > diff --git a/app/test/test_hash_readwrite.c
>> > > b/app/test/test_hash_readwrite.c index 635ed5a9f..a9429091c 100644
>> > > --- a/app/test/test_hash_readwrite.c
>> > > +++ b/app/test/test_hash_readwrite.c
>> > > @@ -121,7 +121,7 @@
>> > test_hash_readwrite_worker(__attribute__((unused))
>> > > void *arg) }
>> > >
>> > > static int
>> > > -init_params(int use_ext, int use_htm, int use_jhash)
>> > > +init_params(int use_ext, int use_htm, int rw_lf, int use_jhash)
>> > > {
>> > > unsigned int i;
>> > >
>> > > @@ -140,15 +140,16 @@ init_params(int use_ext, int use_htm, int
>> > use_jhash)
>> > > else
>> > > hash_params.hash_func = rte_hash_crc;
>> > >
>> > > + hash_params.extra_flag =
>> > > + RTE_HASH_EXTRA_FLAGS_MULTI_WRITER_ADD;
>> > > if (use_htm)
>> > > - hash_params.extra_flag =
>> > > - RTE_HASH_EXTRA_FLAGS_TRANS_MEM_SUPPORT |
>> > > - RTE_HASH_EXTRA_FLAGS_RW_CONCURRENCY |
>> > > - RTE_HASH_EXTRA_FLAGS_MULTI_WRITER_ADD;
>> > > + hash_params.extra_flag |=
>> > > + RTE_HASH_EXTRA_FLAGS_TRANS_MEM_SUPPORT;
[Wang, Yipeng] Thanks for the patch Honnappa. Here I think we still need the RTE_HASH_EXTRA_FLAGS_RW_CONCURRENCY
Flag even with HTM.
Other commits in this series look good to me and seems David already applied.
Thanks!
More information about the dev
mailing list