Messages in this thread | | | Date | Wed, 14 Mar 2012 16:35:00 -0300 | From | Mauro Carvalho Chehab <> | Subject | Re: [PATCH 0/6] Add a per-dimm structure |
| |
Em 13-03-2012 20:32, Greg KH escreveu: > On Wed, Mar 07, 2012 at 08:40:32AM -0300, Mauro Carvalho Chehab wrote: >> Prepare the internal structures to represent the memory properties per dimm, >> instead of per csrow. >> >> This is needed for modern controllers with more than 2 channels, as the memories >> at the same slot number but on different channels (or channel pairs) may be >> different. >> >> Mauro Carvalho Chehab (6): >> edac: Create a dimm struct and move the labels into it >> edac: Add per dimm's sysfs nodes > > You need Documentation/ABI entries for these new sysfs files.
Sure. I'll provide it on the final patchset.
Boris suggested some alternatives for the error counter sysfs nodes, but the discussion ended by diverging into an implementation detail of hiding the UE error nodes, without any consensus about the sysfs structure for it.
At the current patchset, the error counter nodes are all under sys/devices/system/edac/mc/mc?/
He thinks that a multi-layer struct should be created inside that directory (it could have 2 or 3 levels of directories, depending on how the memory is organized at the memory controller), instead of having a large number of files there.
Anyway, before adding unnedded complexity, I'd like to hear more comments from the others before writing a complex patch to create such structure.
So, maybe I could just add what it was there as ABI/testing, and give more time for kernel and userspace developers to work with it and provide us a better feedback.
Thanks, Mauro
| |