Messages in this thread | | | Date | Thu, 10 Jun 2010 16:46:07 -0700 | From | Andrew Morton <> | Subject | Re: [PATCH 1/2] Add HW_ERR printk prefix for hardware error logging |
| |
On Tue, 08 Jun 2010 14:35:38 +0800 Huang Ying <ying.huang@intel.com> wrote:
> This makes hardware error related log in printk log more explicit. So > that the users can report it to hardware vendor instead of LKML or > software vendor. >
Spose so. Some additional words explaining why you think this would result in an improved kernel would help here. Are you seeing misdirected problem reports? Are we missing out on opportunities to fix hardware? What is driving for this change? Will it bring out some improvement in something which you're seeing within Intel or was it just a random hey-lets-try-this thing?
The kernel's whole approach to messaging is pretty haphazard and lame and sad. There have been various proposals to improve the usefulness and to rationally categorise things in way which are more useful to operators, but nothing seems to ever get over the line.
> --- > include/linux/kernel.h | 7 +++++++ > 1 files changed, 7 insertions(+), 0 deletions(-) > > diff --git a/include/linux/kernel.h b/include/linux/kernel.h > index 8317ec4..3bf740b 100644 > --- a/include/linux/kernel.h > +++ b/include/linux/kernel.h > @@ -247,6 +247,13 @@ extern struct pid *session_of_pgrp(struct pid *pgrp); > #define FW_WARN "[Firmware Warn]: " > #define FW_INFO "[Firmware Info]: " > > +/* > + * HW_ERR > + * Add this to a message for hardware errors, so that user can report > + * it to hardware vendor instead of LKML or software vendor. > + */ > +#define HW_ERR "[Hardware Error]: "
I'm trying to think of a rational reason for capitalising "Error", and failing.
Oh well, that's what the other strings do and the kernel already stands as a punctuation/grammar how-not-to guide. ho hum.
| |