[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-id: <20130812083355.47c1bae8@samsung.com>
Date: Mon, 12 Aug 2013 08:33:55 -0300
From: Mauro Carvalho Chehab <m.chehab@...sung.com>
To: Borislav Petkov <bp@...en8.de>
Cc: "Naveen N. Rao" <naveen.n.rao@...ux.vnet.ibm.com>,
tony.luck@...el.com, bhelgaas@...gle.com, rostedt@...dmis.org,
rjw@...k.pl, lance.ortiz@...com, linux-pci@...r.kernel.org,
linux-acpi@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 3/3] mce: acpi/apei: trace: Enable ghes memory error trace
event
Em Sat, 10 Aug 2013 20:03:22 +0200
Borislav Petkov <bp@...en8.de> escreveu:
> On Thu, Aug 08, 2013 at 04:38:22PM -0300, Mauro Carvalho Chehab wrote:
> > Em Thu, 08 Aug 2013 23:57:51 +0530
> > "Naveen N. Rao" <naveen.n.rao@...ux.vnet.ibm.com> escreveu:
> >
> > > Enable memory error trace event in cper.c
> >
> > Why do we need to do that? Memory error events are already handled
> > via edac_ghes module,
>
> If APEI gives me all the required information in order to deal with the
> hardware error - and it looks like it does - then the additional layer
> of ghes_edac is not needed.
APEI is just the mechanism that collects the data, not the mechanism
that reports to userspace.
The current implementation is that APEI already reports those errors
via ghes_edac driver. It also reports the very same error via MCE
(although the APEI interface to MCE is currently broken for everything
that it is not Nehalem-EX - as it basically emulates the MCE log for
that specific architecture).
I really don't see any sense on adding yet-another-way to report the
very same error.
Regards,
Mauro
--
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