NVDIMM Device and Persistent Memory development
 help / color / mirror / Atom feed
From: Ira Weiny <ira.weiny@intel.com>
To: Dave Jiang <dave.jiang@intel.com>, Fan Ni <fan.ni@samsung.com>,
	 Jonathan Cameron <Jonathan.Cameron@huawei.com>,
	 Navneet Singh <navneet.singh@intel.com>,
	Jonathan Corbet <corbet@lwn.net>,
	 Andrew Morton <akpm@linux-foundation.org>
Cc: Dan Williams <dan.j.williams@intel.com>,
	 Davidlohr Bueso <dave@stgolabs.net>,
	 Alison Schofield <alison.schofield@intel.com>,
	 Vishal Verma <vishal.l.verma@intel.com>,
	Ira Weiny <ira.weiny@intel.com>,
	 linux-cxl@vger.kernel.org, linux-doc@vger.kernel.org,
	 nvdimm@lists.linux.dev, linux-kernel@vger.kernel.org
Subject: [PATCH v6 03/27] dax: Document struct dev_dax_range
Date: Tue, 05 Nov 2024 12:38:25 -0600	[thread overview]
Message-ID: <20241105-dcd-type2-upstream-v6-3-85c7fa2140fe@intel.com> (raw)
In-Reply-To: <20241105-dcd-type2-upstream-v6-0-85c7fa2140fe@intel.com>

The device DAX structure is being enhanced to track additional DCD
information.  Specifically the range tuple needs additional parameters.
The current range tuple is not fully documented and is large enough to
warrant its own definition.

Separate the struct dax_dev_range definition and document it prior to
adding information for DC.

Suggested-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Reviewed-by: Dave Jiang <dave.jiang@intel.com>
Signed-off-by: Ira Weiny <ira.weiny@intel.com>
---
 drivers/dax/dax-private.h | 26 ++++++++++++++++++++------
 1 file changed, 20 insertions(+), 6 deletions(-)

diff --git a/drivers/dax/dax-private.h b/drivers/dax/dax-private.h
index 446617b73aeab2e6f5a2ec3ca4c3f740e1b3e719..0867115aeef2e1b2d4c88b5c38b6648a404b1060 100644
--- a/drivers/dax/dax-private.h
+++ b/drivers/dax/dax-private.h
@@ -40,12 +40,30 @@ struct dax_region {
 	struct device *youngest;
 };
 
+/**
+ * struct dax_mapping - device to display mapping range attributes
+ * @dev: device representing this range
+ * @range_id: index within dev_dax ranges array
+ * @id: ida of this mapping
+ */
 struct dax_mapping {
 	struct device dev;
 	int range_id;
 	int id;
 };
 
