On 12/8/23 17:59, David Marchand wrote:
This is a preparation step before the next change.

Many libraries have their own logging helpers that do not add a newline
in their format string.
Some previous changes fixed places where some of those helpers are
called without a trailing newline.
Using RTE_LOG_LINE in the existing helpers will ensure we don't
introduce new issues in the future.

The problem is that if we simply convert to the RTE_LOG_LINE helper,
a future fix may introduce a regression since the logging helper
change won't be backported.

To address this concern, rename existing helpers: backporting a call to
them will trigger some conflict or build issue in LTS branches.

Note:
- bpf and vhost that still has some debug multilines messages, a direct
   call to RTE_LOG/RTE_LOG_DP is used: this will make it easier to notice
   such special cases,
- about previously publicly exposed logging helpers, when such helper is
   not publicly used (iow in public inline API), it is removed from the
   public API (this is the case for the member library),

Signed-off-by: David Marchand <david.march...@redhat.com>

For ethdev
Reviewed-by: Andrew Rybchenko <andrew.rybche...@oktetlabs.ru>


Reply via email to