lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20100610164607.b2908334.akpm@linux-foundation.org>
Date:	Thu, 10 Jun 2010 16:46:07 -0700
From:	Andrew Morton <akpm@...ux-foundation.org>
To:	Huang Ying <ying.huang@...el.com>
Cc:	"H. Peter Anvin" <hpa@...or.com>, Andi Kleen <ak@...ux.intel.com>,
	Hidetoshi Seto <seto.hidetoshi@...fujitsu.com>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
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@...el.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.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