All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Monjalon <thomas@monjalon.net>
To: Bruce Richardson <bruce.richardson@intel.com>
Cc: Kevin Laatz <kevin.laatz@intel.com>,
	ciara.power@intel.com, dev@dpdk.org, david.marchand@redhat.com
Subject: Re: [dpdk-dev] telemetry logs
Date: Tue, 02 Mar 2021 13:40:39 +0100	[thread overview]
Message-ID: <18349908.mb9rSiqodV@thomas> (raw)
In-Reply-To: <20210302110630.GD1396@bricha3-MOBL.ger.corp.intel.com>

02/03/2021 12:18, Bruce Richardson:
> On Tue, Mar 02, 2021 at 12:01:12PM +0100, Thomas Monjalon wrote:
> > 02/03/2021 11:42, Bruce Richardson:
> > > On Tue, Mar 02, 2021 at 11:23:41AM +0100, Thomas Monjalon wrote:
> > > > 02/03/2021 11:14, Bruce Richardson:
> > > > > On Mon, Mar 01, 2021 at 08:22:15PM +0100, Thomas Monjalon wrote:
> > > > > > Hi,
> > > > > > Sorry if I already asked this question.
> > > > > > Would it be possible to use rte_log in rte_telemetry
> > > > > > instead of returning telemetry_log_error at init?
> > > > > > 
> > > > > It probably could be made to work by passing in the log function at init
> > > > > time. Haven't delved too much into the details, though.
> > > > 
> > > > Actually I think a better question is about the need to init telemetry
> > > > if not used. It can generate an error without a need.
> > > > Instead of the existing option --no-telemetry,
> > > > what do you think of initializing the telemetry on its first use?
> > > > This way we could remove the dependency of EAL on telemetry?
> > > > 
> > > There is no real first use - the various libraries all register their callbacks
> > > inside the init functions of the shared libraries. Having it initialized
> > > inside EAL makes things very useful, because it means that all DPDK apps
> > > automatically have telemetry available.
> > 
> > Registering libs should be always possible, yes.
> > But the init of the socket can be deffered to its use, no?
> >
> Not sure what you mean here? When would you see the socket init taking
> place if not in EAL init?

I think my request is just impossible.
My wish is that telemetry socket is initialized only when
we need it. But no, I don't know how to detect a socket query
before creating the socket...

> Of course, the other alternative to this problem is a solution you were
> previously suggesting: to move basic services such as logging to a separate
> library so that we split EAL into two parts, with the init being in the
> second higher-level part. Obviously this is a very significant amount of
> work though, so unlikely to be undertaken quickly.

I agree it would be an interesting and huge work.




      reply	other threads:[~2021-03-02 12:40 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-03-01 19:22 [dpdk-dev] telemetry logs Thomas Monjalon
2021-03-02 10:14 ` Bruce Richardson
2021-03-02 10:23   ` Thomas Monjalon
2021-03-02 10:42     ` Bruce Richardson
2021-03-02 11:01       ` Thomas Monjalon
2021-03-02 11:18         ` Bruce Richardson
2021-03-02 12:40           ` Thomas Monjalon [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=18349908.mb9rSiqodV@thomas \
    --to=thomas@monjalon.net \
    --cc=bruce.richardson@intel.com \
    --cc=ciara.power@intel.com \
    --cc=david.marchand@redhat.com \
    --cc=dev@dpdk.org \
    --cc=kevin.laatz@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.