app/procinfo: call EAL cleanup before exit
[dpdk.git] / app / proc_info / main.c
index d576b42..2f53e3c 100644 (file)
@@ -1,34 +1,5 @@
-/*
- *   BSD LICENSE
- *
- *   Copyright(c) 2010-2017 Intel Corporation. All rights reserved.
- *   All rights reserved.
- *
- *   Redistribution and use in source and binary forms, with or without
- *   modification, are permitted provided that the following conditions
- *   are met:
- *
- *     * Redistributions of source code must retain the above copyright
- *       notice, this list of conditions and the following disclaimer.
- *     * Redistributions in binary form must reproduce the above copyright
- *       notice, this list of conditions and the following disclaimer in
- *       the documentation and/or other materials provided with the
- *       distribution.
- *     * Neither the name of Intel Corporation nor the names of its
- *       contributors may be used to endorse or promote products derived
- *       from this software without specific prior written permission.
- *
- *   THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
- *   "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
- *   LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
- *   A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
- *   OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
- *   SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
- *   LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
- *   DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
- *   THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- *   (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- *   OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+/* SPDX-License-Identifier: BSD-3-Clause
+ * Copyright(c) 2010-2017 Intel Corporation
  */
 
 #include <stdio.h>
@@ -53,7 +24,6 @@
 #include <rte_tailq.h>
 #include <rte_per_lcore.h>
 #include <rte_lcore.h>
-#include <rte_debug.h>
 #include <rte_log.h>
 #include <rte_atomic.h>
 #include <rte_branch_prediction.h>
@@ -86,6 +56,14 @@ static uint32_t reset_stats;
 static uint32_t reset_xstats;
 /**< Enable memory info. */
 static uint32_t mem_info;
+/**< Enable displaying xstat name. */
+static uint32_t enable_xstats_name;
+static char *xstats_name;
+
+/**< Enable xstats by ids. */
+#define MAX_NB_XSTATS_IDS 1024
+static uint32_t nb_xstats_ids;
+static uint64_t xstats_ids[MAX_NB_XSTATS_IDS];
 
 /**< display usage */
 static void
@@ -99,6 +77,9 @@ proc_info_usage(const char *prgname)
                        "default\n"
                "  --metrics: to display derived metrics of the ports, disabled by "
                        "default\n"
+               "  --xstats-name NAME: to display single xstat id by NAME\n"
+               "  --xstats-ids IDLIST: to display xstat values by id. "
+                       "The argument is comma-separated list of xstat ids to print out.\n"
                "  --stats-reset: to reset port statistics\n"
                "  --xstats-reset: to reset port extended statistics\n"
                "  --collectd-format: to print statistics to STDOUT in expected by collectd format\n"
@@ -132,6 +113,33 @@ parse_portmask(const char *portmask)
 
 }
 
+/*
+ * Parse ids value list into array
+ */
+static int
+parse_xstats_ids(char *list, uint64_t *ids, int limit) {
+       int length;
+       char *token;
+       char *ctx = NULL;
+       char *endptr;
+
+       length = 0;
+       token = strtok_r(list, ",", &ctx);
+       while (token != NULL) {
+               ids[length] = strtoull(token, &endptr, 10);
+               if (*endptr != '\0')
+                       return -EINVAL;
+
+               length++;
+               if (length >= limit)
+                       return -E2BIG;
+
+               token = strtok_r(NULL, ",", &ctx);
+       }
+
+       return length;
+}
+
 static int
 proc_info_preparse_args(int argc, char **argv)
 {
@@ -178,7 +186,9 @@ proc_info_parse_args(int argc, char **argv)
                {"xstats", 0, NULL, 0},
                {"metrics", 0, NULL, 0},
                {"xstats-reset", 0, NULL, 0},
+               {"xstats-name", required_argument, NULL, 1},
                {"collectd-format", 0, NULL, 0},
+               {"xstats-ids", 1, NULL, 1},
                {"host-id", 0, NULL, 0},
                {NULL, 0, 0, 0}
        };
@@ -224,7 +234,28 @@ proc_info_parse_args(int argc, char **argv)
                                        MAX_LONG_OPT_SZ))
                                reset_xstats = 1;
                        break;
+               case 1:
+                       /* Print xstat single value given by name*/
+                       if (!strncmp(long_option[option_index].name,
+                                       "xstats-name", MAX_LONG_OPT_SZ)) {
+                               enable_xstats_name = 1;
+                               xstats_name = optarg;
+                               printf("name:%s:%s\n",
+                                               long_option[option_index].name,
+                                               optarg);
+                       } else if (!strncmp(long_option[option_index].name,
+                                       "xstats-ids",
+                                       MAX_LONG_OPT_SZ))       {
+                               nb_xstats_ids = parse_xstats_ids(optarg,
+                                               xstats_ids, MAX_NB_XSTATS_IDS);
+
+                               if (nb_xstats_ids <= 0) {
+                                       printf("xstats-id list parse error.\n");
+                                       return -1;
+                               }
 
+                       }
+                       break;
                default:
                        proc_info_usage(prgname);
                        return -1;
@@ -250,7 +281,7 @@ meminfo_display(void)
 }
 
 static void
-nic_stats_display(uint8_t port_id)
+nic_stats_display(uint16_t port_id)
 {
        struct rte_eth_stats stats;
        uint8_t i;
@@ -289,7 +320,7 @@ nic_stats_display(uint8_t port_id)
 }
 
 static void
-nic_stats_clear(uint8_t port_id)
+nic_stats_clear(uint16_t port_id)
 {
        printf("\n Clearing NIC stats for port %d\n", port_id);
        rte_eth_stats_reset(port_id);
@@ -351,20 +382,82 @@ static void collectd_resolve_cnt_type(char *cnt_type, size_t cnt_type_len,
 }
 
 static void
-nic_xstats_display(uint8_t port_id)
+nic_xstats_by_name_display(uint16_t port_id, char *name)
+{
+       uint64_t id;
+
+       printf("###### NIC statistics for port %-2d, statistic name '%s':\n",
+                          port_id, name);
+
+       if (rte_eth_xstats_get_id_by_name(port_id, name, &id) == 0)
+               printf("%s: %"PRIu64"\n", name, id);
+       else
+               printf("Statistic not found...\n");
+
+}
+
+static void
+nic_xstats_by_ids_display(uint16_t port_id, uint64_t *ids, int len)
 {
        struct rte_eth_xstat_name *xstats_names;
-       struct rte_eth_xstat *xstats;
+       uint64_t *values;
+       int ret, i;
+       static const char *nic_stats_border = "########################";
+
+       values = malloc(sizeof(*values) * len);
+       if (values == NULL) {
+               printf("Cannot allocate memory for xstats\n");
+               return;
+       }
+
+       xstats_names = malloc(sizeof(struct rte_eth_xstat_name) * len);
+       if (xstats_names == NULL) {
+               printf("Cannot allocate memory for xstat names\n");
+               free(values);
+               return;
+       }
+
+       if (len != rte_eth_xstats_get_names_by_id(
+                       port_id, xstats_names, len, ids)) {
+               printf("Cannot get xstat names\n");
+               goto err;
+       }
+
+       printf("###### NIC extended statistics for port %-2d #########\n",
+                          port_id);
+       printf("%s############################\n", nic_stats_border);
+       ret = rte_eth_xstats_get_by_id(port_id, ids, values, len);
+       if (ret < 0 || ret > len) {
+               printf("Cannot get xstats\n");
+               goto err;
+       }
+
+       for (i = 0; i < len; i++)
+               printf("%s: %"PRIu64"\n",
+                       xstats_names[i].name,
+                       values[i]);
+
+       printf("%s############################\n", nic_stats_border);
+err:
+       free(values);
+       free(xstats_names);
+}
+
+static void
+nic_xstats_display(uint16_t port_id)
+{
+       struct rte_eth_xstat_name *xstats_names;
+       uint64_t *values;
        int len, ret, i;
        static const char *nic_stats_border = "########################";
 
-       len = rte_eth_xstats_get_names(port_id, NULL, 0);
+       len = rte_eth_xstats_get_names_by_id(port_id, NULL, 0, NULL);
        if (len < 0) {
                printf("Cannot get xstats count\n");
                return;
        }
-       xstats = malloc(sizeof(xstats[0]) * len);
-       if (xstats == NULL) {
+       values = malloc(sizeof(*values) * len);
+       if (values == NULL) {
                printf("Cannot allocate memory for xstats\n");
                return;
        }
@@ -372,11 +465,11 @@ nic_xstats_display(uint8_t port_id)
        xstats_names = malloc(sizeof(struct rte_eth_xstat_name) * len);
        if (xstats_names == NULL) {
                printf("Cannot allocate memory for xstat names\n");
-               free(xstats);
+               free(values);
                return;
        }
-       if (len != rte_eth_xstats_get_names(
-                       port_id, xstats_names, len)) {
+       if (len != rte_eth_xstats_get_names_by_id(
+                       port_id, xstats_names, len, NULL)) {
                printf("Cannot get xstat names\n");
                goto err;
        }
@@ -385,7 +478,7 @@ nic_xstats_display(uint8_t port_id)
                           port_id);
        printf("%s############################\n",
                           nic_stats_border);
-       ret = rte_eth_xstats_get(port_id, xstats, len);
+       ret = rte_eth_xstats_get_by_id(port_id, NULL, values, len);
        if (ret < 0 || ret > len) {
                printf("Cannot get xstats\n");
                goto err;
@@ -401,23 +494,25 @@ nic_xstats_display(uint8_t port_id)
                                                  xstats_names[i].name);
                        sprintf(buf, "PUTVAL %s/dpdkstat-port.%u/%s-%s N:%"
                                PRIu64"\n", host_id, port_id, counter_type,
-                               xstats_names[i].name, xstats[i].value);
-                       write(stdout_fd, buf, strlen(buf));
+                               xstats_names[i].name, values[i]);
+                       ret = write(stdout_fd, buf, strlen(buf));
+                       if (ret < 0)
+                               goto err;
                } else {
                        printf("%s: %"PRIu64"\n", xstats_names[i].name,
-                              xstats[i].value);
+                                       values[i]);
                }
        }
 
        printf("%s############################\n",
                           nic_stats_border);
 err:
