Re: [PATCH] introduce boot_printk()

From: H. Peter Anvin
Date: Tue Sep 16 2008 - 13:54:41 EST


Yinghai Lu wrote:

it seem using pci_printk or acpi_printk.. could be more flexible.

otherwise will need to keep update linux/kernel.h to add so call subsys tags...

use only need to
have DEFINE_LOGLEVEL_SETUP(xxx) in c
and have DECLARE_LOGLEVEL(xxx) in .h
then just use xxx_printk


Uh? You can add the subsys tags elsewhere if you prefer too.

However, there are a few important differences to keep in mind. Tags get logged in the dmesg buffer even if not printed to the kernel, whereas a jump-based architecture will not just suppress the output, but even the formatting of the output. Either is a good thing for certain things, and a bad thing for other things.

I personally think the kind of things discussed belong in the category of "always put in the dmesg buffer", and like the idea of subsystem-tagging them.

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