Re: [PATCH 3/4] x86, mce: Add persistent MCE event

From: Borislav Petkov
Date: Tue May 03 2011 - 11:23:01 EST


On Tue, May 03, 2011 at 11:14:16AM -0400, Joe Perches wrote:
> On Tue, 2011-05-03 at 09:18 +0200, Borislav Petkov wrote:
> > On Tue, May 03, 2011 at 02:44:01AM -0400, Ingo Molnar wrote:
> > > * Borislav Petkov <bp@xxxxxxxxx> wrote:
> []
> > > > +static struct dentry *mce_add_event_debugfs(struct perf_event *event, int cpu)
> > > > +{
> > > > + char buf[14];
> > > > +
> > > > + sprintf(buf, "mce_record%d", cpu);
>
> trivia:
>
> buf[14] doesn't fit 10k+ cpus.

I'll fix that when someone comes up with a x86 system that supports 10k+
CPUs with a single system image :).

--
Regards/Gruss,
Boris.

Advanced Micro Devices GmbH
Einsteinring 24, 85609 Dornach
General Managers: Alberto Bozzo, Andrew Bowd
Registration: Dornach, Gemeinde Aschheim, Landkreis Muenchen
Registergericht Muenchen, HRB Nr. 43632
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/