On 6/18/21 11:32 AM, Li, Xiaoyun wrote:

quoted

-----Original Message-----
From: Andrew Rybchenko <redacted>
Sent: Thursday, June 17, 2021 22:20
To: Li, Xiaoyun <redacted>; Ori Kam <redacted>
Cc: dev@dpdk.org; Richardson, Bruce <redacted>; Yigit,
Ferruh [off-list ref]; Singh, Aman Deep
[off-list ref]
Subject: [PATCH v3] app/testpmd: send failure logs to stderr

Running with stdout suppressed or redirected for further processing
is very confusing in the case of errors. Fix it by logging errors and
warnings to stderr.

Since lines with log messages are touched anyway concatenate split
format strings to make it easier to search using grep.

Fix indent of format string arguments.

Signed-off-by: Andrew Rybchenko <redacted>
---
v3:
  - fix typo in the patch description
  - fix applicable coding style issues reported by patchwork

v2:
  - switch from printf() to fpritnf(stderr, ...) in more cases
  - do not inherit acks from the previous version since the patch is
    much bigger
  - fix style in few cases (TAB vs spaces, missing space separtor etc)
  - still don't use TESTPMD_LOG() since the patch does not add new logs.
    Also switching to TESTPMD_LOG() will add "testpmd: " prefix to log
    messages and it is a real change and could be a pain for automation.

  app/test-pmd/bpf_cmd.c                 |   6 +-
  app/test-pmd/cmdline.c                 | 957 ++++++++++++++-----------
  app/test-pmd/cmdline_flow.c            |  20 +-
  app/test-pmd/cmdline_mtr.c             |   8 +-
  app/test-pmd/cmdline_tm.c              |  33 +-
  app/test-pmd/config.c                  | 455 ++++++------
  app/test-pmd/csumonly.c                |   5 +-
  app/test-pmd/parameters.c              |  21 +-
  app/test-pmd/testpmd.c                 | 298 ++++----
  app/test-pmd/util.c                    |  19 +-
  doc/guides/rel_notes/release_21_08.rst |   5 +
  11 files changed, 1004 insertions(+), 823 deletions(-)

Acked-by: Xiaoyun Li <redacted>

It looks like Mellanox test failures are unrelated to the patch.

There are a number of patches in queue. Which order is the right now?
Apply this one and require appropriate changes in other testpmd patches?
Or vise versa?