RE: [PATCH 3/4] ACPI/MRRM: Add "node" symlink to /sys/devices/system/memory/rangeX

From: Luck, Tony
Date: Tue Feb 11 2025 - 12:06:07 EST


>> + if (node)
>> + ret = sysfs_create_link(&entry->dev.kobj, &node->dev.kobj, "node");
>
> What is going to remove this symlink if the memory goes away? Or do
> these never get removed?

There's currently no method for runtime changes to these memory ranges. They
are described by a static ACPI table. I need to poke the folks that came up
with this to ask how memory hotplug will be handled (since CXL seems to be
making that fashionable again).

> symlinks in sysfs created like this always worry me. What is going to
> use it?

<hand waves>User space tools that want to understand what the "per-region"
monitoring and control features are actually operating on.</hand waves>

-Tony