summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorStephen Hemminger <stephen@networkplumber.org>2019-11-01 13:12:55 -0700
committerFerruh Yigit <ferruh.yigit@intel.com>2019-11-11 14:23:02 +0100
commitbd67b6772492cdd9f7249bdff7163e9698bdcc16 (patch)
treee237b2591a932ac6c3db49910d75c65d9424a92d
parentc19beb3f38cd8463f87f01e1f5941a2a821b8ac6 (diff)
downloaddpdk-bd67b6772492cdd9f7249bdff7163e9698bdcc16.zip
dpdk-bd67b6772492cdd9f7249bdff7163e9698bdcc16.tar.gz
dpdk-bd67b6772492cdd9f7249bdff7163e9698bdcc16.tar.xz
app/testpmd: block xstats for hidden ports
All the other testpmd commands block access to devices that are owned. Looks like xstat got overlooked. Fixes: bfd5051b43b5 ("app/testpmd: new command to get extended statistics") Cc: stable@dpdk.org Signed-off-by: Stephen Hemminger <stephen@networkplumber.org> Acked-by: Bernard Iremonger <bernard.iremonger@intel.com>
-rw-r--r--app/test-pmd/config.c8
1 files changed, 8 insertions, 0 deletions
diff --git a/app/test-pmd/config.c b/app/test-pmd/config.c
index b603974..2a51d96 100644
--- a/app/test-pmd/config.c
+++ b/app/test-pmd/config.c
@@ -238,6 +238,10 @@ nic_xstats_display(portid_t port_id)
int cnt_xstats, idx_xstat;
struct rte_eth_xstat_name *xstats_names;
+ if (port_id_is_invalid(port_id, ENABLED_WARN)) {
+ print_valid_ports();
+ return;
+ }
printf("###### NIC extended statistics for port %-2d\n", port_id);
if (!rte_eth_dev_is_valid_port(port_id)) {
printf("Error: Invalid port number %i\n", port_id);
@@ -295,6 +299,10 @@ nic_xstats_clear(portid_t port_id)
{
int ret;
+ if (port_id_is_invalid(port_id, ENABLED_WARN)) {
+ print_valid_ports();
+ return;
+ }
ret = rte_eth_xstats_reset(port_id);
if (ret != 0) {
printf("%s: Error: failed to reset xstats (port %u): %s",