+/**
+ * struct dev_dax_range - tuple represenging a range of memory used by dev_dax
+ * @pgoff: page offset
+ * @range: resource-span
+ * @mapping: reference to the dax_mapping for this range
+ */
+struct dev_dax_range {
+	unsigned long pgoff;
+	struct range range;
+	struct dax_mapping *mapping;
+};
+
 /**
  * struct dev_dax - instance data for a subdivision of a dax region, and
  * data while the device is activated in the driver.
@@ -58,7 +76,7 @@ struct dax_mapping {
  * @dev - device core
  * @pgmap - pgmap for memmap setup / lifetime (driver owned)
  * @nr_range: size of @ranges
- * @ranges: resource-span + pgoff tuples for the instance
+ * @ranges: range tuples of memory used
  */
 struct dev_dax {
 	struct dax_region *region;
@@ -72,11 +90,7 @@ struct dev_dax {
 	struct dev_pagemap *pgmap;
 	bool memmap_on_memory;
 	int nr_range;
-	struct dev_dax_range {
-		unsigned long pgoff;
-		struct range range;
-		struct dax_mapping *mapping;
-	} *ranges;
+	struct dev_dax_range *ranges;
 };
 
 /*

-- 
2.47.0


  parent reply	other threads:[~2024-11-05 18:38 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-11-05 18:38 [PATCH v6 00/27] DCD: Add support for Dynamic Capacity Devices (DCD) Ira Weiny
2024-11-05 18:38 ` [PATCH v6 01/27] range: Add range_overlaps() Ira Weiny
2024-11-05 18:38 ` [PATCH v6 02/27] ACPI/CDAT: Add CDAT/DSMAS shared and read only flag values Ira Weiny
2024-11-05 20:04   ` Rafael J. Wysocki
2024-11-05 18:38 ` Ira Weiny [this message]
2024-11-05 18:38 ` [PATCH v6 04/27] cxl/pci: Delay event buffer allocation Ira Weiny
2024-11-05 18:38 ` [PATCH v6 05/27] cxl/hdm: Use guard() in cxl_dpa_set_mode() Ira Weiny
2024-11-07 11:08   ` Jonathan Cameron
2024-11-07 19:46     ` Ira Weiny
2024-11-05 18:38 ` [PATCH v6 06/27] cxl/region: Refactor common create region code Ira Weiny
2024-11-05 18:38 ` [PATCH v6 07/27] cxl/mbox: Flag support for Dynamic Capacity Devices (DCD) ira.weiny
2024-11-05 18:38 ` [PATCH v6 08/27] cxl/mem: Read dynamic capacity configuration from the device ira.weiny
2024-11-05 18:38 ` [PATCH v6 09/27] cxl/core: Separate region mode from decoder mode ira.weiny
2024-11-05 18:38 ` [PATCH v6 10/27] cxl/region: Add dynamic capacity decoder and region modes ira.weiny
2024-11-05 18:38 ` [PATCH v6 11/27] cxl/hdm: Add dynamic capacity size support to endpoint decoders ira.weiny
2024-11-05 18:38 ` [PATCH v6 12/27] cxl/cdat: Gather DSMAS data for DCD regions Ira Weiny
2024-11-05 18:38 ` [PATCH v6 13/27] cxl/mem: Expose DCD partition capabilities in sysfs ira.weiny
2024-11-05 18:38 ` [PATCH v6 14/27] cxl/port: Add endpoint decoder DC mode support to sysfs ira.weiny
2024-11-05 18:38 ` [PATCH v6 15/27] cxl/region: Add sparse DAX region support ira.weiny
2024-11-05 18:38 ` [PATCH v6 16/27] cxl/events: Split event msgnum configuration from irq setup Ira Weiny
2024-11-05 18:38 ` [PATCH v6 17/27] cxl/pci: Factor out interrupt policy check Ira Weiny
2024-11-05 18:38 ` [PATCH v6 18/27] cxl/mem: Configure dynamic capacity interrupts ira.weiny
2024-11-05 18:38 ` [PATCH v6 19/27] cxl/core: Return endpoint decoder information from region search Ira Weiny
2024-11-05 18:38 ` [PATCH v6 20/27] cxl/extent: Process DCD events and realize region extents ira.weiny
2024-11-06 15:13   ` Li, Ming4
2024-11-07 11:29   ` Jonathan Cameron
2024-11-05 18:38 ` [PATCH v6 21/27] cxl/region/extent: Expose region extent information in sysfs ira.weiny
2024-11-05 18:38 ` [PATCH v6 22/27] dax/bus: Factor out dev dax resize logic Ira Weiny
2024-11-05 18:38 ` [PATCH v6 23/27] dax/region: Create resources on sparse DAX regions ira.weiny
2024-11-07 11:32   ` Jonathan Cameron
2024-11-05 18:38 ` [PATCH v6 24/27] cxl/region: Read existing extents on region creation ira.weiny
2024-11-05 18:38 ` [PATCH v6 25/27] cxl/mem: Trace Dynamic capacity Event Record ira.weiny
2024-11-05 18:38 ` [PATCH v6 26/27] tools/testing/cxl: Make event logs dynamic Ira Weiny
2024-11-05 18:38 ` [PATCH v6 27/27] tools/testing/cxl: Add DC Regions to mock mem data Ira Weiny

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20241105-dcd-type2-upstream-v6-3-85c7fa2140fe@intel.com \
    --to=ira.weiny@intel.com \
    --cc=Jonathan.Cameron@huawei.com \
    --cc=akpm@linux-foundation.org \
    --cc=alison.schofield@intel.com \
    --cc=corbet@lwn.net \
    --cc=dan.j.williams@intel.com \
    --cc=dave.jiang@intel.com \
    --cc=dave@stgolabs.net \
    --cc=fan.ni@samsung.com \
    --cc=linux-cxl@vger.kernel.org \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=navneet.singh@intel.com \
    --cc=nvdimm@lists.linux.dev \
    --cc=vishal.l.verma@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox