linux-kernel - Re: [PATCH 02/26] cxl/core: Separate region mode from decoder mode

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: <b6ff085c-8cdc-4867-83fd-9566d59d5946@intel.com>
Date: Thu, 28 Mar 2024 13:09:38 -0700
From: Dave Jiang <dave.jiang@...el.com>
To: Ira Weiny <ira.weiny@...el.com>, Davidlohr Bueso <dave@...olabs.net>
Cc: Fan Ni <fan.ni@...sung.com>,
 Jonathan Cameron <Jonathan.Cameron@...wei.com>,
 Navneet Singh <navneet.singh@...el.com>,
 Dan Williams <dan.j.williams@...el.com>,
 Alison Schofield <alison.schofield@...el.com>,
 Vishal Verma <vishal.l.verma@...el.com>, linux-btrfs@...r.kernel.org,
 linux-cxl@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 02/26] cxl/core: Separate region mode from decoder mode



On 3/27/24 10:22 PM, Ira Weiny wrote:
> Davidlohr Bueso wrote:
>> On Sun, 24 Mar 2024, ira.weiny@...el.com wrote:
>>
>>> From: Navneet Singh <navneet.singh@...el.com>
>>>
>>> Until now region modes and decoder modes were equivalent in that they
>>> were either PMEM or RAM.  With the upcoming addition of Dynamic Capacity
>>> regions (which will represent an array of device regions [better named
>>> partitions] the index of which could be different on different
>>> interleaved devices), the mode of an endpoint decoder and a region will
>>> no longer be equivalent.
>>>
>>> Define a new region mode enumeration and adjust the code for it.
>>
>> Could this could also be picked up regardless of dcd?
> 
> It could but there is no need for it without DCD.
> 
> I will work on re-ordering the cleanups if Dave will agree to take them
> early.

There's no reason for the change unless it comes with DCD right? And probably no urgent need to taking it ahead then?
> 
> Ira