linux-kernel - Re: [PATCH 1/13] EDAC i5100 new intel chipset driver

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] [day] [month] [year] [list]

Date:	Mon, 30 Jun 2008 13:13:40 -0700
From:	Arthur Jones <ajones@...erbed.com>
To:	Doug Thompson <norsk5@...oo.com>
CC:	Andi Kleen <andi@...stfloor.org>,
	"dougthompson@...ssion.com" <dougthompson@...ssion.com>,
	"akpm@...ux-foundation.org" <akpm@...ux-foundation.org>,
	"bluesmoke-devel@...ts.sourceforge.net" 
	<bluesmoke-devel@...ts.sourceforge.net>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 1/13] EDAC i5100 new intel chipset driver

Hi Doug,  ...

On Mon, Jun 30, 2008 at 12:58:59PM -0700, Doug Thompson wrote:
> 
> --- Arthur Jones <ajones@...erbed.com> wrote:
> 
> > > Arthur, could you add some more text to the driver's output, before the call to the
> > > core's output function, doing exactly that? Explaining just what you DO know, and what you
> > don't
> > > know?
> >
> > Everything should be good except the DIMM label.  There
> > are all kinds of ways this could be wrong (mainboard labelling,
> > mainboard routing, ...).  I think it might be better just
> > to leave this blank.
> >
> > Are there any EDAC drivers that properly report the DIMM label?
> 
> The drivers don't print the information just because of these issues. The labeling action is left
> to user space via edac-utils.  The label attributes in sysfs are initialized via a bootup daemon.
> Thereby EDAC (and the driver) provides a mechanism of storing the DIMM Label. (The driver is free
> to populate label if it so chooses, but that too could be overridden by userspace).
> 
> One driver I did for the Sicortex MIPS cluster only had TWO DIMMS, "East" and "West" on a 6 core
> processor card. The driver labeled those inside itself.
> 
> >
> > How do they get around these issues?
> 
> Late binding. Allow the setting of the label by userspace. edac-utils utilizes a file database of
> motherboards, which must be maintained yes, but it is flexible. An additional userspace program
> could source the labels via another mechanism.
> 
> Lawrence Livermore National Labs has tens of thousands of nodes, with 2, 4 or 8 CPUs per node, and
> they are the ones who developed the edac-utils userspace daemon and scripts.
> 
> http://sourceforge.net/projects/edac-utils/
> 
> The model for EDAC modules IS to leave the DIMM label blank for late binding of that information.

Thanks, I didn't know about edac-utils, that sounds much better
than what I was proposing.  I'll go have a look...

Given this, I think it would be best to pull out the labels from
the driver.  I'll see if I can put together a patch for that today
or tomorrow...

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