Re: [PATCH] bio.c: reduce verbosity at boot

From: Matthew Wilcox
Date: Thu Aug 21 2003 - 14:42:35 EST


On Thu, Aug 21, 2003 at 10:51:27AM -0600, Lou Langholtz wrote:
> Jeff Garzik wrote:
> >. . . removing the messages outright might not serve the best
> >interests the developer. Since even KERN_DEBUG still spams dmesg, in
> >these situations I usually change these type of messages to be
> >conditionally printed iff a debug macro is enabled.
>
> How about using KERN_DEBUG and augmenting the dmesg store so that the
> level that is saved is configurable? Even compile time configurable
> seems reasonable to start. But axeing out even the possibility of boot
> time info seems bad to me.

But why is it interesting to have this information at boot time? As a
user, I certainly don't care. As a developer, I don't find it interesting
information. Maybe the maintainer of this particular piece of code finds
it useful (Do you, Jens?), but does it need to be reported at boot time?

Perhaps it would be more useful to add some kind of reporting to
mm/mempool.c along the same lines as /proc/slabinfo so we can get the
dynamic information about pools at runtime rather than knowing their
initial state at boot time.

--
"It's not Hollywood. War is real, war is primarily not about defeat or
victory, it is about death. I've seen thousands and thousands of dead bodies.
Do you think I want to have an academic debate on this subject?" -- Robert Fisk
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/