[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <09f8d807-f4ba-de51-177b-c37ca3a9e377@amazon.com>
Date: Wed, 17 Jul 2019 17:45:38 +0300
From: "Hawa, Hanna" <hhhawa@...zon.com>
To: Jan Glauber <jglauber@...vell.com>
CC: "thor.thayer@...ux.intel.com" <thor.thayer@...ux.intel.com>,
"bp@...en8.de" <bp@...en8.de>,
"mchehab@...nel.org" <mchehab@...nel.org>,
"james.morse@....com" <james.morse@....com>,
"rric@...nel.org" <rric@...nel.org>,
"morbidrsa@...il.com" <morbidrsa@...il.com>,
"ralf@...ux-mips.org" <ralf@...ux-mips.org>,
"david.daney@...ium.com" <david.daney@...ium.com>,
"andy.gross@...aro.org" <andy.gross@...aro.org>,
"david.brown@...aro.org" <david.brown@...aro.org>,
"ckadabi@...eaurora.org" <ckadabi@...eaurora.org>,
"vnkgutta@...eaurora.org" <vnkgutta@...eaurora.org>,
"jglauber@...ium.com" <jglauber@...ium.com>,
"khuong@...amperecomputing.com" <khuong@...amperecomputing.com>,
"dwmw@...zon.co.uk" <dwmw@...zon.co.uk>,
"benh@...zon.com" <benh@...zon.com>,
"ronenk@...zon.com" <ronenk@...zon.com>,
"talel@...zon.com" <talel@...zon.com>,
"jonnyc@...zon.com" <jonnyc@...zon.com>,
"hanochu@...zon.com" <hanochu@...zon.com>,
"linux-edac@...r.kernel.org" <linux-edac@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"linux-mips@...r.kernel.org" <linux-mips@...r.kernel.org>,
"linux-arm-msm@...r.kernel.org" <linux-arm-msm@...r.kernel.org>
Subject: Re: [RFC 1/1] edac: Add a counter parameter for
edac_device_handle_ue/ce()
Hi Jan,
On 7/17/2019 3:06 PM, Jan Glauber wrote:
> Hi Hanna,
>
> I'm probably missing something but this patch looks like while it adds
> the error_count parameter the passed values all seem to be 1. So is the
> new parameter used otherwise, maybe in another patch?
Yes in another patch. In Amazon L1/L2 edac driver [1], I'm using loop to
report on multiple L1 or L2 errors. After this patch I'll remove the
loop and pass the errors count.
[1]: https://lkml.org/lkml/2019/7/15/349
Thanks,
Hanna
>
> thanks,
> Jan
Powered by blists - more mailing lists