From 64d0a9097d96e554697f8473f196266bfaf62b9d Mon Sep 17 00:00:00 2001 From: Kevin Laatz Date: Thu, 17 Sep 2020 14:07:40 +0100 Subject: [PATCH] examples/ioat: fix stats print Currently some of the status string at the top of the stats output is being cut off. To fix this, the status string array size has been increased. In addition to this, the "\n" has been moved to the printf, rather than having it in the last string, in case of future formatting issues due to truncation. Bugzilla ID: 536 Fixes: 632bcd9b5d4f ("examples/ioat: print statistics") Cc: stable@dpdk.org Signed-off-by: Kevin Laatz Acked-by: Bruce Richardson --- examples/ioat/ioatfwd.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/examples/ioat/ioatfwd.c b/examples/ioat/ioatfwd.c index e6d1d1236e..8cf606e25b 100644 --- a/examples/ioat/ioatfwd.c +++ b/examples/ioat/ioatfwd.c @@ -168,7 +168,7 @@ print_stats(char *prgname) struct rte_rawdev_xstats_name *names_xstats; uint64_t *xstats; unsigned int *ids_xstats, nb_xstats; - char status_string[120]; /* to print at the top of the output */ + char status_string[255]; /* to print at the top of the output */ int status_strlen; int ret; @@ -194,7 +194,7 @@ print_stats(char *prgname) "Rx Queues = %d, ", nb_queues); status_strlen += snprintf(status_string + status_strlen, sizeof(status_string) - status_strlen, - "Ring Size = %d\n", ring_size); + "Ring Size = %d", ring_size); /* Allocate memory for xstats names and values */ ret = rte_rawdev_xstats_names_get( @@ -251,7 +251,7 @@ print_stats(char *prgname) memset(&delta_ts, 0, sizeof(struct total_statistics)); - printf("%s", status_string); + printf("%s\n", status_string); for (i = 0; i < cfg.nb_ports; i++) { port_id = cfg.ports[i].rxtx_port; -- 2.20.1