[v2] examples/ioat: fix stats print
Checks
Commit Message
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")
Signed-off-by: Kevin Laatz <kevin.laatz@intel.com>
---
v2:
- Increased string array by larger value - ie. up to 255.
- Moved the "\n" to avoid future formatting issues from truncation.
---
examples/ioat/ioatfwd.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
Comments
On Thu, Sep 17, 2020 at 02:07:40PM +0100, Kevin Laatz wrote:
> 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")
>
> Signed-off-by: Kevin Laatz <kevin.laatz@intel.com>
>
Acked-by: Bruce Richardson <bruce.richardson@intel.com>
17/09/2020 15:21, Bruce Richardson:
> On Thu, Sep 17, 2020 at 02:07:40PM +0100, Kevin Laatz wrote:
> > 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 <kevin.laatz@intel.com>
> >
> Acked-by: Bruce Richardson <bruce.richardson@intel.com>
Applied, thanks
@@ -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;