Re: [PATCH v3 03/25] dax: Document dax dev range tuple

2024-08-16 Thread Dave Jiang
Signed-off-by: Ira Weiny Reviewed-by: Dave Jiang > > --- > Changes: > [iweiny: move to start of series] > --- > drivers/dax/dax-private.h | 5 - > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/drivers/dax/dax-private.h b/drivers/dax/dax-private.

Re: [PATCH v3 06/25] cxl/mem: Read dynamic capacity configuration from the device

2024-08-16 Thread Dave Jiang
On 8/16/24 7:44 AM, ira.we...@intel.com wrote: > From: Navneet Singh > > Devices which optionally support Dynamic Capacity (DC) are configured > via mailbox commands. CXL 3.1 requires the host to issue the Get DC > Configuration command in order to properly configure DCDs. Without the > Get

Re: [PATCH v3 07/25] cxl/core: Separate region mode from decoder mode

2024-08-16 Thread Dave Jiang
just the code to process these modes > independently. > > There is no equal to decoder mode dead in region modes. Avoid > constructing regions with decoders which have been flagged as dead. > > Suggested-by: Jonathan Cameron > Signed-off-by: Navneet Singh > Co-developed-by: Ir

Re: [PATCH v3 08/25] cxl/region: Add dynamic capacity decoder and region modes

