patch 'telemetry: make socket handler typedef private' has been queued to stable release 24.11.4

[Kevin Traynor]

Hi,

FYI, your patch has been queued to stable release 24.11.4

Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 11/26/25. 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

This queued commit can be viewed at:
https://github.com/kevintraynor/dpdk-stable/commit/b56af1e73291ea5633306a806654e8b9b9a3b4eb

Thanks.

Kevin

---
>From b56af1e73291ea5633306a806654e8b9b9a3b4eb Mon Sep 17 00:00:00 2001
From: Bruce Richardson <bruce.richardson at intel.com>
Date: Fri, 7 Nov 2025 09:32:59 +0000
Subject: [PATCH] telemetry: make socket handler typedef private

[ upstream commit 572451f7728711b1c377cb46d885f918be4c7763 ]

The socket handler typedef, just called "handler" does not belong in the
public namespace, especially since it is unprefixed. Rename to
"telemetry_sock_handler" and move to internal header.

Doing so also clears shadowing warnings in dmadev library, when it is
built with -Wshadow.

Fixes: 6dd571fd07c3 ("telemetry: introduce new functionality")

Signed-off-by: Bruce Richardson <bruce.richardson at intel.com>
Acked-by: Chengwen Feng <fengchengwen at huawei.com>
---
 lib/telemetry/rte_telemetry.h      | 11 -----------
 lib/telemetry/telemetry.c          |  2 +-
 lib/telemetry/telemetry_internal.h | 11 +++++++++++
 3 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/lib/telemetry/rte_telemetry.h b/lib/telemetry/rte_telemetry.h
index 2ccfc73a5f..7245b7e219 100644
--- a/lib/telemetry/rte_telemetry.h
+++ b/lib/telemetry/rte_telemetry.h
@@ -361,15 +361,4 @@ typedef int (*telemetry_arg_cb)(const char *cmd, const char *params, void *arg,
 		struct rte_tel_data *info);
 
-/**
- * Used for handling data received over a telemetry socket.
- *
- * @param sock_id
- * ID for the socket to be used by the handler.
- *
- * @return
- * Void.
- */
-typedef void * (*handler)(void *sock_id);
-
 /**
  * Used when registering a command and callback function with telemetry.
diff --git a/lib/telemetry/telemetry.c b/lib/telemetry/telemetry.c
index 31a2c91c06..8fe35f362b 100644
--- a/lib/telemetry/telemetry.c
+++ b/lib/telemetry/telemetry.c
@@ -47,5 +47,5 @@ struct socket {
 	int sock;
 	char path[sizeof(((struct sockaddr_un *)0)->sun_path)];
-	handler fn;
+	telemetry_sock_handler fn;
 	RTE_ATOMIC(uint16_t) *num_clients;
 };
diff --git a/lib/telemetry/telemetry_internal.h b/lib/telemetry/telemetry_internal.h
index b331e9458f..2fd9fbd7c1 100644
--- a/lib/telemetry/telemetry_internal.h
+++ b/lib/telemetry/telemetry_internal.h
@@ -26,4 +26,15 @@ enum rte_telemetry_legacy_data_req {
 };
 
+/**
+ * Used for handling data received over a telemetry socket.
+ *
+ * @param sock_id
+ * ID for the socket to be used by the handler.
+ *
+ * @return
+ * Void.
+ */
+typedef void * (*telemetry_sock_handler)(void *sock_id);
+
 /**
  * This telemetry callback is used when registering a legacy telemetry command.
-- 
2.51.0

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2025-11-21 11:05:11.477601187 +0000
+++ 0060-telemetry-make-socket-handler-typedef-private.patch	2025-11-21 11:05:09.537201485 +0000
@@ -1 +1 @@
-From 572451f7728711b1c377cb46d885f918be4c7763 Mon Sep 17 00:00:00 2001
+From b56af1e73291ea5633306a806654e8b9b9a3b4eb Mon Sep 17 00:00:00 2001
@@ -5,0 +6,2 @@
+[ upstream commit 572451f7728711b1c377cb46d885f918be4c7763 ]
+
@@ -14 +15,0 @@
-Cc: stable at dpdk.org
@@ -25 +26 @@
-index b9662a5213..0a58e518f7 100644
+index 2ccfc73a5f..7245b7e219 100644
@@ -28 +29 @@
-@@ -326,15 +326,4 @@ typedef int (*telemetry_arg_cb)(const char *cmd, const char *params, void *arg,
+@@ -361,15 +361,4 @@ typedef int (*telemetry_arg_cb)(const char *cmd, const char *params, void *arg,
@@ -45 +46 @@
-index 1cbbffbf3f..cf4324421d 100644
+index 31a2c91c06..8fe35f362b 100644
@@ -48 +49 @@
-@@ -48,5 +48,5 @@ struct socket {
+@@ -47,5 +47,5 @@ struct socket {