Re: [PATCH 2/2] printk: Remember the message level for multi-lineoutput

From: Ingo Molnar
Date: Mon Apr 14 2008 - 08:22:01 EST



* Johannes Weiner <hannes@xxxxxxxxxxxx> wrote:

> Hi Ingo,
>
> Ingo Molnar <mingo@xxxxxxx> writes:
>
> > btw., we could also start emitting debug warnings that the printk is not
> > conform. Something like:
> >
> > "INFO: the previous printk was done without a KERN_ annotation"
>
> Urgh, that would become very noisy. If, at all, I'd suggest a config
> option like CONFIG_DEBUG_PRINTK_USAGE and then add a prefix string to
> each printk()ed line missing a severity level.
>
> printk("foo\n");
> => "[FIX PRINTK] foo"

yes, of course hidden behind a non-default Kconfig variable, like we do
it with most debug helpers.

Ingo
--
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/