Linux CXL
 help / color / mirror / Atom feed
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



      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