linux-kernel - Re: [RFC PATCH 01/20] nvdimm/label: Introduce NDD_CXL

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 for Android: free password hash cracker in your pocket

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

Message-ID: <1931444790.41751601485259.JavaMail.epsvc@epcpadp2new>
Date: Thu, 3 Jul 2025 15:28:05 +0530
From: Neeraj Kumar <s.neeraj@...sung.com>
To: Ira Weiny <ira.weiny@...el.com>
Cc: dan.j.williams@...el.com, dave@...olabs.net,
	jonathan.cameron@...wei.com, dave.jiang@...el.com,
	alison.schofield@...el.com, vishal.l.verma@...el.com,
	a.manzanares@...sung.com, nifan.cxl@...il.com, anisa.su@...sung.com,
	vishak.g@...sung.com, krish.reddy@...sung.com, arun.george@...sung.com,
	alok.rathore@...sung.com, neeraj.kernel@...il.com,
	linux-kernel@...r.kernel.org, linux-cxl@...r.kernel.org,
	nvdimm@...ts.linux.dev, gost.dev@...sung.com, cpgs@...sung.com
Subject: Re: [RFC PATCH 01/20] nvdimm/label: Introduce NDD_CXL_LABEL flag to
 set cxl label format

On 02/07/25 01:02PM, Ira Weiny wrote:
>Neeraj Kumar wrote:
>> NDD_CXL_LABEL is introduced to set cxl LSA 2.1 label format
>> Accordingly updated label index version
>
>I'm not following why CXL specific code needs to be in nvdimm?

Hi Ira,

Prior to LSA 2.1 version, LSA contain only namespace labels. LSA 2.1 was
introduced in CXL 2.0 Spec, which introduced region label along with
namespace label.

NDD_LABELING is the flag used to inform nvdimm driver about namespace
labels. As region label was introduced in cxl spec. so, I have used
NDD_CXL_LABEL flag to inform nvdimmm driver about it.

I have taken this naming reference from "drivers/nvdimm/label.h"
where region label is defined as "struct cxl_region_label"

Please let me know if i should use some other name in place of this

>
>I did not get a cover letter in this thread and looking at lore I don't
>see one either:
>
>https://lore.kernel.org/all/158453976.61750165203630.JavaMail.epsvc@epcpadp1new/
>
>So perhaps I am completely out of the loop here?  Could you point me at a
>cover letter?

Yes, there seems some issue due to which my patch thread got broken into two.
1. Cover letter:
https://lore.kernel.org/linux-cxl/1931444790.41750165203442.JavaMail.epsvc@epcpadp1new/

2. Rest of the thread:
https://lore.kernel.org/linux-cxl/158453976.61750165203630.JavaMail.epsvc@epcpadp1new/

Regards,
Neeraj