Thread (46 messages) 46 messages, 10 authors, 2020-06-12

Re: [PATCH v3 1/7] Documentation: dynamic-debug: Add description of level bitmask

From: Joe Perches <joe@perches.com>
Date: 2020-06-10 06:35:40
Also in: linux-acpi, linux-arm-msm, linux-btrfs, linux-media, lkml, netdev

On Wed, 2020-06-10 at 08:31 +0200, Greg Kroah-Hartman wrote:
On Tue, Jun 09, 2020 at 09:58:07AM -0700, Joe Perches wrote:
quoted
On Tue, 2020-06-09 at 13:16 +0200, Greg Kroah-Hartman wrote:
quoted
What is wrong with the existing control of dynamic
debug messages that you want to add another type of arbitrary grouping
to it? 
There is no existing grouping mechanism.
info/warn/err/dbg is what I am referring to.
quoted
Many drivers and some subsystems used an internal one
before dynamic debug.

$ git grep "MODULE_PARM.*\bdebug\b"|wc -l
501
Yes, and it's horrid and needs to be cleaned up, not added to.
Or unified so driver authors have a standardized mechanism
rather than reinventing or doing things differently.
In the beginning, yes, adding loads of different types of debugging
options to a driver is needed by the author, but by the time it is added
to the kernel, all of that should be able to be removed and only a
single "enable debug" should be all that is needed.
No one does that.

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help