linux-kernel - Re: [PATCH V3 16/20] cxl/mem: Preserve cxl root decoder during mem probe

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: <680779399.221759726505299.JavaMail.epsvc@epcpadp1new>
Date: Mon, 29 Sep 2025 19:43:54 +0530
From: Neeraj Kumar <s.neeraj@...sung.com>
To: Alison Schofield <alison.schofield@...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 V3 16/20] cxl/mem: Preserve cxl root decoder during mem
 probe

On 24/09/25 02:38PM, Alison Schofield wrote:
>On Wed, Sep 17, 2025 at 07:11:12PM +0530, Neeraj Kumar wrote:
>> Saved root decoder info is required for cxl region persistency
>
>It seem there must be a more detailed story here.
>Saving the root decoder in struct cxl_memdev does not sound
>persistent. Please add more detail on how this step fits
>into the grander scheme.
>

Yes it has story. For region creation using cxl_create_region()
we need to know the root decoder. 

In current case cxl root decocer is provided from ndctl using device
attribute create_region_store(). And saving root decoder (cxlrd) in
cxl_memdev(cxlmd) helps to get root decoder during region creation.

Actually, cxl_memdev instance is saved in struct cxl_nvdimm(cxl_nvd). So
after parsing region information from nvdimm driver we recreate region
and thus cxlmd is available from cxl_nvd along with saved cxlrd

I will elaborate commit message with above information

Regards,
Neeraj