linux-kernel - Re: [PATCH V4 15/17] cxl/pmem_region: Add sysfs attribute cxl region label updation/deletion

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <158453976.61768145283486.JavaMail.epsvc@epcpadp2new>
Date: Fri, 9 Jan 2026 18:01:29 +0530
From: Neeraj Kumar <s.neeraj@...sung.com>
To: Dave Jiang <dave.jiang@...el.com>
Cc: linux-cxl@...r.kernel.org, nvdimm@...ts.linux.dev,
	linux-kernel@...r.kernel.org, gost.dev@...sung.com,
	a.manzanares@...sung.com, vishak.g@...sung.com, neeraj.kernel@...il.com,
	cpgs@...sung.com
Subject: Re: [PATCH V4 15/17] cxl/pmem_region: Add sysfs attribute cxl
 region label updation/deletion

On 19/11/25 04:10PM, Dave Jiang wrote:
>
>
>On 11/19/25 12:52 AM, Neeraj Kumar wrote:
>> Using these attributes region label is added/deleted into LSA. These
>> attributes are called from userspace (ndctl) after region creation.
>>
>> Signed-off-by: Neeraj Kumar <s.neeraj@...sung.com>
>> ---
>>  Documentation/ABI/testing/sysfs-bus-cxl | 22 ++++++
>>  drivers/cxl/core/pmem_region.c          | 93 ++++++++++++++++++++++++-
>>  drivers/cxl/cxl.h                       |  7 ++
>>  3 files changed, 121 insertions(+), 1 deletion(-)
>>
>> diff --git a/Documentation/ABI/testing/sysfs-bus-cxl b/Documentation/ABI/testing/sysfs-bus-cxl
>> index c80a1b5a03db..76d79c03dde4 100644
>> --- a/Documentation/ABI/testing/sysfs-bus-cxl
>> +++ b/Documentation/ABI/testing/sysfs-bus-cxl
>> @@ -624,3 +624,25 @@ Description:
>>  		The count is persistent across power loss and wraps back to 0
>>  		upon overflow. If this file is not present, the device does not
>>  		have the necessary support for dirty tracking.
>> +
>> +
>> +What:		/sys/bus/cxl/devices/regionZ/pmem_regionZ/region_label_update
>> +Date:		Nov, 2025
>> +KernelVersion:	v6.19
>> +Contact:	linux-cxl@...r.kernel.org
>> +Description:
>> +		(RW) Write a boolean 'true' string value to this attribute to
>> +		update cxl region information into LSA as region label. It
>> +		uses nvdimm nd_region_label_update() to update cxl region
>> +		information saved during cxl region creation into LSA. This
>> +		attribute must be called at last during cxl region creation.
>
>Please consider:
>attribute must be written last during cxl region creation.
>
>No need to mention kernel specifics like function names. Just give general description of what the attribute does. Same for the next attribute below.
>
>Also, does this attribute needs to be readable? The documentation above does not explain the read attribute if so.
>
>DJ

Fixed it in V5


Regards,
Neeraj