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 <mchehab@xxxxxxxxxx> > 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 <arozansk@xxxxxxxxxx> > Cc: Doug Thompson <norsk5@xxxxxxxxx> > Cc: Borislav Petkov <borislav.petkov@xxxxxxx> > Cc: Randy Dunlap <rdunlap@xxxxxxxxxxxx> > Cc: Josh Boyer <jwboyer@xxxxxxxxxx> > Cc: Hui Wang <jason77.wang@xxxxxxxxx> > Signed-off-by: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx> > > 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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
