* [PATCH] examples/l3fwd-power: fix telemetry command registration
@ 2025-10-06 10:10 Sivaprasad Tummala
2025-11-06 13:41 ` Bruce Richardson
0 siblings, 1 reply; 2+ messages in thread
From: Sivaprasad Tummala @ 2025-10-06 10:10 UTC (permalink / raw)
To: mb, david.marchand, ktraynor, thomas, konstantin.ananyev,
konstantin.v.ananyev, bruce.richardson, maxime.coquelin, aconole,
ferruh.yigit
Cc: dev, ciara.power, stable
Telemetry command registration fails if the command name contains
characters other than alphanumeric, underscore (_), and forward
slash (/).
The l3fwd-power example previously used "/l3fwd-power/stats"
as telemetry command, which includes a hyphen (-) and causes
registration failure.
This patch fixes the issue by replacing the hyphen with an
underscore, changing the command to "/l3fwd_power/stats"
Fixes: a35919a1139b ("examples/l3fwd-power: use new telemetry")
Cc: ciara.power@intel.com
Cc: stable@dpdk.org
Signed-off-by: Sivaprasad Tummala <sivaprasad.tummala@amd.com>
---
examples/l3fwd-power/main.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/examples/l3fwd-power/main.c b/examples/l3fwd-power/main.c
index e27b8531b5..f9ce9e6698 100644
--- a/examples/l3fwd-power/main.c
+++ b/examples/l3fwd-power/main.c
@@ -2910,7 +2910,7 @@ main(int argc, char **argv)
rte_spinlock_init(&stats[lcore_id].telemetry_lock);
}
rte_timer_init(&telemetry_timer);
- rte_telemetry_register_cmd("/l3fwd-power/stats",
+ rte_telemetry_register_cmd("/l3fwd_power/stats",
handle_app_stats,
"Returns global power stats. Parameters: None");
rte_eal_mp_remote_launch(main_telemetry_loop, NULL,
--
2.43.0
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] examples/l3fwd-power: fix telemetry command registration
2025-10-06 10:10 [PATCH] examples/l3fwd-power: fix telemetry command registration Sivaprasad Tummala
@ 2025-11-06 13:41 ` Bruce Richardson
0 siblings, 0 replies; 2+ messages in thread
From: Bruce Richardson @ 2025-11-06 13:41 UTC (permalink / raw)
To: Sivaprasad Tummala
Cc: mb, david.marchand, ktraynor, thomas, konstantin.ananyev,
konstantin.v.ananyev, maxime.coquelin, aconole, ferruh.yigit,
dev, ciara.power, stable
On Mon, Oct 06, 2025 at 10:10:58AM +0000, Sivaprasad Tummala wrote:
> Telemetry command registration fails if the command name contains
> characters other than alphanumeric, underscore (_), and forward
> slash (/).
>
> The l3fwd-power example previously used "/l3fwd-power/stats"
> as telemetry command, which includes a hyphen (-) and causes
> registration failure.
>
> This patch fixes the issue by replacing the hyphen with an
> underscore, changing the command to "/l3fwd_power/stats"
>
> Fixes: a35919a1139b ("examples/l3fwd-power: use new telemetry")
> Cc: ciara.power@intel.com
> Cc: stable@dpdk.org
>
> Signed-off-by: Sivaprasad Tummala <sivaprasad.tummala@amd.com>
Acked-by: Bruce Richardson <bruce.richardson@intel.com>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2025-11-06 13:42 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-10-06 10:10 [PATCH] examples/l3fwd-power: fix telemetry command registration Sivaprasad Tummala
2025-11-06 13:41 ` Bruce Richardson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).