From: Dave Jiang <dave.jiang@intel.com>
To: Alison Schofield <alison.schofield@intel.com>,
Davidlohr Bueso <dave@stgolabs.net>,
Jonathan Cameron <jonathan.cameron@huawei.com>,
Vishal Verma <vishal.l.verma@intel.com>,
Ira Weiny <ira.weiny@intel.com>,
Dan Williams <dan.j.williams@intel.com>
Cc: linux-cxl@vger.kernel.org
Subject: Re: [PATCH v6 0/2] cxl/region: Support unaligned address translations
Date: Fri, 16 Jan 2026 08:59:36 -0700 [thread overview]
Message-ID: <d5009511-496f-4e82-8692-5307f468ca88@intel.com> (raw)
In-Reply-To: <cover.1768538962.git.alison.schofield@intel.com>
On 1/15/26 9:58 PM, Alison Schofield wrote:
> Changes in v6:
> - Replace 64-bit division operators with div64_u64() helpers (lkp)
>
> Changes in v5:
> - s/reg_ways/region_ways in decode_pos (DaveJ)
> - Add comments in decode_pos
> - Update the CXL spec references to 4.0
>
> Changes in v4:
> - Rebased Patch 1 onto 6.19-rc4 with minimal conflict.
> - Added Patch 2 because 6.19 introduced poison by region offset
> meaning that too needed support for unaligned regions.
>
> Changes in v3:
> - Add bracketing to avert possibly uninitialized usage (lkp)
> - Remove excessive narration in decode_pos() (Jonathan)
> - Calculate devices_per_hb only once after switch in decode_pos() (Jonathan)
> - Reduce scope of hpa and shifted in unaligned_dpa_to_hpa() (Jonathan)
> - Unmix non- and assignment declarations in unaligned_dpa_to_hpa() (Jonathan)
> - Add blank line after return ULLONG_MAX in unaligned_dpa_to_hpa() (Jonathan)
> - s/index/position in for loop in unaligned_dpa_to_hpa() (Jonathan)
> - Initialize bool aligned true
> - Comment that restore_parent() math relies on power-of-2 granularity
> - Tidy-up the restore_parent() comment block
>
> Changes in v2:
> - Add 6 and 12 Host Bridge interleaves to decode_pos() (Jonathan)
> - Limit the unalignment check to MOD3 regions
> - Move the cache_size increment to a single place
> - Updated some in code comments
> - Rebase on v6.18-rc1
>
> Changes in v1 (was RFC):
> - Replace "/" with do_div() to quiet i386 build warning (lkp)
> - Replace 'cxld->interleave_ways' with 'hbiw' for clarity
> - Use div64_u64_rem() for alignment alignment
> - Fix up a printk format specifier (lkp)
> - Update code comments and commit log
> - Rebase on v6.17-rc7
>
>
> Begin Cover Letter:
> With the introduction of MOD3 interleave way support, platforms may
> create regions at starting addresses that are not power-of-2 aligned.
> This allows platforms to avoid gaps in the memory map, but addresses
> within those regions cannot be translated using the existing bit
> manipulation method.
>
> Introduce support for translating these unaligned addresses.
>
>
> Alison Schofield (2):
> cxl/region: Translate DPA->HPA in unaligned MOD3 regions
> cxl/region: Translate HPA to DPA and memdev in unaligned regions
>
> drivers/cxl/core/region.c | 212 ++++++++++++++++++++++++++++++++++++--
> 1 file changed, 204 insertions(+), 8 deletions(-)
>
>
> base-commit: 9ace4753a5202b02191d54e9fdf7f9e3d02b85eb
Reapplied to cxl/next
19885bd10755b9f737742fef8a14cbf554cd84e8
78b50b5984629d362f826e9615ce4599f429716e
prev parent reply other threads:[~2026-01-16 15:59 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-01-16 4:58 [PATCH v6 0/2] cxl/region: Support unaligned address translations Alison Schofield
2026-01-16 4:58 ` [PATCH v6 1/2] cxl/region: Translate DPA->HPA in unaligned MOD3 regions Alison Schofield
2026-01-16 4:58 ` [PATCH v6 2/2] cxl/region: Translate HPA to DPA and memdev in unaligned regions Alison Schofield
2026-01-16 15:59 ` Dave Jiang [this message]
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=d5009511-496f-4e82-8692-5307f468ca88@intel.com \
--to=dave.jiang@intel.com \
--cc=alison.schofield@intel.com \
--cc=dan.j.williams@intel.com \
--cc=dave@stgolabs.net \
--cc=ira.weiny@intel.com \
--cc=jonathan.cameron@huawei.com \
--cc=linux-cxl@vger.kernel.org \
--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