patch 'eventdev: fix memory size for telemetry' has been queued to stable release 22.11.2

Xueming Li xuemingl at nvidia.com
Sun Apr 9 17:23:15 CEST 2023


Hi,

FYI, your patch has been queued to stable release 22.11.2

Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 04/11/23. 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://git.dpdk.org/dpdk-stable/log/?h=22.11-staging

This queued commit can be viewed at:
https://git.dpdk.org/dpdk-stable/log/?h=22.11-staging/commit/f76551552e977078105dcac0cb2e1c5dcaadec06

Thanks.

Xueming Li <xuemingl at nvidia.com>

---
>From f76551552e977078105dcac0cb2e1c5dcaadec06 Mon Sep 17 00:00:00 2001
From: Abdullah Sevincer <abdullah.sevincer at intel.com>
Date: Tue, 28 Feb 2023 15:31:23 -0600
Subject: [PATCH] eventdev: fix memory size for telemetry
Cc: Xueming Li <xuemingl at nvidia.com>

[ upstream commit e0fb0051ff8f621a2b001d9243cebe50f3d42cf5 ]

Fix the invalid memory free operation caused by extending
xstats ID width from 32 to 64 bits.

Telemetry application crashed when freeing memory,
hence, the telemetry app could not display xstats.

Fixes: 1bdfe4d76e98 ("eventdev: increase xstats ID width to 64 bits")

Signed-off-by: Abdullah Sevincer <abdullah.sevincer at intel.com>
Acked-by: Jerin Jacob <jerinj at marvell.com>
---
 lib/eventdev/rte_eventdev.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/eventdev/rte_eventdev.c b/lib/eventdev/rte_eventdev.c
index b0414206d9..78336faa6a 100644
--- a/lib/eventdev/rte_eventdev.c
+++ b/lib/eventdev/rte_eventdev.c
@@ -1678,7 +1678,7 @@ eventdev_build_telemetry_data(int dev_id,
 	if (xstat_names == NULL)
 		return -1;
 
-	ids = malloc((sizeof(unsigned int)) * num_xstats);
+	ids = malloc((sizeof(uint64_t)) * num_xstats);
 	if (ids == NULL) {
 		free(xstat_names);
 		return -1;
-- 
2.25.1

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2023-04-09 21:45:39.015093800 +0800
+++ 0007-eventdev-fix-memory-size-for-telemetry.patch	2023-04-09 21:45:38.569042200 +0800
@@ -1 +1 @@
-From e0fb0051ff8f621a2b001d9243cebe50f3d42cf5 Mon Sep 17 00:00:00 2001
+From f76551552e977078105dcac0cb2e1c5dcaadec06 Mon Sep 17 00:00:00 2001
@@ -4,0 +5,3 @@
+Cc: Xueming Li <xuemingl at nvidia.com>
+
+[ upstream commit e0fb0051ff8f621a2b001d9243cebe50f3d42cf5 ]
@@ -13 +15,0 @@
-Cc: stable at dpdk.org
@@ -22 +24 @@
-index 71a29f3506..6ab4524332 100644
+index b0414206d9..78336faa6a 100644
@@ -25 +27 @@
-@@ -1725,7 +1725,7 @@ eventdev_build_telemetry_data(int dev_id,
+@@ -1678,7 +1678,7 @@ eventdev_build_telemetry_data(int dev_id,


More information about the stable mailing list