EDAC driver for ARMv8 L1/L2 cache

[Thor Thayer]

On 01/12/2018 10:48 AM, York Sun wrote:
> On 01/12/2018 08:35 AM, Thor Thayer wrote:
>> On 01/09/2018 03:51 PM, York Sun wrote:
>>> On 01/09/2018 01:43 PM, Borislav Petkov wrote:
>>>> Adding some more people to CC.
>>>>
>>>> On Tue, Jan 09, 2018 at 08:48:43PM +0000, York Sun wrote:
>>>>> Borislav,
>>>>>
>>>>> Are you aware of any existing (or in development) EDAC driver for ARMv8
>>>>> L1/L2 cache? I am thinking to write one if not available yet.
>>>>
>>>> no I'm not but I see two EDAC drivers for ARM64: thunderx and xgene.
>>>>
>>>> Please synchronize with their authors and ARM people what would be the
>>>> best thing to do and try extracting shared functionality from them
>>>> into a common compilation unit instead of duplicating it. I don't want
>>>> separate drivers per functional unit.
>>>>
>>>
>>> Thanks for the pointer. Thunderx and xgene's drivers have different
>>> implementation on the hardware. I found one patch closer to what I
>>> expect, https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatchwork.kernel.org%2Fpatch%2F7513231%2F&data=02%7C01%7Cyork.sun%40nxp.com%7C29cd81c44431418e2a6308d559da7d3b%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C636513717317947616&sdata=flIZL8mxGc4VWbTpl5nPjkRXaB8uNjcc382HcXO08qE%3D&reserved=0. I don't see
>>> activities after 2015. I will reach out to the author.
>>>
>>> York
>>>
>>
>> Hi York,
>>
>> I'll be adding EDAC support for our Stratix10 (Quad ARM Cortex A-53) at
>> some point but I'm not sure when it is scheduled - probably in the next
>> few months. I'll be going through the same process of comparing our
>> architecture to others. Below are links to the Stratix10.
>>
> 
> I see Stratix10 has A53 core. I am concerned on reading the
> CPUMERRSR_EL1 and L2MERRSR_EL1. The are IMPLEMENTATION DEFINED
> registers. They may not be available on all SoCs, or all time. The
> CPUMERRSR_EL1 needs to be read on each core, the L2MERRSR_EL1 needs to
> be read on each cluster. I am not sure how to handle them in EDAC driver
> yet. I am hoping Mark Rutland can shed some light as he commented on
> similar patches [1][2] before.
> 
> York
> 
> [1] https://patchwork.kernel.org/patch/7460391/
> [2] https://patchwork.kernel.org/patch/7513231/

I haven't started digging into EDAC for the Stratix10 yet so I don't 
have a comment but I'll check out your links. Thanks!

> --
> To unsubscribe from this list: send the line "unsubscribe linux-edac" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>