From: Anoob Joseph <anoobj@marvell.com>
To: Nithin Dabilpuram <ndabilpuram@marvell.com>,
Kiran Kumar K <kirankumark@marvell.com>,
Sunil Kumar Kori <skori@marvell.com>,
Satha Rao <skoteshwar@marvell.com>,
Harman Kalra <hkalra@marvell.com>,
Jerin Jacob <jerinj@marvell.com>
Cc: Ashwin Sekhar T K <asekhar@marvell.com>,
Srikanth Yalavarthi <syalavarthi@marvell.com>,
Pavan Nikhilesh <pbhagavatula@marvell.com>,
"Ferruh Yigit" <ferruh.yigit@amd.com>, <dev@dpdk.org>
Subject: [PATCH] common/cnxk: restore segregation of logs based on module
Date: Tue, 23 Apr 2024 10:44:43 +0000 [thread overview]
Message-ID: <20240423104451.3018314-1-anoobj@marvell.com> (raw)
Originally the logs were segregated under various labels which could be
selectively enabled. It was changed to use 'pmd.common.cnxk' while
changing the macro used for registering logging. Address the same by
restoring the segregation.
Current logs:
...
logtype3
pmd.common.cnxk
pmd.common.iavf
...
Changed to:
...
logtype3
pmd.common.cnxk.base
pmd.common.cnxk.crypto
pmd.common.cnxk.dpi
pmd.common.cnxk.esw
pmd.common.cnxk.event
pmd.common.cnxk.flow
pmd.common.cnxk.mbox
pmd.common.cnxk.mempool
pmd.common.cnxk.ml
pmd.common.cnxk.nix
pmd.common.cnxk.ree
pmd.common.cnxk.rep
pmd.common.cnxk.timer
pmd.common.cnxk.tm
pmd.common.iavf
...
Updated documentation also to reflect the same.
Fixes: 233692f550a1 ("dma/cnxk: rework DMA driver")
Signed-off-by: Anoob Joseph <anoobj@marvell.com>
---
doc/guides/cryptodevs/cnxk.rst | 2 +-
doc/guides/eventdevs/cnxk.rst | 4 ++--
doc/guides/mempool/cnxk.rst | 2 +-
doc/guides/mldevs/cnxk.rst | 2 +-
doc/guides/nics/cnxk.rst | 8 ++++----
doc/guides/platform/cnxk.rst | 4 ++--
drivers/common/cnxk/roc_platform.c | 28 ++++++++++++++--------------
7 files changed, 25 insertions(+), 25 deletions(-)
diff --git a/doc/guides/cryptodevs/cnxk.rst b/doc/guides/cryptodevs/cnxk.rst
index 8dc745dccd..1a3cf80c31 100644
--- a/doc/guides/cryptodevs/cnxk.rst
+++ b/doc/guides/cryptodevs/cnxk.rst
@@ -209,7 +209,7 @@ Debugging Options
+---+------------+-------------------------------------------------------+
| # | Component | EAL log command |
+===+============+=======================================================+
- | 1 | CPT | --log-level='pmd\.crypto\.cnxk,8' |
+ | 1 | CPT | --log-level='pmd\.common\.cnxk\.crypto,8' |
+---+------------+-------------------------------------------------------+
Testing
diff --git a/doc/guides/eventdevs/cnxk.rst b/doc/guides/eventdevs/cnxk.rst
index 9ff1052c53..af925533a3 100644
--- a/doc/guides/eventdevs/cnxk.rst
+++ b/doc/guides/eventdevs/cnxk.rst
@@ -208,9 +208,9 @@ Debugging Options
+---+------------+-------------------------------------------------------+
| # | Component | EAL log command |
+===+============+=======================================================+
- | 1 | SSO | --log-level='pmd\.event\.cnxk,8' |
+ | 1 | SSO | --log-level='pmd\.common\.cnxk\.event,8' |
+---+------------+-------------------------------------------------------+
- | 2 | TIM | --log-level='pmd\.event\.cnxk\.timer,8' |
+ | 2 | TIM | --log-level='pmd\.common\.cnxk\.timer,8' |
+---+------------+-------------------------------------------------------+
Limitations
diff --git a/doc/guides/mempool/cnxk.rst b/doc/guides/mempool/cnxk.rst
index d883b83f7b..ffd9a6de86 100644
--- a/doc/guides/mempool/cnxk.rst
+++ b/doc/guides/mempool/cnxk.rst
@@ -72,7 +72,7 @@ Debugging Options
+---+------------+-------------------------------------------------------+
| # | Component | EAL log command |
+===+============+=======================================================+
- | 1 | NPA | --log-level='pmd\.mempool.cnxk,8' |
+ | 1 | NPA | --log-level='pmd\.common\.cnxk\.mempool,8' |
+---+------------+-------------------------------------------------------+
Standalone mempool device
diff --git a/doc/guides/mldevs/cnxk.rst b/doc/guides/mldevs/cnxk.rst
index ae9e1fae5f..fc1bcd9cdb 100644
--- a/doc/guides/mldevs/cnxk.rst
+++ b/doc/guides/mldevs/cnxk.rst
@@ -383,7 +383,7 @@ Debugging Options
+---+------------+-------------------------------------------------------+
| # | Component | EAL log command |
+===+============+=======================================================+
- | 1 | ML | --log-level='pmd\.ml\.cnxk,8' |
+ | 1 | ML | --log-level='pmd\.common\.cnxk\.ml,8' |
+---+------------+-------------------------------------------------------+
diff --git a/doc/guides/nics/cnxk.rst b/doc/guides/nics/cnxk.rst
index f2d726047e..227270353f 100644
--- a/doc/guides/nics/cnxk.rst
+++ b/doc/guides/nics/cnxk.rst
@@ -701,11 +701,11 @@ Debugging Options
+---+------------+-------------------------------------------------------+
| # | Component | EAL log command |
+===+============+=======================================================+
- | 1 | NIX | --log-level='pmd\.net.cnxk,8' |
+ | 1 | NIX | --log-level='pmd\.common.cnxk\.nix,8' |
+---+------------+-------------------------------------------------------+
- | 2 | NPC | --log-level='pmd\.net.cnxk\.flow,8' |
+ | 2 | NPC | --log-level='pmd\.common.cnxk\.flow,8' |
+---+------------+-------------------------------------------------------+
- | 3 | REP | --log-level='pmd\.net.cnxk\.rep,8' |
+ | 3 | REP | --log-level='pmd\.common.cnxk\.rep,8' |
+---+------------+-------------------------------------------------------+
- | 4 | ESW | --log-level='pmd\.net.cnxk\.esw,8' |
+ | 4 | ESW | --log-level='pmd\.common.cnxk\.esw,8' |
+---+------------+-------------------------------------------------------+
diff --git a/doc/guides/platform/cnxk.rst b/doc/guides/platform/cnxk.rst
index 70065e3d96..0e61bc91d9 100644
--- a/doc/guides/platform/cnxk.rst
+++ b/doc/guides/platform/cnxk.rst
@@ -250,9 +250,9 @@ Debugging Options
+---+------------+-------------------------------------------------------+
| # | Component | EAL log command |
+===+============+=======================================================+
- | 1 | Common | --log-level='pmd\.cnxk\.base,8' |
+ | 1 | Common | --log-level='pmd\.common\.cnxk\.base,8' |
+---+------------+-------------------------------------------------------+
- | 2 | Mailbox | --log-level='pmd\.cnxk\.mbox,8' |
+ | 2 | Mailbox | --log-level='pmd\.common\.cnxk\.mbox,8' |
+---+------------+-------------------------------------------------------+
Debugfs support
diff --git a/drivers/common/cnxk/roc_platform.c b/drivers/common/cnxk/roc_platform.c
index 181902a585..30379c7e5e 100644
--- a/drivers/common/cnxk/roc_platform.c
+++ b/drivers/common/cnxk/roc_platform.c
@@ -85,17 +85,17 @@ roc_plt_init(void)
return 0;
}
-RTE_LOG_REGISTER_DEFAULT(cnxk_logtype_base, NOTICE);
-RTE_LOG_REGISTER_DEFAULT(cnxk_logtype_mbox, NOTICE);
-RTE_LOG_REGISTER_DEFAULT(cnxk_logtype_cpt, NOTICE);
-RTE_LOG_REGISTER_DEFAULT(cnxk_logtype_ml, NOTICE);
-RTE_LOG_REGISTER_DEFAULT(cnxk_logtype_npa, NOTICE);
-RTE_LOG_REGISTER_DEFAULT(cnxk_logtype_nix, NOTICE);
-RTE_LOG_REGISTER_DEFAULT(cnxk_logtype_npc, NOTICE);
-RTE_LOG_REGISTER_DEFAULT(cnxk_logtype_sso, NOTICE);
-RTE_LOG_REGISTER_DEFAULT(cnxk_logtype_tim, NOTICE);
-RTE_LOG_REGISTER_DEFAULT(cnxk_logtype_tm, NOTICE);
-RTE_LOG_REGISTER_DEFAULT(cnxk_logtype_dpi, NOTICE);
-RTE_LOG_REGISTER_DEFAULT(cnxk_logtype_rep, NOTICE);
-RTE_LOG_REGISTER_DEFAULT(cnxk_logtype_esw, NOTICE);
-RTE_LOG_REGISTER_DEFAULT(cnxk_logtype_ree, NOTICE);
+RTE_LOG_REGISTER_SUFFIX(cnxk_logtype_base, base, NOTICE);
+RTE_LOG_REGISTER_SUFFIX(cnxk_logtype_mbox, mbox, NOTICE);
+RTE_LOG_REGISTER_SUFFIX(cnxk_logtype_cpt, crypto, NOTICE);
+RTE_LOG_REGISTER_SUFFIX(cnxk_logtype_ml, ml, NOTICE);
+RTE_LOG_REGISTER_SUFFIX(cnxk_logtype_npa, mempool, NOTICE);
+RTE_LOG_REGISTER_SUFFIX(cnxk_logtype_nix, nix, NOTICE);
+RTE_LOG_REGISTER_SUFFIX(cnxk_logtype_npc, flow, NOTICE);
+RTE_LOG_REGISTER_SUFFIX(cnxk_logtype_sso, event, NOTICE);
+RTE_LOG_REGISTER_SUFFIX(cnxk_logtype_tim, timer, NOTICE);
+RTE_LOG_REGISTER_SUFFIX(cnxk_logtype_tm, tm, NOTICE);
+RTE_LOG_REGISTER_SUFFIX(cnxk_logtype_dpi, dpi, NOTICE);
+RTE_LOG_REGISTER_SUFFIX(cnxk_logtype_rep, rep, NOTICE);
+RTE_LOG_REGISTER_SUFFIX(cnxk_logtype_esw, esw, NOTICE);
+RTE_LOG_REGISTER_SUFFIX(cnxk_logtype_ree, ree, NOTICE);
--
2.43.2
reply other threads:[~2024-04-23 10:45 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
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=20240423104451.3018314-1-anoobj@marvell.com \
--to=anoobj@marvell.com \
--cc=asekhar@marvell.com \
--cc=dev@dpdk.org \
--cc=ferruh.yigit@amd.com \
--cc=hkalra@marvell.com \
--cc=jerinj@marvell.com \
--cc=kirankumark@marvell.com \
--cc=ndabilpuram@marvell.com \
--cc=pbhagavatula@marvell.com \
--cc=skori@marvell.com \
--cc=skoteshwar@marvell.com \
--cc=syalavarthi@marvell.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 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).