Re: [NDCTL PATCH 2/5] cxl/test: Fix cxl-poison.sh to detect the correct elc sysfs attrib

Alison Schofield
2025-11-15T00:54:30Z

On Fri, Oct 31, 2025 at 10:40:00AM -0700, Dave Jiang wrote:
> The cxl-poison.sh script attempts to read the extended linear cache
> size sysfs attribute but is using the incorrect attribute name.
>
> Signed-off-by: Dave Jiang <dave.jiang@intel.com>

Reviewed, tested and updated to the no-op comment
[ alisons: update the no-op attribute comment ]

Applied: https://github.com/pmem/ndctl/commits/pending

Thanks!

> ---
> test/cxl-poison.sh | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/test/cxl-poison.sh b/test/cxl-poison.sh
> index 231a0733f096..8e81baceeb24 100644
> --- a/test/cxl-poison.sh
> +++ b/test/cxl-poison.sh
> @@ -197,8 +197,8 @@ test_poison_by_region_offset_negative()
>
> # This case is a no-op until cxl-test ELC mocking arrives
> # Try to get cache_size if the attribute exists
> - if [ -f "/sys/bus/cxl/devices/$region/cache_size" ]; then
> - cache_size=$(cat /sys/bus/cxl/devices/"$region"/cache_size)
> + if [ -f "/sys/bus/cxl/devices/$region/extended_linear_cache_size" ]; then
> + cache_size=$(cat /sys/bus/cxl/devices/"$region"/extended_linear_cache_size)
> fi
>
> # Offset within extended linear cache (if cache_size > 0)
> --
> 2.51.0
>