* Missing message prefix in target_core_device.c
@ 2021-06-30 13:50 David Sterba
2021-07-19 1:03 ` Martin K. Petersen
0 siblings, 1 reply; 2+ messages in thread
From: David Sterba @ 2021-06-30 13:50 UTC (permalink / raw)
To: linux-scsi, target-devel; +Cc: martin.petersen
Hi,
I've noticed that after loading the target core there's a message in
syslog
"Rounding down aligned max_sectors from 4294967295 to 4294967288"
but without a prefix, printed from drivers/target/target_core_device.c.
It's pr_info so the prefix should be defined at the top of the file as
pr_fmt and it'll be picked by the pr_* macros automatically.
I'm not sending a patch as I don't know what would be the preferred
prefix, eg. drivers/target/target_core_user.c prints "tcmu daemon" but
does not use pr_*, and there are several bare printk calls in the same
file. This should be unified in some way but I'm not familiar with the
target code.
d.
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: Missing message prefix in target_core_device.c
2021-06-30 13:50 Missing message prefix in target_core_device.c David Sterba
@ 2021-07-19 1:03 ` Martin K. Petersen
0 siblings, 0 replies; 2+ messages in thread
From: Martin K. Petersen @ 2021-07-19 1:03 UTC (permalink / raw)
To: David Sterba; +Cc: linux-scsi, target-devel, martin.petersen
David,
> "Rounding down aligned max_sectors from 4294967295 to 4294967288"
>
> but without a prefix, printed from drivers/target/target_core_device.c.
> It's pr_info so the prefix should be defined at the top of the file as
> pr_fmt and it'll be picked by the pr_* macros automatically.
>
> I'm not sending a patch as I don't know what would be the preferred
> prefix, eg. drivers/target/target_core_user.c prints "tcmu daemon" but
> does not use pr_*, and there are several bare printk calls in the same
> file. This should be unified in some way but I'm not familiar with the
> target code.
I agree that the target subsystem could do with some improvements in
this area. Many different drivers and components, no consistency in how
things are logged.
Would be a great project for someone. I'll put it on the list.
--
Martin K. Petersen Oracle Linux Engineering
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-07-19 1:03 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-06-30 13:50 Missing message prefix in target_core_device.c David Sterba
2021-07-19 1:03 ` Martin K. Petersen
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).