Re: [PATCH] trace: take live traces via telemetry

2022-10-19 Thread David Marchand
On Wed, Oct 19, 2022 at 12:53 PM Bruce Richardson wrote: > > On Tue, Oct 18, 2022 at 02:14:37PM +0100, Bruce Richardson wrote: > > On Thu, Oct 13, 2022 at 09:49:28AM +0200, David Marchand wrote: > > > Register telemetry commands to list and configure trace points and later > > > save traces for a

Re: [PATCH] trace: take live traces via telemetry

2022-10-19 Thread Bruce Richardson
On Tue, Oct 18, 2022 at 02:14:37PM +0100, Bruce Richardson wrote: > On Thu, Oct 13, 2022 at 09:49:28AM +0200, David Marchand wrote: > > Register telemetry commands to list and configure trace points and later > > save traces for a running DPDK application. > > > > Note: trace point names contain a

Re: [PATCH] trace: take live traces via telemetry

2022-10-19 Thread David Marchand
On Wed, Oct 19, 2022 at 10:21 AM Morten Brørup wrote: > > > It seems to me that these values are Boolean, and should be true or > > false (not surrounded by quotation marks), instead of some string > > representing a Boolean value. Note: This would require expanding the > > telemetry library with

RE: [PATCH] trace: take live traces via telemetry

2022-10-19 Thread Morten Brørup
> From: David Marchand [mailto:david.march...@redhat.com] > Sent: Wednesday, 19 October 2022 09.39 > > Hello Morten, Bruce, > > On Tue, Oct 18, 2022 at 4:34 PM Morten Brørup > wrote: > > > --> /trace/list,lib.ethdev.* > > > {"/trace/list": {"lib.ethdev.configure": "Disabled", > > > "lib.ethd

Re: [PATCH] trace: take live traces via telemetry

2022-10-19 Thread David Marchand
Hello Morten, Bruce, On Tue, Oct 18, 2022 at 4:34 PM Morten Brørup wrote: > > --> /trace/list,lib.ethdev.* > > {"/trace/list": {"lib.ethdev.configure": "Disabled", > > "lib.ethdev.rxq.setup": "Disabled", > > "lib.ethdev.txq.setup": "Disabled", > > "lib.ethdev.start": "Disabled", > >

Re: [PATCH] trace: take live traces via telemetry

2022-10-18 Thread Bruce Richardson
On Tue, Oct 18, 2022 at 04:33:49PM +0200, Morten Brørup wrote: > > From: David Marchand [mailto:david.march...@redhat.com] > > Sent: Thursday, 13 October 2022 09.49 > > > > Register telemetry commands to list and configure trace points and > > later > > save traces for a running DPDK application.

RE: [PATCH] trace: take live traces via telemetry

2022-10-18 Thread Morten Brørup
> From: David Marchand [mailto:david.march...@redhat.com] > Sent: Thursday, 13 October 2022 09.49 > > Register telemetry commands to list and configure trace points and > later > save traces for a running DPDK application. > > Note: trace point names contain a '.', so the list of valid characters

Re: [PATCH] trace: take live traces via telemetry

2022-10-18 Thread Bruce Richardson
On Thu, Oct 13, 2022 at 09:49:28AM +0200, David Marchand wrote: > Register telemetry commands to list and configure trace points and later > save traces for a running DPDK application. > > Note: trace point names contain a '.', so the list of valid characters > used in telemetry commands and dicti

Re: [PATCH] trace: take live traces via telemetry

2022-10-13 Thread Jerin Jacob
On Thu, Oct 13, 2022 at 1:20 PM David Marchand wrote: > I would suggest to change the subject as "trace: enable trace operations via telemetry" or so > Register telemetry commands to list and configure trace points and later > save traces for a running DPDK application. > > Note: trace point nam

[PATCH] trace: take live traces via telemetry

2022-10-13 Thread David Marchand
Register telemetry commands to list and configure trace points and later save traces for a running DPDK application. Note: trace point names contain a '.', so the list of valid characters used in telemetry commands and dictionary keys has been extended. Example with testpmd running with two net/n