Re: [PATCH v4 4/4] ACPI: Add documentation for exposing MRRM data

Fenghua Yu
2025-05-05T06:34:55Z

Hi, Tony,

On 4/29/25 13:24, Tony Luck wrote:
> Initial implementation provides enumeration of the address ranges
> NUMA node numbers, and BIOS assigned region IDs for each range.
>
> Signed-off-by: Tony Luck <tony.luck@intel.com>
> ---
> Documentation/ABI/testing/sysfs-firmware-acpi | 21 +++++++++++++++++++
> 1 file changed, 21 insertions(+)
>
> diff --git a/Documentation/ABI/testing/sysfs-firmware-acpi b/Documentation/ABI/testing/sysfs-firmware-acpi
> index 5249ad5a96d9..fffba38f9ce1 100644
> --- a/Documentation/ABI/testing/sysfs-firmware-acpi
> +++ b/Documentation/ABI/testing/sysfs-firmware-acpi
> @@ -248,3 +248,24 @@ Description:
> # cat ff_pwr_btn
> 7 enabled
>
> +What: /sys/firmware/acpi/memory_ranges/rangeX
> +Date: February 2025
> +Contact: Tony Luck <tony.luck@intel.com>
> +Description:
> + On systems with the ACPI MRRM table reports the
> + parameters for each range.

Is there a need to explain what's "X" here? The "X" is not a number
directly reported by MRRM, right?

Maybe something like "range ID is enumerated from MRRM starting from 0."?

> +
> + base: Starting system physical address.
> +
> + length: Length of this range in bytes.
> +
> + node: NUMA node that this range belongs to. Negative numbers
> + indicate that the node number could not be determined (e.g
> + for an address range that is reserved for future hot add of
> + memory).
> +
> + local_region_id: ID associated with access by agents
> + local to this range of addresses.
> +
> + remote_region_id: ID associated with access by agents
> + non-local to this range of addresses.

Thanks.

-Fenghua