[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <158453976.61751601485478.JavaMail.epsvc@epcpadp2new>
Date: Thu, 3 Jul 2025 15:34:21 +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 02/20] nvdimm/label: Prep patch to accommodate cxl
lsa 2.1 support
On 02/07/25 12:55PM, Ira Weiny wrote:
>Neeraj Kumar wrote:
>> In order to accommodate cxl lsa 2.1 format region label, renamed
>> nd_namespace_label to nd_lsa_label.
>
>This does not really make it clear why a name change is required.
>
>Could you elaborate here?
Hi Ira,
LSA 2.1 format introduces region label, which can also reside into LSA
along with only namespace label as per v1.1 and v1.2
As both namespace and region labels are of same size, i.e, 256 bytes.
Therefore I have introduced nd_lsa_label as following
struct nd_lsa_label {
union {
struct nd_namespace_label ns_label;
struct cxl_region_label rg_label;
};
};
Sure, I will update commit message with above information in next
patch-set.
Thanks,
Neeraj
Powered by blists - more mailing lists