* [PATCH v3] cxl: Remove noisy dev_dbg() outputs
@ 2023-09-20 20:50 Dave Jiang
2023-09-21 15:53 ` Ira Weiny
2023-09-21 17:24 ` Dave Jiang
0 siblings, 2 replies; 3+ messages in thread
From: Dave Jiang @ 2023-09-20 20:50 UTC (permalink / raw)
To: linux-cxl
Cc: dave, jonathan.cameron, alison.schofield, vishal.l.verma,
ira.weiny, dan.j.williams
Remove noisy enumeration of commands and CEL opcodes via dev_dbg()
emit. These outputs were useful during early development. However they are
now unnecessary and creates excessive noise in the debug log. On certain
hardware up to 500+ entries have been observed.
Suggested-by: Alison Schofield <alison.schofield@intel.com>
Signed-off-by: Dave Jiang <dave.jiang@intel.com>
---
https://lore.kernel.org/linux-cxl/b72946f0-31dd-b22a-f2e8-a12eb2f522ae@intel.com/T/#t
v3:
- Drop eventtrace and just remove the dev_dbg(). (Ira & Alison)
v2:
- Add enabling note in commit log (Ira)
---
drivers/cxl/core/mbox.c | 11 +----------
1 file changed, 1 insertion(+), 10 deletions(-)
diff --git a/drivers/cxl/core/mbox.c b/drivers/cxl/core/mbox.c
index ca60bb8114f2..963c668a4eb4 100644
--- a/drivers/cxl/core/mbox.c
+++ b/drivers/cxl/core/mbox.c
@@ -707,7 +707,6 @@ static void cxl_walk_cel(struct cxl_memdev_state *mds, size_t size, u8 *cel)
{
struct cxl_cel_entry *cel_entry;
const int cel_entries = size / sizeof(*cel_entry);
- struct device *dev = mds->cxlds.dev;
int i;
cel_entry = (struct cxl_cel_entry *) cel;
@@ -717,11 +716,8 @@ static void cxl_walk_cel(struct cxl_memdev_state *mds, size_t size, u8 *cel)
struct cxl_mem_command *cmd = cxl_mem_find_command(opcode);
if (!cmd && (!cxl_is_poison_command(opcode) ||
- !cxl_is_security_command(opcode))) {
- dev_dbg(dev,
- "Opcode 0x%04x unsupported by driver\n", opcode);
+ !cxl_is_security_command(opcode)))
continue;
- }
if (cmd)
set_bit(cmd->info.id, mds->enabled_cmds);
@@ -731,8 +727,6 @@ static void cxl_walk_cel(struct cxl_memdev_state *mds, size_t size, u8 *cel)
if (cxl_is_security_command(opcode))
cxl_set_security_cmd_enabled(&mds->security, opcode);
-
- dev_dbg(dev, "Opcode 0x%04x enabled\n", opcode);
}
}
@@ -787,7 +781,6 @@ static const uuid_t log_uuid[] = {
int cxl_enumerate_cmds(struct cxl_memdev_state *mds)
{
struct cxl_mbox_get_supported_logs *gsl;
- struct device *dev = mds->cxlds.dev;
struct cxl_mem_command *cmd;
int i, rc;
@@ -801,8 +794,6 @@ int cxl_enumerate_cmds(struct cxl_memdev_state *mds)
uuid_t uuid = gsl->entry[i].uuid;
u8 *log;
- dev_dbg(dev, "Found LOG type %pU of size %d", &uuid, size);
-
if (!uuid_equal(&uuid, &log_uuid[CEL_UUID]))
continue;
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH v3] cxl: Remove noisy dev_dbg() outputs
2023-09-20 20:50 [PATCH v3] cxl: Remove noisy dev_dbg() outputs Dave Jiang
@ 2023-09-21 15:53 ` Ira Weiny
2023-09-21 17:24 ` Dave Jiang
1 sibling, 0 replies; 3+ messages in thread
From: Ira Weiny @ 2023-09-21 15:53 UTC (permalink / raw)
To: Dave Jiang, linux-cxl
Cc: dave, jonathan.cameron, alison.schofield, vishal.l.verma,
ira.weiny, dan.j.williams
Dave Jiang wrote:
> Remove noisy enumeration of commands and CEL opcodes via dev_dbg()
> emit. These outputs were useful during early development. However they are
> now unnecessary and creates excessive noise in the debug log. On certain
> hardware up to 500+ entries have been observed.
>
> Suggested-by: Alison Schofield <alison.schofield@intel.com>
> Signed-off-by: Dave Jiang <dave.jiang@intel.com>
This looks good to me but it's going to conflict with Dan's rework here:
https://lore.kernel.org/all/6500e8a179440_12747294a3@dwillia2-xfh.jf.intel.com.notmuch/
It would probably be nice to rebase on that for Dan but in case he wants
to merge it.
Reviewed-by: Ira Weiny <ira.weiny@intel.com>
>
> ---
> https://lore.kernel.org/linux-cxl/b72946f0-31dd-b22a-f2e8-a12eb2f522ae@intel.com/T/#t
>
> v3:
> - Drop eventtrace and just remove the dev_dbg(). (Ira & Alison)
> v2:
> - Add enabling note in commit log (Ira)
> ---
> drivers/cxl/core/mbox.c | 11 +----------
> 1 file changed, 1 insertion(+), 10 deletions(-)
>
> diff --git a/drivers/cxl/core/mbox.c b/drivers/cxl/core/mbox.c
> index ca60bb8114f2..963c668a4eb4 100644
> --- a/drivers/cxl/core/mbox.c
> +++ b/drivers/cxl/core/mbox.c
> @@ -707,7 +707,6 @@ static void cxl_walk_cel(struct cxl_memdev_state *mds, size_t size, u8 *cel)
> {
> struct cxl_cel_entry *cel_entry;
> const int cel_entries = size / sizeof(*cel_entry);
> - struct device *dev = mds->cxlds.dev;
> int i;
>
> cel_entry = (struct cxl_cel_entry *) cel;
> @@ -717,11 +716,8 @@ static void cxl_walk_cel(struct cxl_memdev_state *mds, size_t size, u8 *cel)
> struct cxl_mem_command *cmd = cxl_mem_find_command(opcode);
>
> if (!cmd && (!cxl_is_poison_command(opcode) ||
> - !cxl_is_security_command(opcode))) {
> - dev_dbg(dev,
> - "Opcode 0x%04x unsupported by driver\n", opcode);
> + !cxl_is_security_command(opcode)))
> continue;
> - }
>
> if (cmd)
> set_bit(cmd->info.id, mds->enabled_cmds);
> @@ -731,8 +727,6 @@ static void cxl_walk_cel(struct cxl_memdev_state *mds, size_t size, u8 *cel)
>
> if (cxl_is_security_command(opcode))
> cxl_set_security_cmd_enabled(&mds->security, opcode);
> -
> - dev_dbg(dev, "Opcode 0x%04x enabled\n", opcode);
> }
> }
>
> @@ -787,7 +781,6 @@ static const uuid_t log_uuid[] = {
> int cxl_enumerate_cmds(struct cxl_memdev_state *mds)
> {
> struct cxl_mbox_get_supported_logs *gsl;
> - struct device *dev = mds->cxlds.dev;
> struct cxl_mem_command *cmd;
> int i, rc;
>
> @@ -801,8 +794,6 @@ int cxl_enumerate_cmds(struct cxl_memdev_state *mds)
> uuid_t uuid = gsl->entry[i].uuid;
> u8 *log;
>
> - dev_dbg(dev, "Found LOG type %pU of size %d", &uuid, size);
> -
> if (!uuid_equal(&uuid, &log_uuid[CEL_UUID]))
> continue;
>
>
>
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH v3] cxl: Remove noisy dev_dbg() outputs
2023-09-20 20:50 [PATCH v3] cxl: Remove noisy dev_dbg() outputs Dave Jiang
2023-09-21 15:53 ` Ira Weiny
@ 2023-09-21 17:24 ` Dave Jiang
1 sibling, 0 replies; 3+ messages in thread
From: Dave Jiang @ 2023-09-21 17:24 UTC (permalink / raw)
To: linux-cxl
Cc: dave, jonathan.cameron, alison.schofield, vishal.l.verma,
ira.weiny, dan.j.williams
On 9/20/23 13:50, Dave Jiang wrote:
> Remove noisy enumeration of commands and CEL opcodes via dev_dbg()
> emit. These outputs were useful during early development. However they are
> now unnecessary and creates excessive noise in the debug log. On certain
> hardware up to 500+ entries have been observed.
>
> Suggested-by: Alison Schofield <alison.schofield@intel.com>
> Signed-off-by: Dave Jiang <dave.jiang@intel.com>
After discussing with Ira and Dan offline, this patch can be ignored.
To reduce noise, something like this can be done instead:
options cxl_core dyndbg="+fp; func cxl_walk_cel -p"
in /etc/modprobe.d/cxl-debug.conf
>
> ---
> https://lore.kernel.org/linux-cxl/b72946f0-31dd-b22a-f2e8-a12eb2f522ae@intel.com/T/#t
>
> v3:
> - Drop eventtrace and just remove the dev_dbg(). (Ira & Alison)
> v2:
> - Add enabling note in commit log (Ira)
> ---
> drivers/cxl/core/mbox.c | 11 +----------
> 1 file changed, 1 insertion(+), 10 deletions(-)
>
> diff --git a/drivers/cxl/core/mbox.c b/drivers/cxl/core/mbox.c
> index ca60bb8114f2..963c668a4eb4 100644
> --- a/drivers/cxl/core/mbox.c
> +++ b/drivers/cxl/core/mbox.c
> @@ -707,7 +707,6 @@ static void cxl_walk_cel(struct cxl_memdev_state *mds, size_t size, u8 *cel)
> {
> struct cxl_cel_entry *cel_entry;
> const int cel_entries = size / sizeof(*cel_entry);
> - struct device *dev = mds->cxlds.dev;
> int i;
>
> cel_entry = (struct cxl_cel_entry *) cel;
> @@ -717,11 +716,8 @@ static void cxl_walk_cel(struct cxl_memdev_state *mds, size_t size, u8 *cel)
> struct cxl_mem_command *cmd = cxl_mem_find_command(opcode);
>
> if (!cmd && (!cxl_is_poison_command(opcode) ||
> - !cxl_is_security_command(opcode))) {
> - dev_dbg(dev,
> - "Opcode 0x%04x unsupported by driver\n", opcode);
> + !cxl_is_security_command(opcode)))
> continue;
> - }
>
> if (cmd)
> set_bit(cmd->info.id, mds->enabled_cmds);
> @@ -731,8 +727,6 @@ static void cxl_walk_cel(struct cxl_memdev_state *mds, size_t size, u8 *cel)
>
> if (cxl_is_security_command(opcode))
> cxl_set_security_cmd_enabled(&mds->security, opcode);
> -
> - dev_dbg(dev, "Opcode 0x%04x enabled\n", opcode);
> }
> }
>
> @@ -787,7 +781,6 @@ static const uuid_t log_uuid[] = {
> int cxl_enumerate_cmds(struct cxl_memdev_state *mds)
> {
> struct cxl_mbox_get_supported_logs *gsl;
> - struct device *dev = mds->cxlds.dev;
> struct cxl_mem_command *cmd;
> int i, rc;
>
> @@ -801,8 +794,6 @@ int cxl_enumerate_cmds(struct cxl_memdev_state *mds)
> uuid_t uuid = gsl->entry[i].uuid;
> u8 *log;
>
> - dev_dbg(dev, "Found LOG type %pU of size %d", &uuid, size);
> -
> if (!uuid_equal(&uuid, &log_uuid[CEL_UUID]))
> continue;
>
>
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-09-21 19:05 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-09-20 20:50 [PATCH v3] cxl: Remove noisy dev_dbg() outputs Dave Jiang
2023-09-21 15:53 ` Ira Weiny
2023-09-21 17:24 ` Dave Jiang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox