Commit ea24be6cdd for qemu.org

commit ea24be6cdd9def87b354869ed9c76049d44bb902
Author: Alireza Sanaee <alireza.sanaee@huawei.com>
Date:   Wed Feb 4 15:00:00 2026 +0000

    hw/cxl: Use HPA in cxl_cfmws_find_device() rather than offset in window.

    This function will shortly be used to help find if there is a route to a
    device, serving an HPA, under a particular fixed memory window. Rather than
    having that new use case subtract the base address in the caller, only to
    add it again in cxl_cfmws_find_device(), push the responsibility for
    calculating the HPA to the caller.

    This also reduces the inconsistency in the meaning of the hwaddr addr
    parameter between this function and the calls made within it that access
    the HDM decoders that operating on HPA.

    Reviewed-by: Li Zhijian <lizhijian@fujitsu.com>
    Reviewed-by: Jonathan Cameron <jonathan.cameron@huawei.com>
    Reviewed-by: Gregory Price <gourry@gourry.net>
    Tested-by: Gregory Price <gourry@gourry.net>
    Signed-off-by: Alireza Sanaee <alireza.sanaee@huawei.com>
    Message-ID: <20260318171918.146-2-alireza.sanaee@huawei.com>
    Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>

diff --git a/hw/cxl/cxl-host.c b/hw/cxl/cxl-host.c
index f3479b1991..a94b893e99 100644
--- a/hw/cxl/cxl-host.c
+++ b/hw/cxl/cxl-host.c
@@ -168,9 +168,6 @@ static PCIDevice *cxl_cfmws_find_device(CXLFixedWindow *fw, hwaddr addr)
     bool target_found;
     PCIDevice *rp, *d;

-    /* Address is relative to memory region. Convert to HPA */
-    addr += fw->base;
-
     rb_index = (addr / cxl_decode_ig(fw->enc_int_gran)) % fw->num_targets;
     hb = PCI_HOST_BRIDGE(fw->target_hbs[rb_index]->cxl_host_bridge);
     if (!hb || !hb->bus || !pci_bus_is_cxl(hb->bus)) {
@@ -254,7 +251,7 @@ static MemTxResult cxl_read_cfmws(void *opaque, hwaddr addr, uint64_t *data,
     CXLFixedWindow *fw = opaque;
     PCIDevice *d;

-    d = cxl_cfmws_find_device(fw, addr);
+    d = cxl_cfmws_find_device(fw, addr + fw->base);
     if (d == NULL) {
         *data = 0;
         /* Reads to invalid address return poison */
@@ -271,7 +268,7 @@ static MemTxResult cxl_write_cfmws(void *opaque, hwaddr addr,
     CXLFixedWindow *fw = opaque;
     PCIDevice *d;

-    d = cxl_cfmws_find_device(fw, addr);
+    d = cxl_cfmws_find_device(fw, addr + fw->base);
     if (d == NULL) {
         /* Writes to invalid address are silent */
         return MEMTX_OK;