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

[Neeraj Kumar <s.neeraj@samsung.com>]

[-- Attachment #1: Type: text/plain, Size: 1548 bytes --]

On 23/01/26 10:52AM, Dave Jiang wrote:
>
>
>On 1/23/26 4:30 AM, Neeraj Kumar wrote:
>> Introduction:
>> =============
>> CXL Persistent Memory (Pmem) devices region, namespace and content must be
>> persistent across system reboot. In order to achieve this persistency, it
>> uses Label Storage Area (LSA) to store respective metadata. During system
>> reboot, stored metadata in LSA is used to bring back the region, namespace
>> and content of CXL device in its previous state.
>> CXL specification provides Get_LSA (4102h) and Set_LSA (4103h) mailbox
>> commands to access the LSA area. nvdimm driver is using same commands to
>> get/set LSA data.
>
>Hi Neeraj,
>I'm attempting to vet the series before applying to cxl/next. But it seems to be failing multiple unit regression tests in cxl_test. We need to figure out why before the series can be merged. Most likely the changes in this series also require cxl_test updates. Can you please take a look? I will also take a look and try to help you address the issues. Thanks
>
>Ok:                 10
>Expected Fail:      0
>Fail:               4
>Unexpected Pass:    0
>Skipped:            1
>Timeout:            0
>
>The script below [1] can get you up and running with a qemu environment where you can run cxl_test.
>
>.../run_qemu/run_qemu.sh --cxl --cxl-test --cxl-debug --hmat -r wipe
>
>cd ndctl
>meson test -C build --suite=cxl
>
>[1] https://github.com/pmem/run_qemu
>
>DJ

Sure Dave,

I am able to reproduce the same, I will analyze and fix them in next patch series.


Regards,
Neeraj

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]