DPDK patches and discussions
 help / color / mirror / Atom feed
* testpmd logging
@ 2022-11-08 13:02 Thomas Monjalon
  2022-11-08 14:37 ` Honnappa Nagarahalli
  0 siblings, 1 reply; 5+ messages in thread
From: Thomas Monjalon @ 2022-11-08 13:02 UTC (permalink / raw)
  To: Aman Singh, Yuying Zhang; +Cc: david.marchand, dev

Hello,

I see testpmd is doing log with EAL entity. Example:
	RTE_LOG(ERR, EAL, "The device: %s has been added!\n"
We could also discuss the log level.

Please can we work on replacing these logs?
We need to choose how to log (printf or registered log entity)
and be consistent.
What do you think?



^ permalink raw reply	[flat|nested] 5+ messages in thread

* RE: testpmd logging
  2022-11-08 13:02 testpmd logging Thomas Monjalon
@ 2022-11-08 14:37 ` Honnappa Nagarahalli
  2022-11-08 16:24   ` Owen Hilyard
  0 siblings, 1 reply; 5+ messages in thread
From: Honnappa Nagarahalli @ 2022-11-08 14:37 UTC (permalink / raw)
  To: thomas, Aman Singh, Yuying Zhang
  Cc: david.marchand, dev, Lijuan Tu, Owen Hilyard, Juraj Linkeš, nd, nd

+ Lijuan, Owen, Juraj

> -----Original Message-----
> From: Thomas Monjalon <thomas@monjalon.net>
> Sent: Tuesday, November 8, 2022 7:02 AM
> To: Aman Singh <aman.deep.singh@intel.com>; Yuying Zhang
> <yuying.zhang@intel.com>
> Cc: david.marchand@redhat.com; dev@dpdk.org
> Subject: testpmd logging
> 
> Hello,
> 
> I see testpmd is doing log with EAL entity. Example:
> 	RTE_LOG(ERR, EAL, "The device: %s has been added!\n"
> We could also discuss the log level.
> 
> Please can we work on replacing these logs?
DTS uses some of the logs from testpmd to validate the results. I am not sure what all logs are used.

> We need to choose how to log (printf or registered log entity) and be
> consistent.
> What do you think?
> 


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: testpmd logging
  2022-11-08 14:37 ` Honnappa Nagarahalli
@ 2022-11-08 16:24   ` Owen Hilyard
  2022-11-08 16:40     ` Stephen Hemminger
  0 siblings, 1 reply; 5+ messages in thread
From: Owen Hilyard @ 2022-11-08 16:24 UTC (permalink / raw)
  To: Honnappa Nagarahalli
  Cc: thomas, Aman Singh, Yuying Zhang, david.marchand, dev, Lijuan Tu,
	Juraj Linkeš,
	nd

[-- Attachment #1: Type: text/plain, Size: 1566 bytes --]

On Tue, Nov 8, 2022 at 9:37 AM Honnappa Nagarahalli <
Honnappa.Nagarahalli@arm.com> wrote:

> + Lijuan, Owen, Juraj
>
> > -----Original Message-----
> > From: Thomas Monjalon <thomas@monjalon.net>
> > Sent: Tuesday, November 8, 2022 7:02 AM
> > To: Aman Singh <aman.deep.singh@intel.com>; Yuying Zhang
> > <yuying.zhang@intel.com>
> > Cc: david.marchand@redhat.com; dev@dpdk.org
> > Subject: testpmd logging
> >
> > Hello,
> >
> > I see testpmd is doing log with EAL entity. Example:
> >       RTE_LOG(ERR, EAL, "The device: %s has been added!\n"
> > We could also discuss the log level.
> >
> > Please can we work on replacing these logs?
>

If we want to revisit logging in testpmd, even though I know I'm invoking a
case of second system syndrome here, I would love to see some form of
structured logging output. Even just writing out records to a CSV file with
proper escaping would work great, or doing a json object per log record as
a backup. JSON might be easier since libjansson is already in DPDK, and we
could disable the json logging flag if libjansson was not present at
compile time.


> DTS uses some of the logs from testpmd to validate the results. I am not
> sure what all logs are used.


The old version of dts (the one with its own repository) is fairly tightly
bound to testpmd's logging.

> We need to choose how to log (printf or registered log entity) and be
> > consistent.
> > What do you think?
>

Registered log entries are much better in my opinion since they provide a
more machine-friendly format and DTS has to parse testpmd output.

[-- Attachment #2: Type: text/html, Size: 2706 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: testpmd logging
  2022-11-08 16:24   ` Owen Hilyard
@ 2022-11-08 16:40     ` Stephen Hemminger
  2022-11-11  8:26       ` Zhang, Yuying
  0 siblings, 1 reply; 5+ messages in thread
From: Stephen Hemminger @ 2022-11-08 16:40 UTC (permalink / raw)
  To: Owen Hilyard
  Cc: Honnappa Nagarahalli, thomas, Aman Singh, Yuying Zhang,
	david.marchand, dev, Lijuan Tu, Juraj Linkeš,
	nd

On Tue, 8 Nov 2022 11:24:53 -0500
Owen Hilyard <ohilyard@iol.unh.edu> wrote:

> On Tue, Nov 8, 2022 at 9:37 AM Honnappa Nagarahalli <
> Honnappa.Nagarahalli@arm.com> wrote:  
> 
> > + Lijuan, Owen, Juraj
> >  
> > > -----Original Message-----
> > > From: Thomas Monjalon <thomas@monjalon.net>
> > > Sent: Tuesday, November 8, 2022 7:02 AM
> > > To: Aman Singh <aman.deep.singh@intel.com>; Yuying Zhang
> > > <yuying.zhang@intel.com>
> > > Cc: david.marchand@redhat.com; dev@dpdk.org
> > > Subject: testpmd logging
> > >
> > > Hello,
> > >
> > > I see testpmd is doing log with EAL entity. Example:
> > >       RTE_LOG(ERR, EAL, "The device: %s has been added!\n"
> > > We could also discuss the log level.
> > >
> > > Please can we work on replacing these logs?  
> >  
> 
> If we want to revisit logging in testpmd, even though I know I'm invoking a
> case of second system syndrome here, I would love to see some form of
> structured logging output. Even just writing out records to a CSV file with
> proper escaping would work great, or doing a json object per log record as
> a backup. JSON might be easier since libjansson is already in DPDK, and we
> could disable the json logging flag if libjansson was not present at
> compile time.
> 
> 
> > DTS uses some of the logs from testpmd to validate the results. I am not
> > sure what all logs are used.  
> 
> 
> The old version of dts (the one with its own repository) is fairly tightly
> bound to testpmd's logging.
> 
> > We need to choose how to log (printf or registered log entity) and be  
> > > consistent.
> > > What do you think?  
> >  
> 
> Registered log entries are much better in my opinion since they provide a
> more machine-friendly format and DTS has to parse testpmd output.

My thoughts.
  1. testpmd should use its own log level, not EAL
  2. testpmd could easily have an option "--json" to format output in JSON
  3. testpmd is a mess some logs use RTE_LOG() some go to stdout, some to stderr.


^ permalink raw reply	[flat|nested] 5+ messages in thread

* RE: testpmd logging
  2022-11-08 16:40     ` Stephen Hemminger
@ 2022-11-11  8:26       ` Zhang, Yuying
  0 siblings, 0 replies; 5+ messages in thread
From: Zhang, Yuying @ 2022-11-11  8:26 UTC (permalink / raw)
  To: Stephen Hemminger, Owen Hilyard
  Cc: Honnappa Nagarahalli, thomas, Singh, Aman Deep, david.marchand,
	dev, Tu, Lijuan, Juraj Linkeš,
	nd



> -----Original Message-----
> From: Stephen Hemminger <stephen@networkplumber.org>
> Sent: 2022年11月9日 0:40
> To: Owen Hilyard <ohilyard@iol.unh.edu>
> Cc: Honnappa Nagarahalli <Honnappa.Nagarahalli@arm.com>;
> thomas@monjalon.net; Singh, Aman Deep <aman.deep.singh@intel.com>;
> Zhang, Yuying <yuying.zhang@intel.com>; david.marchand@redhat.com;
> dev@dpdk.org; Tu, Lijuan <lijuan.tu@intel.com>; Juraj Linkeš
> <juraj.linkes@pantheon.tech>; nd <nd@arm.com>
> Subject: Re: testpmd logging
> 
> On Tue, 8 Nov 2022 11:24:53 -0500
> Owen Hilyard <ohilyard@iol.unh.edu> wrote:
> 
> > On Tue, Nov 8, 2022 at 9:37 AM Honnappa Nagarahalli <
> > Honnappa.Nagarahalli@arm.com> wrote:
> >
> > > + Lijuan, Owen, Juraj
> > >
> > > > -----Original Message-----
> > > > From: Thomas Monjalon <thomas@monjalon.net>
> > > > Sent: Tuesday, November 8, 2022 7:02 AM
> > > > To: Aman Singh <aman.deep.singh@intel.com>; Yuying Zhang
> > > > <yuying.zhang@intel.com>
> > > > Cc: david.marchand@redhat.com; dev@dpdk.org
> > > > Subject: testpmd logging
> > > >
> > > > Hello,
> > > >
> > > > I see testpmd is doing log with EAL entity. Example:
> > > >       RTE_LOG(ERR, EAL, "The device: %s has been added!\n"
> > > > We could also discuss the log level.
> > > >
> > > > Please can we work on replacing these logs?
> > >
> >
> > If we want to revisit logging in testpmd, even though I know I'm
> > invoking a case of second system syndrome here, I would love to see
> > some form of structured logging output. Even just writing out records
> > to a CSV file with proper escaping would work great, or doing a json
> > object per log record as a backup. JSON might be easier since
> > libjansson is already in DPDK, and we could disable the json logging
> > flag if libjansson was not present at compile time.
> >
> >
> > > DTS uses some of the logs from testpmd to validate the results. I am
> > > not sure what all logs are used.
> >
> >
> > The old version of dts (the one with its own repository) is fairly
> > tightly bound to testpmd's logging.
> >
> > > We need to choose how to log (printf or registered log entity) and
> > > be
> > > > consistent.
> > > > What do you think?
> > >
> >
> > Registered log entries are much better in my opinion since they
> > provide a more machine-friendly format and DTS has to parse testpmd output.
> 
> My thoughts.
>   1. testpmd should use its own log level, not EAL
>   2. testpmd could easily have an option "--json" to format output in JSON
>   3. testpmd is a mess some logs use RTE_LOG() some go to stdout, some to
> stderr.

The same as Hemminger. testpmd should refine log functions then.


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2022-11-11  8:26 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-11-08 13:02 testpmd logging Thomas Monjalon
2022-11-08 14:37 ` Honnappa Nagarahalli
2022-11-08 16:24   ` Owen Hilyard
2022-11-08 16:40     ` Stephen Hemminger
2022-11-11  8:26       ` Zhang, Yuying

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).