-       free(xstats);
+       free(values);
        free(xstats_names);
 }
 
 static void
-nic_xstats_clear(uint8_t port_id)
+nic_xstats_clear(uint16_t port_id)
 {
        printf("\n Clearing NIC xstats for port %d\n", port_id);
        rte_eth_xstats_reset(port_id);
@@ -494,7 +589,7 @@ main(int argc, char **argv)
        char n_flag[] = "-n4";
        char mp_flag[] = "--proc-type=secondary";
        char *argp[argc + 3];
-       uint8_t nb_ports;
+       uint16_t nb_ports;
 
        /* preparse app arguments */
        ret = proc_info_preparse_args(argc, argv);
@@ -551,6 +646,11 @@ main(int argc, char **argv)
                                nic_stats_clear(i);
                        else if (reset_xstats)
                                nic_xstats_clear(i);
+                       else if (enable_xstats_name)
+                               nic_xstats_by_name_display(i, xstats_name);
+                       else if (nb_xstats_ids > 0)
+                               nic_xstats_by_ids_display(i, xstats_ids,
+                                               nb_xstats_ids);
                        else if (enable_metrics)
                                metrics_display(i);
                }
@@ -560,5 +660,9 @@ main(int argc, char **argv)
        if (enable_metrics)
                metrics_display(RTE_METRICS_GLOBAL);
 
+       ret = rte_eal_cleanup();
+       if (ret)
+               printf("Error from rte_eal_cleanup(), %d\n", ret);
+
        return 0;
 }