RE: [PATCH v20 00/15] EDAC: Scrub: introduce generic EDAC RAS control feature driver + CXL/ACPI-RAS2 drivers

[Shiju Jose]

>-----Original Message-----
>From: Daniel Ferguson <danielf@xxxxxxxxxxxxxxxxxxxxxx>
>Sent: 06 March 2025 18:18
>To: Shiju Jose <shiju.jose@xxxxxxxxxx>; linux-edac@xxxxxxxxxxxxxxx; linux-
>cxl@xxxxxxxxxxxxxxx; linux-acpi@xxxxxxxxxxxxxxx; linux-mm@xxxxxxxxx; linux-
>kernel@xxxxxxxxxxxxxxx
>Cc: linux-doc@xxxxxxxxxxxxxxx; bp@xxxxxxxxx; tony.luck@xxxxxxxxx;
>rafael@xxxxxxxxxx; lenb@xxxxxxxxxx; mchehab@xxxxxxxxxx;
>dan.j.williams@xxxxxxxxx; dave@xxxxxxxxxxxx; Jonathan Cameron
><jonathan.cameron@xxxxxxxxxx>; dave.jiang@xxxxxxxxx;
>alison.schofield@xxxxxxxxx; vishal.l.verma@xxxxxxxxx; ira.weiny@xxxxxxxxx;
>david@xxxxxxxxxx; Vilas.Sridharan@xxxxxxx; leo.duran@xxxxxxx;
>Yazen.Ghannam@xxxxxxx; rientjes@xxxxxxxxxx; jiaqiyan@xxxxxxxxxx;
>Jon.Grimm@xxxxxxx; dave.hansen@xxxxxxxxxxxxxxx;
>naoya.horiguchi@xxxxxxx; james.morse@xxxxxxx; jthoughton@xxxxxxxxxx;
>somasundaram.a@xxxxxxx; erdemaktas@xxxxxxxxxx; pgonda@xxxxxxxxxx;
>duenwen@xxxxxxxxxx; gthelen@xxxxxxxxxx;
>wschwartz@xxxxxxxxxxxxxxxxxxx; dferguson@xxxxxxxxxxxxxxxxxxx;
>wbs@xxxxxxxxxxxxxxxxxxxxxx; nifan.cxl@xxxxxxxxx; tanxiaofei
><tanxiaofei@xxxxxxxxxx>; Zengtao (B) <prime.zeng@xxxxxxxxxxxxx>; Roberto
>Sassu <roberto.sassu@xxxxxxxxxx>; kangkang.shen@xxxxxxxxxxxxx;
>wanghuiqiang <wanghuiqiang@xxxxxxxxxx>; Linuxarm
><linuxarm@xxxxxxxxxx>
>Subject: Re: [PATCH v20 00/15] EDAC: Scrub: introduce generic EDAC RAS
>control feature driver + CXL/ACPI-RAS2 drivers
>
>>  +--------------+-----------+-----------+-----------+-----------+
>>  |              |           |           |           |           |
>>  | Setting      | Supported | Supported | No        | No        |
>>  | scrub rate   |           |           |           |           |
>>  |              |           |           |           |           |
>>  +--------------+-----------+-----------+-----------+-----------+
>>  |              |           |           |           |           |
>>  | Unit for     | Not       | in hours  | No        | No        |
>>  | scrub rate   | Defined   |           |           |           |
>>  |              |           |           |           |           |
>
>It is our opinion that the Unit for scrub rate should be unit agnostic
>(implementation defined), and definitely not specified in Hours. We believe that
>the spirit of the RAS2 scrub rate was intended to be implementation defined.
>
>It takes our hardware less than a few seconds to scrub a ton of memory.
Hi Daniel, 

The table illustrates the unit for the scrub rate of the ACPI RAS2 scrub feature as "Not Defined" according to the ACPI 6.5 specification, and for the CXL scrub feature, it is defined as "in hours" according to the CXL spec rev 3.2.

Currently, the RAS2 driver converts the scrub rate set in seconds via the EDAC scrub sysfs to hours and passes it to the firmware in the  field: Configure Scrub Parameters (INPUT) - Bits [15:8]: Requested scrub rate of Table 5.87: Parameter Block Structure for PATROL_SCRUB.

Did you mean that the RAS2 driver needs to pass the scrub rate in seconds to the firmware?  then I think the max scrub rate supported is limited to 256 seconds?

Thanks,
Shiju