From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 3D0B9A046B for ; Fri, 23 Aug 2019 11:44:15 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 32DB81BF9B; Fri, 23 Aug 2019 11:44:15 +0200 (CEST) Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id BE3151BFA6 for ; Fri, 23 Aug 2019 11:44:13 +0200 (CEST) Received: from smtp.corp.redhat.com (int-mx06.intmail.prod.int.phx2.redhat.com [10.5.11.16]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 40EC789AC6; Fri, 23 Aug 2019 09:44:13 +0000 (UTC) Received: from rh.redhat.com (ovpn-116-48.ams2.redhat.com [10.36.116.48]) by smtp.corp.redhat.com (Postfix) with ESMTP id 04C375C226; Fri, 23 Aug 2019 09:44:11 +0000 (UTC) From: Kevin Traynor To: David Marchand Cc: Aaron Conole , Yipeng Wang , dpdk stable Date: Fri, 23 Aug 2019 10:42:58 +0100 Message-Id: <20190823094336.12078-8-ktraynor@redhat.com> In-Reply-To: <20190823094336.12078-1-ktraynor@redhat.com> References: <20190823094336.12078-1-ktraynor@redhat.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Scanned-By: MIMEDefang 2.79 on 10.5.11.16 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.26]); Fri, 23 Aug 2019 09:44:13 +0000 (UTC) Subject: [dpdk-stable] patch 'test/hash: fix off-by-one check on core count' has been queued to LTS release 18.11.3 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" Hi, FYI, your patch has been queued to LTS release 18.11.3 Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet. It will be pushed if I get no objections before 08/28/19. So please shout if anyone has objections. Also note that after the patch there's a diff of the upstream commit vs the patch applied to the branch. This will indicate if there was any rebasing needed to apply to the stable branch. If there were code changes for rebasing (ie: not only metadata diffs), please double check that the rebase was correctly done. Queued patches are on a temporary branch at: https://github.com/kevintraynor/dpdk-stable-queue This queued commit can be viewed at: https://github.com/kevintraynor/dpdk-stable-queue/commit/88e2855e59c2ba76818e47b779f826399740972f Thanks. Kevin Traynor --- >From 88e2855e59c2ba76818e47b779f826399740972f Mon Sep 17 00:00:00 2001 From: David Marchand Date: Sat, 15 Jun 2019 08:42:19 +0200 Subject: [PATCH] test/hash: fix off-by-one check on core count [ upstream commit bf7a24b997dc1f7516f38a38a8ce3c364d64aff4 ] This subtest wants to start rwc_core_cnt[n] reader threads, while the master core is waiting for them to report. Fixes: c7eb0972e74b ("test/hash: add lock-free r/w concurrency") Signed-off-by: David Marchand Acked-by: Aaron Conole Acked-by: Yipeng Wang --- test/test/test_hash_readwrite_lf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/test/test_hash_readwrite_lf.c b/test/test/test_hash_readwrite_lf.c index 934512ddb..f0fc63b9f 100644 --- a/test/test/test_hash_readwrite_lf.c +++ b/test/test/test_hash_readwrite_lf.c @@ -821,5 +821,5 @@ test_hash_add_ks_lookup_hit_sp(struct rwc_perf *rwc_perf_results, int rwc_lf, for (n = 0; n < NUM_TEST; n++) { unsigned int tot_lcore = rte_lcore_count(); - if (tot_lcore < rwc_core_cnt[n]) + if (tot_lcore < rwc_core_cnt[n] + 1) goto finish; -- 2.20.1 --- Diff of the applied patch vs upstream commit (please double-check if non-empty: --- --- - 2019-08-22 19:38:21.001953057 +0100 +++ 0008-test-hash-fix-off-by-one-check-on-core-count.patch 2019-08-22 19:38:20.399027785 +0100 @@ -1 +1 @@ -From bf7a24b997dc1f7516f38a38a8ce3c364d64aff4 Mon Sep 17 00:00:00 2001 +From 88e2855e59c2ba76818e47b779f826399740972f Mon Sep 17 00:00:00 2001 @@ -5,0 +6,2 @@ +[ upstream commit bf7a24b997dc1f7516f38a38a8ce3c364d64aff4 ] + @@ -10 +11,0 @@ -Cc: stable@dpdk.org @@ -16 +17 @@ - app/test/test_hash_readwrite_lf.c | 2 +- + test/test/test_hash_readwrite_lf.c | 2 +- @@ -19,5 +20,5 @@ -diff --git a/app/test/test_hash_readwrite_lf.c b/app/test/test_hash_readwrite_lf.c -index 343a338b4..f9f233a73 100644 ---- a/app/test/test_hash_readwrite_lf.c -+++ b/app/test/test_hash_readwrite_lf.c -@@ -940,5 +940,5 @@ test_hash_add_ks_lookup_hit_sp(struct rwc_perf *rwc_perf_results, int rwc_lf, +diff --git a/test/test/test_hash_readwrite_lf.c b/test/test/test_hash_readwrite_lf.c +index 934512ddb..f0fc63b9f 100644 +--- a/test/test/test_hash_readwrite_lf.c ++++ b/test/test/test_hash_readwrite_lf.c +@@ -821,5 +821,5 @@ test_hash_add_ks_lookup_hit_sp(struct rwc_perf *rwc_perf_results, int rwc_lf,