linux-kernel - Re: [PATCH 08/26] cxl/mem: Expose device dynamic capacity capabilities

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]

Date: Thu, 4 Apr 2024 09:44:22 +0100
From: Jonathan Cameron <Jonathan.Cameron@...wei.com>
To: fan <nifan.cxl@...il.com>
CC: Davidlohr Bueso <dave@...olabs.net>, <ira.weiny@...el.com>, Dave Jiang
	<dave.jiang@...el.com>, Fan Ni <fan.ni@...sung.com>, Navneet Singh
	<navneet.singh@...el.com>, Dan Williams <dan.j.williams@...el.com>, "Alison
 Schofield" <alison.schofield@...el.com>, Vishal Verma
	<vishal.l.verma@...el.com>, <linux-btrfs@...r.kernel.org>,
	<linux-cxl@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 08/26] cxl/mem: Expose device dynamic capacity
 capabilities

On Tue, 26 Mar 2024 11:30:38 -0700
fan <nifan.cxl@...il.com> wrote:

> On Mon, Mar 25, 2024 at 04:40:16PM -0700, Davidlohr Bueso wrote:
> > On Sun, 24 Mar 2024, ira.weiny@...el.com wrote:
> >   
> > > +What:		/sys/bus/cxl/devices/memX/dc/region_count
> > > +Date:		June, 2024
> > > +KernelVersion:	v6.10
> > > +Contact:	linux-cxl@...r.kernel.org
> > > +Description:
> > > +		(RO) Number of Dynamic Capacity (DC) regions supported on the
> > > +		device.  May be 0 if the device does not support Dynamic
> > > +		Capacity.  
> > 
> > If dcd is not supported then we should not have the dc/ directory
> > altogether.
> > 
> > Thanks,
> > Davidlohr   
> 
> I also think so. However, I also noticed one thing (not DCD related).
> Even for a PMEM device, for example, we have a ram directory under the
> device directory.

True, but it's new ABI so we don't have to copy and Dan's patch to
allow for hiding static attribute directories has landed in the meantime.
So I vote for hiding it.

> 
> ===================
> root@DT:~# cxl list
> [
>   {
>     "memdev":"mem0",
>     "pmem_size":536870912,
>     "serial":0,
>     "host":"0000:0d:00.0"
>   }
> ]
> root@DT:~# ls /sys/bus/cxl/devices/mem0/
> dc  dev  driver  firmware  firmware_version  label_storage_size  numa_node  payload_max  pmem  pmem0  ram  security  serial  subsystem	trigger_poison_list  uevent
> root@DT:~#
> ===================
> 
> Fan
>