patch 'test/latency: fix loop boundary' has been queued to stable release 19.11.11

christian.ehrhardt at canonical.com christian.ehrhardt at canonical.com
Tue Nov 30 17:34:19 CET 2021


Hi,

FYI, your patch has been queued to stable release 19.11.11

Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before December 10th 2021. 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/cpaelzer/dpdk-stable-queue

This queued commit can be viewed at:
https://github.com/cpaelzer/dpdk-stable-queue/commit/1132c3fa61435174b5eaa5ddd49e9f3fc2312fd9

Thanks.

Christian Ehrhardt <christian.ehrhardt at canonical.com>

---
>From 1132c3fa61435174b5eaa5ddd49e9f3fc2312fd9 Mon Sep 17 00:00:00 2001
From: David Marchand <david.marchand at redhat.com>
Date: Sat, 2 Oct 2021 18:24:31 +0200
Subject: [PATCH] test/latency: fix loop boundary

[ upstream commit cb3dd14c77f4a44c204b7ac10898ecc33afeb9a0 ]

Caught running ASAN.

lat_stats_strings[] is an array containing NUM_STATS strings.

Fixes: 1e3676a06e4c ("test/latency: add unit tests for latencystats library")

Signed-off-by: David Marchand <david.marchand at redhat.com>
Acked-by: Reshma Pattan <reshma.pattan at intel.com>
---
 app/test/test_latencystats.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/test/test_latencystats.c b/app/test/test_latencystats.c
index 968e0bc470..fcc9e83558 100644
--- a/app/test/test_latencystats.c
+++ b/app/test/test_latencystats.c
@@ -77,7 +77,7 @@ static int test_latencystats_get_names(void)
 	/* Success Test: Valid names and size */
 	size = NUM_STATS;
 	ret = rte_latencystats_get_names(names, size);
-	for (i = 0; i <= NUM_STATS; i++) {
+	for (i = 0; i < NUM_STATS; i++) {
 		if (strcmp(lat_stats_strings[i].name, names[i].name) == 0)
 			printf(" %s\n", names[i].name);
 		else
-- 
2.34.0

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2021-11-30 16:50:09.239620831 +0100
+++ 0055-test-latency-fix-loop-boundary.patch	2021-11-30 16:50:05.706872913 +0100
@@ -1 +1 @@
-From cb3dd14c77f4a44c204b7ac10898ecc33afeb9a0 Mon Sep 17 00:00:00 2001
+From 1132c3fa61435174b5eaa5ddd49e9f3fc2312fd9 Mon Sep 17 00:00:00 2001
@@ -5,0 +6,2 @@
+[ upstream commit cb3dd14c77f4a44c204b7ac10898ecc33afeb9a0 ]
+
@@ -11 +12,0 @@
-Cc: stable at dpdk.org
@@ -20 +21 @@
-index 427339904d..724acbc315 100644
+index 968e0bc470..fcc9e83558 100644
@@ -23 +24 @@
-@@ -80,7 +80,7 @@ static int test_latencystats_get_names(void)
+@@ -77,7 +77,7 @@ static int test_latencystats_get_names(void)


More information about the stable mailing list