patches for DPDK stable branches
 help / color / mirror / Atom feed
* [dpdk-stable] [PATCH] doc: fix incorrect example for specifying log level
@ 2020-03-17  8:31 Xiaolong Ye
  2020-03-17 17:18 ` [dpdk-stable] [dpdk-dev] " Stephen Hemminger
  2020-03-18  0:58 ` [dpdk-stable] [PATCH v2] " Xiaolong Ye
  0 siblings, 2 replies; 5+ messages in thread
From: Xiaolong Ye @ 2020-03-17  8:31 UTC (permalink / raw)
  To: John McNamara, Marko Kovacevic; +Cc: dev, Xiaolong Ye, stable

Now we need to add prefix like lib. to enable the log.

Fixes: ffb9fd1b0808 ("log: update legacy modules dynamic logs regex")
Cc: stable@dpdk.org

Reported-by: Haiyue Wang <haiyue.wang@intel.com>
Signed-off-by: Xiaolong Ye <xiaolong.ye@intel.com>
---
 doc/guides/linux_gsg/eal_args.include.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/doc/guides/linux_gsg/eal_args.include.rst b/doc/guides/linux_gsg/eal_args.include.rst
index ed8b0e35b..b48b23927 100644
--- a/doc/guides/linux_gsg/eal_args.include.rst
+++ b/doc/guides/linux_gsg/eal_args.include.rst
@@ -132,7 +132,7 @@ Debugging options
 
     Specify log level for a specific component. For example::
 
-        --log-level eal:8
+        --log-level lib.eal:8
 
     Can be specified multiple times.
 
-- 
2.17.1


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

* Re: [dpdk-stable] [dpdk-dev] [PATCH] doc: fix incorrect example for specifying log level
  2020-03-17  8:31 [dpdk-stable] [PATCH] doc: fix incorrect example for specifying log level Xiaolong Ye
@ 2020-03-17 17:18 ` Stephen Hemminger
  2020-03-18  0:57   ` Ye Xiaolong
  2020-03-18  0:58 ` [dpdk-stable] [PATCH v2] " Xiaolong Ye
  1 sibling, 1 reply; 5+ messages in thread
From: Stephen Hemminger @ 2020-03-17 17:18 UTC (permalink / raw)
  To: Xiaolong Ye; +Cc: John McNamara, Marko Kovacevic, dev, stable

On Tue, 17 Mar 2020 16:31:06 +0800
Xiaolong Ye <xiaolong.ye@intel.com> wrote:

> Now we need to add prefix like lib. to enable the log.
> 
> Fixes: ffb9fd1b0808 ("log: update legacy modules dynamic logs regex")
> Cc: stable@dpdk.org
> 
> Reported-by: Haiyue Wang <haiyue.wang@intel.com>
> Signed-off-by: Xiaolong Ye <xiaolong.ye@intel.com>
> ---
>  doc/guides/linux_gsg/eal_args.include.rst | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/doc/guides/linux_gsg/eal_args.include.rst b/doc/guides/linux_gsg/eal_args.include.rst
> index ed8b0e35b..b48b23927 100644
> --- a/doc/guides/linux_gsg/eal_args.include.rst
> +++ b/doc/guides/linux_gsg/eal_args.include.rst
> @@ -132,7 +132,7 @@ Debugging options
>  
>      Specify log level for a specific component. For example::
>  
> -        --log-level eal:8
> +        --log-level lib.eal:8
>  
>      Can be specified multiple times.
>  

Please change numeric value to name as well. Recent versions of
DPDK allow "debug" instead of 8

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

* Re: [dpdk-stable] [dpdk-dev] [PATCH] doc: fix incorrect example for specifying log level
  2020-03-17 17:18 ` [dpdk-stable] [dpdk-dev] " Stephen Hemminger
