linux-kernel - Re: [PATCH] amd64_edac: Memory size reported double on processor family 0Fh

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: <505C7F91.4040100@akamai.com>
Date:	Fri, 21 Sep 2012 09:54:09 -0500
From:	Josh Hunt <johunt@...mai.com>
To:	Borislav Petkov <bp@...64.org>
CC:	"linux-edac@...r.kernel.org" <linux-edac@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] amd64_edac: Memory size reported double on processor
 family 0Fh

On 09/21/2012 09:01 AM, Borislav Petkov wrote:
> Ah ok, my bad, I have 4 more patches ontop which should deal with the
> channel doubling, please repull the same branch which I've just updated.
>
> git://git.kernel.org/pub/scm/linux/kernel/git/bp/bp.git edac-queue
>
> and retest.
>
> And you're right about WIDTH_128 - it actually denotes the channel width
> and the code wrongly uses it to say that there are two channels on K8.
>
> However, this has been the case since the driver went upstream and I
> need to think a bit harder when correcting that so that I don't upset
> something else in it.
>
> Also, it would be very helpful if you send me full dmesg after you've
> applied the latest patches so that I can have another reference when
> changing the WIDTH_128 thing.
>
> Thanks for your help.

OK so progress, but still not there.

So this is correct now:
root@....x.x:/sys/devices/system/edac/mc/mc0# grep . csrow*/size_mb
csrow0/size_mb:1024
csrow1/size_mb:1024
csrow2/size_mb:1024
csrow3/size_mb:1024

But this is not:
root@....x.x:/sys/devices/system/edac/mc/mc0# grep . size_mb
8192

Attached patch seems to resolve reading size_mb from the mc in sysfs. 
I'm also attaching the new dmesg.

Josh



View attachment "0001-EDAC-Fix-mc-size-reported-in-sysfs.patch" of type "text/x-patch" (1369 bytes)

View attachment "dmesg.log.new" of type "text/plain" (56407 bytes)