On 06/11/2012 09:23 AM, Mauro Carvalho Chehab wrote: > This is an automatic generated email to let you know that the following patch > were queued at the > http://git.linuxtv.org/media_tree.git tree:
Wrong git tree ????? > Subject: edac: add a new per-dimm API and make the old per-virtual-rank API > obsolete > Author: Mauro Carvalho Chehab <[email protected]> > Date: Wed Mar 21 17:06:53 2012 -0300 > > The old EDAC API is broken. It only works fine for systems manufatured > before 2005 and for AMD 64. The reason is that it forces all memory > controller drivers to discover rank info. > > Also, it doesn't allow grouping the several ranks into a DIMM. > > So, what almost all modern drivers do is to create a fake virtual-rank > information, and use it to cheat the EDAC core to accept the driver. > > While this works if the user has enough time to discover what DIMM slot > corresponds to each "virtual-rank" information, it prevents EDAC usage > for users with less available time. It also makes life hard for vendors > that may want to provide a table with their motherboards to the userspace > tool (edac-utils) as each driver has its own logic for the virtual > mapping. > > So, the old API should be removed, in favor of a more flexible API that > allows newer drivers to not lie to the EDAC core. > > Reviewed-by: Aristeu Rozanski <[email protected]> > Cc: Doug Thompson <[email protected]> > Cc: Borislav Petkov <[email protected]> > Cc: Randy Dunlap <[email protected]> > Cc: Josh Boyer <[email protected]> > Cc: Hui Wang <[email protected]> > Signed-off-by: Mauro Carvalho Chehab <[email protected]> > > drivers/edac/Kconfig | 8 ++ > drivers/edac/edac_mc_sysfs.c | 165 > +++++++++++++++++++++++++++++++++++++++++- > 2 files changed, 172 insertions(+), 1 deletions(-) > > --- -- ~Randy -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
