linux-kernel - Re: [PATCH V6 00/18] Add CXL LSA 2.1 format support in nvdimm and cxl pmem

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] [day] [month] [year] [list]

Message-ID: <20260123125459.00002ac9@huawei.com>
Date: Fri, 23 Jan 2026 12:54:59 +0000
From: Jonathan Cameron <jonathan.cameron@...wei.com>
To: Neeraj Kumar <s.neeraj@...sung.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>
Subject: Re: [PATCH V6 00/18] Add CXL LSA 2.1 format support in nvdimm and
 cxl pmem


Trivial comment on the change log. 

To me this series looks good now.

However, there are some other series touching some of the functions
you modify. E.g cxl_endpoint_port_probe()

I'm not suggesting you rebase on those (Dave might ask for it however!),
but it is probably useful to try a rebase and reply to this cover letter if
there are any significant issues.  There is a lot of code movement going
on so it might just be fuzz.

From the cxl tree, try it on top of the next branch.
https://git.kernel.org/pub/scm/linux/kernel/git/cxl/cxl.git/log/?h=next

Thanks,

Jonathan



> Changes in v5->v6
> -----------------
> - Find v5 link at [4]
> - Find v4 link at [5]
> - Find v3 link at [6]
> - Find v2 link at [7]
> - v1 patch-set was broken. Find the v1 links at [8] and [9]
> 
> [Misc]
> - Rebase with for-7.0/cxl-init
> 
> [PATCH 01/18]
> - Add Ira RB tag
> 
> [PATCH 03/18]
> - Use export_uuid() instead of uuid_copy() [Jonathan]
> - Add Jonathan RB tag
> - Add Ira RB tag
> 
> [PATCH 04/18]
> - Move the assignments at relevant place [Jonathan]
> - Rename lslot to label_slot [Jonathan]
> - Add Ira RB tag
> 
> [PATCH 05/18]
> - Add Jonathan RB tag
> - Add Ira RB tag
> 
> [PATCH 06/18]
> - Add Jonathan RB tag
> - Add Ira RB tag
> 
> [PATCH 07/18]
> - Add Ira RB tag
> 
> [PATCH 08/18]
> - use export_uuid() to avoid casts to uuid_t * [Jonathan]
> - Add Jonathan RB tag
> - Add Ira RB tag
> 
> [PATCH 09/18]
> - Add Jonathan RB tag
> - Add Ira RB tag
> 
> [PATCH 10/18]
> - Initialize struct cxl_memdev_attach variable [Jonathan]
> - Add Jonathan RB tag
> 
> [PATCH 10/18]
I briefly got confused when using this to see what chagned!
This one is patch 11
> - Its a new patch addition in this series
> - Seperate out renaming of __create_region() to cxl_create_region() [Jonathan]
> 
> [PATCH 11/18]
This patch 12.
> - Remove usage of cxl_dpa_free() from alloc_region_dpa() [Jonathan]
> 
> [PATCH 13/18]
> - Add Jonathan RB tag
> 
> [PATCH 15/18]
> - Add Jonathan RB tag
> 
> [PATCH 16/18]
> - Fix wrong name state with region_label_state [Jonathan]
> - Add Jonathan RB tag
> 
> [PATCH 17/18]
> - Optimize loop matching [Jonathan]
> - Use reverse xmas tree stype [Jonathan]
> 
> [PATCH 18/18]
> - Add Jonathan RB tag
>