>-----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>
