[PATCH v4 12/14] lib: convert to per line logging
David Marchand
david.marchand at redhat.com
Wed Dec 20 15:00:09 CET 2023
On Wed, Dec 20, 2023 at 2:46 PM Thomas Monjalon <thomas at monjalon.net> wrote:
>
> 18/12/2023 15:38, David Marchand:
> > Convert many libraries that call RTE_LOG(... "\n", ...) to RTE_LOG_LINE.
> >
> > Note:
> > - for acl and sched libraries that still has some debug multilines
> > messages, a direct call to RTE_LOG is used: this will make it easier to
> > notice such special cases,
>
> [...]
> > 129 files changed, 2280 insertions(+), 2279 deletions(-)
>
> It would be nice to keep RTE_LOG() only for multi-line as you do,
> and use RTE_LOG_LINE only in wrapper macros specific per library.
> This patch is using RTE_LOG_LINE directly in multiple places
> in libraries which do not have a dedicated wrapper macro.
> Would it be possible to introduce new wrappers for these libraries
> in this patch?
It is doable and not too difficult to achieve with some scripting.
I'll try in the v5 (since I need to send a new revision for build with
-pedantic).
--
David Marchand
More information about the dev
mailing list