Hello, On Wed, May 22, 2019 at 2:16 AM Wang, Yipeng1 <yipeng1.w...@intel.com> wrote:
> >-----Original Message----- > >From: David Marchand [mailto:david.march...@redhat.com] > >Sent: Wednesday, May 15, 2019 12:57 AM > >To: dev@dpdk.org > >Cc: tho...@monjalon.net; step...@networkplumber.org; sta...@dpdk.org; > Wang, Yipeng1 <yipeng1.w...@intel.com>; Gobriel, > >Sameh <sameh.gobr...@intel.com>; Richardson, Bruce < > bruce.richard...@intel.com>; De Lara Guarch, Pablo > ><pablo.de.lara.gua...@intel.com> > >Subject: [PATCH 1/2] test/hash: use existing lcore API > > > >Prefer the existing apis rather than direct access the configuration > >structure. > > > >Fixes: c7eb0972e74b ("test/hash: add lock-free r/w concurrency") > >Fixes: 3f9aab961ed3 ("test/hash: check lock-free extendable bucket") > >Cc: sta...@dpdk.org > > > >Signed-off-by: David Marchand <david.march...@redhat.com> > >--- > > app/test/test_hash_readwrite_lf.c | 29 ++++++++++++++--------------- > > 1 file changed, 14 insertions(+), 15 deletions(-) > > > > [Wang, Yipeng] > > Hi, David, I tested the patch and the test case seems now frozen after > running in the middle. Have you seen same thing? > > I include the original author Dharmik for the file for more reviews. > > > Output: > > " > .... > > Test: Hash add - key shift, Hash lookup - miss > > Number of readers: 1 > Cycles per lookup: 112 > > Number of readers: 2 > Cycles per lookup: 113 > > Number of readers: 4 > Cycles per lookup: 116 > > ** With bulk-lookup ** > > Number of readers: 1 > Cycles per lookup: 41 > > Number of readers: 2 > Cycles per lookup: 40 > > Number of readers: 4 > Cycles per lookup: 40 > > Test: Multi-add-lookup > > Number of writers: 2 > Number of readers: 1 > Cycles per lookup: 166 > > Number of writers: 2 > Number of readers: 2 > " > <Freezing here> > Thanks for reporting. I reproduced the issue. I did not catch it as I was starting the test with 4 cores on my laptop. I will post a fix and/or an updated patch. -- David Marchand
