From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id B0D4948B69 for ; Fri, 21 Nov 2025 12:23:48 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id AAD2C4026F; Fri, 21 Nov 2025 12:23:48 +0100 (CET) Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by mails.dpdk.org (Postfix) with ESMTP id 9E9B04026F for ; Fri, 21 Nov 2025 12:23:47 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1763724227; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=WGCNPzwE0v05tew0wggltXnIUBF/6Pvprlzs1Vf+82M=; b=DEpNhJIsD8Gs8AEdzi/M6laD+89Wmi12ZE+jEkjyubYCyteqRLktKvVQgjPH9xSEnyDSP5 pqruUUbhxXDK3Lp+SwvmfMzOi6DAJx6iXEDmx9yFpZjmF2L1lA0iMACdzlWDlelp74CaQE wIViyapy9JzgjsKhIauDPt7bkZaOlBY= Received: from mx-prod-mc-06.mail-002.prod.us-west-2.aws.redhat.com (ec2-35-165-154-97.us-west-2.compute.amazonaws.com [35.165.154.97]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-81-wcoqSXKTNH-J_B2VLe0mcg-1; Fri, 21 Nov 2025 06:23:45 -0500 X-MC-Unique: wcoqSXKTNH-J_B2VLe0mcg-1 X-Mimecast-MFC-AGG-ID: wcoqSXKTNH-J_B2VLe0mcg_1763724225 Received: from mx-prod-int-01.mail-002.prod.us-west-2.aws.redhat.com (mx-prod-int-01.mail-002.prod.us-west-2.aws.redhat.com [10.30.177.4]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by mx-prod-mc-06.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTPS id EF37D180047F; Fri, 21 Nov 2025 11:23:44 +0000 (UTC) Received: from rh.redhat.com (unknown [10.42.28.165]) by mx-prod-int-01.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTP id 65BD930044DB; Fri, 21 Nov 2025 11:23:43 +0000 (UTC) From: Kevin Traynor To: Bruce Richardson Cc: Chengwen Feng , dpdk stable Subject: patch 'telemetry: make socket handler typedef private' has been queued to stable release 24.11.4 Date: Fri, 21 Nov 2025 11:20:44 +0000 Message-ID: <20251121112128.485623-60-ktraynor@redhat.com> In-Reply-To: <20251121112128.485623-1-ktraynor@redhat.com> References: <20251121112128.485623-1-ktraynor@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.4.1 on 10.30.177.4 X-Mimecast-Spam-Score: 0 X-Mimecast-MFC-PROC-ID: HP7I_-NP59aR9G-5x4FOn9RAgAtknR7JhvGxWcvmDA4_1763724225 X-Mimecast-Originator: redhat.com Content-Transfer-Encoding: 8bit content-type: text/plain; charset="US-ASCII"; x-default=true X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.29 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 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 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 Acked-by: Chengwen Feng --- 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@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 {