On Tue, Jun 09, 2020 at 09:58:07AM -0700, Joe Perches wrote:

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.

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.

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.

We do not need each individual driver thinking it needs to have some
sort of special classification of each type of debug message.  Just use
the framework that we have, you can enable/disable them on a
line-by-line basis as needed.

This is an attempt to unify those homebrew mechanisms.

All of those should just be removed.

Stanimir attempted to add one for his driver via a
driver specific standardized format substring for level.

quoted

And who defines that grouping?

Individual driver authors

That way lies madness, let's try to fix all of that up.

greg k-h