A next commit will protect all telemetry commands. Prefer exposing all commands regardless of OS, and return an error when invoked on Windows.
Signed-off-by: David Marchand <david.march...@redhat.com> --- lib/ethdev/rte_ethdev_telemetry.c | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/lib/ethdev/rte_ethdev_telemetry.c b/lib/ethdev/rte_ethdev_telemetry.c index 6b873e7abe..8031a58595 100644 --- a/lib/ethdev/rte_ethdev_telemetry.c +++ b/lib/ethdev/rte_ethdev_telemetry.c @@ -227,7 +227,15 @@ eth_dev_handle_port_dump_priv(const char *cmd __rte_unused, free(buf); return 0; } -#endif /* !RTE_EXEC_ENV_WINDOWS */ +#else /* !RTE_EXEC_ENV_WINDOWS */ +static int +eth_dev_handle_port_dump_priv(const char *cmd __rte_unused, + const char *params __rte_unused, + struct rte_tel_data *d __rte_unused) +{ + return -EINVAL; +} +#endif /* RTE_EXEC_ENV_WINDOWS */ static int eth_dev_handle_port_link_status(const char *cmd __rte_unused, @@ -1403,10 +1411,8 @@ RTE_INIT(ethdev_init_telemetry) "Returns the common stats for a port. Parameters: int port_id"); rte_telemetry_register_cmd("/ethdev/xstats", eth_dev_handle_port_xstats, "Returns the extended stats for a port. Parameters: int port_id,hide_zero=true|false(Optional for indicates hide zero xstats)"); -#ifndef RTE_EXEC_ENV_WINDOWS rte_telemetry_register_cmd("/ethdev/dump_priv", eth_dev_handle_port_dump_priv, "Returns dump private information for a port. Parameters: int port_id"); -#endif rte_telemetry_register_cmd("/ethdev/link_status", eth_dev_handle_port_link_status, "Returns the link status for a port. Parameters: int port_id"); -- 2.46.2