2024-08-16 Thread Dave Jiang
-by: Navneet Singh > Co-developed-by: Ira Weiny > Signed-off-by: Ira Weiny Reviewed-by: Dave Jiang > > --- > Changes: > [iweiny: keep tags on simple patch] > [Fan: s/partitions/partition/] > [djiang: New wording for the commit message] > [iweiny: reword commit mess

Re: [PATCH v3 09/25] cxl/hdm: Add dynamic capacity size support to endpoint decoders

2024-08-16 Thread Dave Jiang
On 8/16/24 7:44 AM, ira.we...@intel.com wrote: > From: Navneet Singh > > To support Dynamic Capacity Devices (DCD) endpoint decoders will need to > map DC partitions (regions). In addition to assigning the size of the > DC partition, the decoder must assign any skip value from the previous >

Re: [PATCH v3 10/25] cxl/port: Add endpoint decoder DC mode support to sysfs

2024-08-16 Thread Dave Jiang
> Signed-off-by: Navneet Singh > Co-developed-by: Ira Weiny > Signed-off-by: Ira Weiny Reviewed-by: Dave Jiang > > --- > Changes: > [Fan: change mode range logic] > [Fan: use !resource_size()] > [djiang: use the static mode name string array in mode_store()] >

Re: [PATCH v3 11/25] cxl/mem: Expose DCD partition capabilities in sysfs

2024-08-16 Thread Dave Jiang
On 8/16/24 7:44 AM, ira.we...@intel.com wrote: > From: Navneet Singh > > To properly configure CXL regions on Dynamic Capacity Devices (DCD), > user space will need to know the details of the DC partitions available. > > Expose dynamic capacity capabilities through sysfs. > > Signed-off-by:

Re: [PATCH v3 12/25] cxl/region: Refactor common create region code

2024-08-16 Thread Dave Jiang
ion_store() and create_ram_region_store() to use > a single common function to be used in subsequent DC code. > > Suggested-by: Fan Ni > Signed-off-by: Ira Weiny Reviewed-by: Dave Jiang > --- > drivers/cxl/core/region.c | 28 +++- > 1 file changed, 11 inse

Re: [PATCH v3 13/25] cxl/region: Add sparse DAX region support

2024-08-16 Thread Dave Jiang
ax devices. There is no > known use case for range mapping on sparse regions. Avoid the > complication by preventing range mapping of dax devices on sparse > regions. > > Interleaving is deferred for now. Add checks. > > Signed-off-by: Navneet Singh > Co-d

Re: [PATCH v3 14/25] cxl/events: Split event msgnum configuration from irq setup

2024-08-16 Thread Dave Jiang
onfiguration. > > Split cxl_event_config_msgnums() from irq setup in preparation for > separate DCD interrupts configuration. > > Signed-off-by: Ira Weiny Reviewed-by: Dave Jiang > --- > drivers/cxl/pci.c | 24 > 1 file changed, 12 insertions(+), 12 de

Re: [PATCH v3 16/25] cxl/mem: Configure dynamic capacity interrupts

2024-08-16 Thread Dave Jiang
by the FW if FW first > has been selected by the BIOS. > > Signed-off-by: Navneet Singh > Co-developed-by: Ira Weiny > Signed-off-by: Ira Weiny Reviewed-by: Dave Jiang > > --- > Changes: > [iweiny: update commit message] > [iweiny: rebase to upstream irq code] &g

Re: [PATCH v3 17/25] cxl/core: Return endpoint decoder information from region search

2024-08-19 Thread Dave Jiang
addition, well behaved > extents should be contained within an endpoint decoder. > > Return the endpoint decoder found to be used in subsequent DCD code. > > Signed-off-by: Ira Weiny Reviewed-by: Dave Jiang > --- > drivers/cxl/core/core.h | 6 -- > drivers/cxl/c

Re: [PATCH v3 18/25] cxl/extent: Process DCD events and realize region extents

2024-08-19 Thread Dave Jiang
Process DCD events and create region devices. > > Signed-off-by: Navneet Singh > Co-developed-by: Ira Weiny > Signed-off-by: Ira Weiny A few nits below, but in general Reviewed-by: Dave Jiang > > --- > Changes: > [iweiny: combine this with the extent surface patches to be

Re: [PATCH v3 19/25] cxl/region/extent: Expose region extent information in sysfs

2024-08-19 Thread Dave Jiang
On 8/16/24 7:44 AM, ira.we...@intel.com wrote: > From: Navneet Singh > > Extent information can be helpful to the user to coordinate memory usage > with the external orchestrator and FM. > > Expose the details of region extents by creating the following > sysfs entries. > > /sys/bus/

Re: [PATCH v3 20/25] dax/bus: Factor out dev dax resize logic

2024-08-19 Thread Dave Jiang
e tree > which is less complicated for finding space. > > In preparation for this change, factor out the dev_dax_resize logic. > For static regions use dax_region->res as the parent to find space for > the dax ranges. Future patches will use the same algorithm with > individual exte

Re: [PATCH v3 21/25] dax/region: Create resources on sparse DAX regions

2024-08-19 Thread Dave Jiang
On 8/16/24 7:44 AM, ira.we...@intel.com wrote: > From: Navneet Singh > > DAX regions which map dynamic capacity partitions require that memory be > allowed to come and go. Recall sparse regions were created for this > purpose. Now that extents can be realized within DAX regions the DAX > reg

Re: [PATCH v3 22/25] cxl/region: Read existing extents on region creation

2024-08-19 Thread Dave Jiang
On 8/16/24 7:44 AM, ira.we...@intel.com wrote: > From: Navneet Singh > > Dynamic capacity device extents may be left in an accepted state on a > device due to an unexpected host crash. In this case it is expected > that the creation of a new region on top of a DC partition can read > those ex

Re: [PATCH v3 23/25] cxl/mem: Trace Dynamic capacity Event Record

2024-08-20 Thread Dave Jiang
ed-off-by: Navneet Singh > Signed-off-by: Ira Weiny Reviewed-by: Dave Jiang small nit below > > --- > Changes: > [Alison: Update commit message] > --- > drivers/cxl/core/mbox.c | 4 +++ > drivers/cxl/core/trace.h | 65 >

Re: [PATCH v3 24/25] tools/testing/cxl: Make event logs dynamic

2024-08-20 Thread Dave Jiang
e concurrency required when user space is allowed to > control DCD extents > > Signed-off-by: Ira Weiny Reviewed-by: Dave Jiang > > --- > Changes: > [iweiny: rebase] > --- > tools/testing/cxl/test/mem.c | 278 > ++- > 1

Re: [PATCH v3 11/25] cxl/mem: Expose DCD partition capabilities in sysfs

2024-08-23 Thread Dave Jiang
On 8/22/24 7:28 PM, Ira Weiny wrote: > Dave Jiang wrote: >> >> >> On 8/16/24 7:44 AM, ira.we...@intel.com wrote: >>> From: Navneet Singh >>> >>> To properly configure CXL regions on Dynamic Capacity Devices (DCD), >>> user space wil

Re: [PATCH v7 00/27] DCD: Add support for Dynamic Capacity Devices (DCD)

2024-11-08 Thread Dave Jiang
On 11/7/24 1:58 PM, Ira Weiny wrote: > A git tree of this series can be found here: > > https://github.com/weiny2/linux-kernel/tree/dcd-v4-2024-11-07 > > This is a quick spin with minor clean ups Dave was going to apply as > well as a couple of clean ups I had slated for after V4 landed.

Re: [PATCH v5 00/27] DCD: Add support for Dynamic Capacity Devices (DCD)

2024-10-31 Thread Dave Jiang
On 10/30/24 7:48 AM, Jonathan Cameron wrote: > On Tue, 29 Oct 2024 15:34:35 -0500 > Ira Weiny wrote: > >> A git tree of this series can be found here: >> >> https://github.com/weiny2/linux-kernel/tree/dcd-v4-2024-10-29 >> >> Series info >> === >> >> This series has 4 parts: >> >>