| 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
| ||
|
Date: Sat, 28 Jun 2008 07:07:21 +0200 From: Andi Kleen <andi@...stfloor.org> To: dougthompson@...ssion.com Cc: bluesmoke-devel@...ts.sourceforge.net, linux-kernel@...r.kernel.org, akpm@...ux-foundation.org Subject: Re: [PATCH 1/13] EDAC i5100 new intel chipset driver dougthompson@...ssion.com writes: > > 2) I have not yet tackled the de-interleaving of the > rank/controller address space into the physical address > space of the CPU. There is nothing fundamentally missing, > it is just ending up to be a lot of code, and I'd rather > keep it separate for now, esp since it doesn't work yet... > > 3) The code depends on a particular i5100 chip select > to DIMM mainboard chip select mapping. This mapping > seems obvious to me in order to support dual and single > ranked memory, but it is not unique and DIMM labels > could be wrong on other mainboards. There is no way > to query this mapping that I know of. Since there's a non negligible probability that the output of this driver is completely misleading because of (2) and (3) and probably (4) too [reporting the wrong DIMMs etc.] would it be possible to add some flag to EDAC that warns the user that the output is not fully reliable? -Andi (who can just see people replacing the wrong DIMMs and then blaming Linux) -- 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