[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20120531142229.GF14515@aftab.osrc.amd.com>
Date: Thu, 31 May 2012 16:22:29 +0200
From: Borislav Petkov <bp@...64.org>
To: Mauro Carvalho Chehab <mchehab@...hat.com>
Cc: "Luck, Tony" <tony.luck@...el.com>,
Linux Edac Mailing List <linux-edac@...r.kernel.org>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
Aristeu Rozanski <arozansk@...hat.com>,
Doug Thompson <norsk5@...oo.com>,
Steven Rostedt <rostedt@...dmis.org>,
Frederic Weisbecker <fweisbec@...il.com>,
Ingo Molnar <mingo@...hat.com>
Subject: Re: [PATCH] RAS: Add a tracepoint for reporting memory controller
events
On Thu, May 31, 2012 at 10:56:28AM -0300, Mauro Carvalho Chehab wrote:
> Em 31-05-2012 09:17, Borislav Petkov escreveu:
> > On Thu, May 31, 2012 at 07:33:25AM -0300, Mauro Carvalho Chehab wrote:
> >> Huh? Tony said that some errors report at 4K granularity while, others
> >> are at cache line size.
> >
> > Cacheline size is from MCE. This tracepoint is for memory controller
> > errors where granularity should be pretty stable.
>
> Both sb_edac and i7core_edac get errors via MCE. They're currently providing
> a fake granularity, but I'm planning to add proper grain support on them,
> by retrieving such information from the MCE registers.
Those are only 2 drivers and not _all_ edac drivers. For the remaining
ones grain is a pretty stable, seldomly changing value. Which means it
should be part of const char driver_detail only on those two drivers and
not a driver-global field in the tracepoint.
--
Regards/Gruss,
Boris.
Advanced Micro Devices GmbH
Einsteinring 24, 85609 Dornach
GM: Alberto Bozzo
Reg: Dornach, Landkreis Muenchen
HRB Nr. 43632 WEEE Registernr: 129 19551
--
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