From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by dpdk.space (Postfix) with ESMTP id 917C8A05D3 for ; Wed, 22 May 2019 14:40:53 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 58628A69; Wed, 22 May 2019 14:40:53 +0200 (CEST) Received: from mail-ua1-f66.google.com (mail-ua1-f66.google.com [209.85.222.66]) by dpdk.org (Postfix) with ESMTP id 5924FA69 for ; Wed, 22 May 2019 14:40:52 +0200 (CEST) Received: by mail-ua1-f66.google.com with SMTP id r19so792968uap.5 for ; Wed, 22 May 2019 05:40:52 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=C/X4plxzCYJSqcXbuySM1pGV+ksli7gjkAHqndnRShw=; b=GRoaGTHRngM1W4CBG3nsT8ttBhr1ytmuxT+WwWhjlLZKLnjxl0+dvMS4YMVxIhP7Px IlFhHk0GlCGaEeCnFkJSu3MQgB1mThpAKFGBMHxiyrR7YrxWzCYviGAwHX0XHcCNlLMA NtWRxrxHXHKW2LEXmUyGIOFyMHFElNYRfLM04ZyK6ivA2vdr2moiBt9c3REtn05egnX3 xAc4rWMxJ+vy2yk/ueP+4TSR2WFHz1YmgnWE12zQZVMTBEFTkNYr0D65CYQ8qGS5XlEs vGa6VLoXYT6dVWPfLeD+iMSPA1vf+9XH0hluO8sW0S1NXKzXSqzwhfjeCeaMGcUQhRZQ xdjw== X-Gm-Message-State: APjAAAVtSVNadNMEhhrbdpZjbUviVCldvG34Xd2n915rD4VlLKDOf4tI pV1H4oCjzS1Jue3P854WM7tLVapBry7blHyxQr2VdQ== X-Google-Smtp-Source: APXvYqx+Uf119W6GmO6jsFoUqpdUvjqwwyosnI+TXqwSg6xiRaw1Eh5TR2RaP3on905BeKkLAFdhc9ABuWpfRJQEows= X-Received: by 2002:ab0:2bc6:: with SMTP id s6mr8120622uar.86.1558528851706; Wed, 22 May 2019 05:40:51 -0700 (PDT) MIME-Version: 1.0 References: <1557907020-1548-1-git-send-email-david.marchand@redhat.com> In-Reply-To: From: David Marchand Date: Wed, 22 May 2019 14:40:40 +0200 Message-ID: To: "Wang, Yipeng1" Cc: "dev@dpdk.org" , Dharmik Thakkar , "thomas@monjalon.net" , "stephen@networkplumber.org" , "stable@dpdk.org" , "Gobriel, Sameh" , "Richardson, Bruce" , "De Lara Guarch, Pablo" Content-Type: text/plain; charset="UTF-8" X-Content-Filtered-By: Mailman/MimeDel 2.1.15 Subject: Re: [dpdk-stable] [PATCH 1/2] test/hash: use existing lcore API X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Sender: "stable" Hello, On Wed, May 22, 2019 at 2:16 AM Wang, Yipeng1 wrote: > >-----Original Message----- > >From: David Marchand [mailto:david.marchand@redhat.com] > >Sent: Wednesday, May 15, 2019 12:57 AM > >To: dev@dpdk.org > >Cc: thomas@monjalon.net; stephen@networkplumber.org; stable@dpdk.org; > Wang, Yipeng1 ; Gobriel, > >Sameh ; Richardson, Bruce < > bruce.richardson@intel.com>; De Lara Guarch, Pablo > > > >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: stable@dpdk.org > > > >Signed-off-by: David Marchand > >--- > > 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 > " > > 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