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

2024-08-23 Thread Fan Ni
On Fri, Aug 16, 2024 at 09:44:26AM -0500, ira.we...@intel.com wrote: > From: Navneet Singh > > A dynamic capacity device (DCD) sends events to signal the host for > changes in the availability of Dynamic Capacity (DC) memory. These > events contain extents describing a DPA range and meta data fo

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

2024-08-23 Thread Ira Weiny
Dave Jiang wrote: > > > 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

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

2024-08-23 Thread Jonathan Cameron
On Fri, 16 Aug 2024 09:44:27 -0500 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. > >

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

2024-08-23 Thread Jonathan Cameron
On Thu, 22 Aug 2024 21:58:02 -0500 Ira Weiny wrote: > Dave Jiang wrote: > > > > > > 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. > >

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

2024-08-23 Thread Jonathan Cameron
On Fri, 16 Aug 2024 09:44:25 -0500 Ira Weiny wrote: > cxl_dpa_to_region() finds the region from a tuple. > The search involves finding the device endpoint decoder as well. > > Dynamic capacity extent processing uses the endpoint decoder HPA > information to calculate the HPA offset. In additio

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

2024-08-23 Thread Jonathan Cameron
On Fri, 16 Aug 2024 09:44:24 -0500 ira.we...@intel.com wrote: > From: Navneet Singh > > Dynamic Capacity Devices (DCD) support extent change notifications > through the event log mechanism. The interrupt mailbox commands were > extended in CXL 3.1 to support these notifications. Firmware can't

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

2024-08-23 Thread Jonathan Cameron
On Fri, 16 Aug 2024 09:44:22 -0500 Ira Weiny wrote: > Dynamic Capacity Devices (DCD) require event interrupts to process > memory addition or removal. BIOS may have control over non-DCD event > processing. DCD interrupt configuration needs to be separate from > memory event interrupt configurat

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

2024-08-23 Thread Jonathan Cameron
On Fri, 16 Aug 2024 09:44:21 -0500 ira.we...@intel.com wrote: > From: Navneet Singh > > Dynamic Capacity CXL regions must allow memory to be added or removed > dynamically. In addition to the quantity of memory available the > location of the memory within a DC partition is dynamic based on the

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

2024-08-23 Thread Jonathan Cameron
On Fri, 16 Aug 2024 09:44:20 -0500 Ira Weiny wrote: > create_pmem_region_store() and create_ram_region_store() are identical > with the exception of the region mode. With the addition of DC region > mode this would end up being 3 copies of the same code. > > Refactor create_pmem_region_store()

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

2024-08-23 Thread Jonathan Cameron
On Thu, 22 Aug 2024 21:28:41 -0500 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 will need to know the details of the

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

2024-08-23 Thread Jonathan Cameron
On Fri, 16 Aug 2024 09:44:18 -0500 ira.we...@intel.com wrote: > From: Navneet Singh > > Endpoint decoder mode is used to represent the partition the decoder > points to such as ram or pmem. > > Expand the mode to allow a decoder to point to a specific DC partition > (Region). > > Signed-off-by

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

2024-08-23 Thread Jonathan Cameron
On Fri, 16 Aug 2024 09:44:17 -0500 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 th

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

2024-08-23 Thread Jonathan Cameron
On Fri, 16 Aug 2024 09:44:15 -0500 ira.we...@intel.com wrote: > From: Navneet Singh > > Until now region modes and decoder modes were equivalent in that both > modes were either PMEM or RAM. The addition of Dynamic > Capacity partitions defines up to 8 DC partitions per device. > > The region

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

2024-08-23 Thread Jonathan Cameron
On Fri, 16 Aug 2024 09:44:14 -0500 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. Witho

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

2024-08-23 Thread Jonathan Cameron
On Fri, 16 Aug 2024 09:44:11 -0500 Ira Weiny wrote: > The device DAX structure is being enhanced to track additional DCD > information. > > The current range tuple was not fully documented. Document it prior to > adding information for DC. > > Suggested-by: Jonathan Cameron > Signed-off-by: I

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 will need to know the details of the DC partitions available. >>>

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

2024-08-23 Thread Ira Weiny
Dave Jiang wrote: > > > On 8/16/24 7:44 AM, ira.we...@intel.com wrote: > > From: Navneet Singh > > [snip] > > diff --git a/drivers/cxl/core/extent.c b/drivers/cxl/core/extent.c > > index d7d526a51e2b..103b0bec3a4a 100644 > > --- a/drivers/cxl/core/extent.c > > +++ b/drivers/cxl/core/extent.c