@ 2020-03-18  0:57   ` Ye Xiaolong
  0 siblings, 0 replies; 5+ messages in thread
From: Ye Xiaolong @ 2020-03-18  0:57 UTC (permalink / raw)
  To: Stephen Hemminger; +Cc: John McNamara, Marko Kovacevic, dev, stable

On 03/17, Stephen Hemminger wrote:
>On Tue, 17 Mar 2020 16:31:06 +0800
>Xiaolong Ye <xiaolong.ye@intel.com> wrote:
>
>> Now we need to add prefix like lib. to enable the log.
>> 
>> Fixes: ffb9fd1b0808 ("log: update legacy modules dynamic logs regex")
>> Cc: stable@dpdk.org
>> 
>> Reported-by: Haiyue Wang <haiyue.wang@intel.com>
>> Signed-off-by: Xiaolong Ye <xiaolong.ye@intel.com>
>> ---
>>  doc/guides/linux_gsg/eal_args.include.rst | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>> 
>> diff --git a/doc/guides/linux_gsg/eal_args.include.rst b/doc/guides/linux_gsg/eal_args.include.rst
>> index ed8b0e35b..b48b23927 100644
>> --- a/doc/guides/linux_gsg/eal_args.include.rst
>> +++ b/doc/guides/linux_gsg/eal_args.include.rst
>> @@ -132,7 +132,7 @@ Debugging options
>>  
>>      Specify log level for a specific component. For example::
>>  
>> -        --log-level eal:8
>> +        --log-level lib.eal:8
>>  
>>      Can be specified multiple times.
>>  
>
>Please change numeric value to name as well. Recent versions of
>DPDK allow "debug" instead of 8

Make sense, will update it.

Thanks,
Xiaolong

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

* [dpdk-stable] [PATCH v2] doc: fix incorrect example for specifying log level
  2020-03-17  8:31 [dpdk-stable] [PATCH] doc: fix incorrect example for specifying log level Xiaolong Ye
  2020-03-17 17:18 ` [dpdk-stable] [dpdk-dev] " Stephen Hemminger
@ 2020-03-18  0:58 ` Xiaolong Ye
  2020-04-26 20:25   ` [dpdk-stable] [dpdk-dev] " Thomas Monjalon
  1 sibling, 1 reply; 5+ messages in thread
From: Xiaolong Ye @ 2020-03-18  0:58 UTC (permalink / raw)
  To: John McNamara, Marko Kovacevic; +Cc: dev, stephen, Xiaolong Ye, stable

Now we need to add prefix like lib. to enable the log, also changing val 8 to
"debug"" which would be more descriptive.

Fixes: ffb9fd1b0808 ("log: update legacy modules dynamic logs regex")
Cc: stable@dpdk.org

Reported-by: Haiyue Wang <haiyue.wang@intel.com>
Signed-off-by: Xiaolong Ye <xiaolong.ye@intel.com>
---
 doc/guides/linux_gsg/eal_args.include.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/doc/guides/linux_gsg/eal_args.include.rst b/doc/guides/linux_gsg/eal_args.include.rst
index ed8b0e35b..7b2f6b1d4 100644
--- a/doc/guides/linux_gsg/eal_args.include.rst
+++ b/doc/guides/linux_gsg/eal_args.include.rst
@@ -132,7 +132,7 @@ Debugging options
 
     Specify log level for a specific component. For example::
 
-        --log-level eal:8
+        --log-level lib.eal:debug
 
     Can be specified multiple times.
 
-- 
2.17.1


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

* Re: [dpdk-stable] [dpdk-dev] [PATCH v2] doc: fix incorrect example for specifying log level
  2020-03-18  0:58 ` [dpdk-stable] [PATCH v2] " Xiaolong Ye
@ 2020-04-26 20:25   ` Thomas Monjalon
  0 siblings, 0 replies; 5+ messages in thread
From: Thomas Monjalon @ 2020-04-26 20:25 UTC (permalink / raw)
  To: Xiaolong Ye; +Cc: John McNamara, Marko Kovacevic, dev, stephen, stable

18/03/2020 01:58, Xiaolong Ye:
> Now we need to add prefix like lib. to enable the log, also changing val 8 to
> "debug"" which would be more descriptive.
> 
> Fixes: ffb9fd1b0808 ("log: update legacy modules dynamic logs regex")
> Cc: stable@dpdk.org
> 
> Reported-by: Haiyue Wang <haiyue.wang@intel.com>
> Signed-off-by: Xiaolong Ye <xiaolong.ye@intel.com>

Applied, thanks




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

end of thread, other threads:[~2020-04-26 20:26 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-17  8:31 [dpdk-stable] [PATCH] doc: fix incorrect example for specifying log level Xiaolong Ye
2020-03-17 17:18 ` [dpdk-stable] [dpdk-dev] " Stephen Hemminger
2020-03-18  0:57   ` Ye Xiaolong
2020-03-18  0:58 ` [dpdk-stable] [PATCH v2] " Xiaolong Ye
2020-04-26 20:25   ` [dpdk-stable] [dpdk-dev] " Thomas Monjalon

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).