[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[PATCH v2 1/1] tools/libs/light: fix BAR memory address truncation


  • To: <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: Jiqian Chen <Jiqian.Chen@xxxxxxx>
  • Date: Mon, 13 Oct 2025 15:45:33 +0800
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=lists.xenproject.org smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0)
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=QKkvZoHOBdDKlx71dnvWCdLS+ZSNf7woIcpOlU/GFLk=; b=vhPZheAuZjdEL9t+4OjJRS1iz0qCc2NBHUpE2q8lCazlbbc8/8hqzn+ytZ6tpFcOOctX7W4zqw4DY7qhfT7+Zi+RevOmu/jTibwAhRZ7OGLD+HQ2OuxKknUQar122yJZZVG7nC5Nm9VCEHPVMXwPkZgJl1aTUgCUicpkslPhFoCFHVrtufhLJHXiOHV2XWJCg45DjXkzEoh879uvO3cDyyj01QEUZyAYQ0pS5XjdHZCvVjkul3bGP/vfyY9Ms3NvPQLp4CRTLt3wBXYSpX+SPWMKxmQLAEvSiNi4bYh/txb+r0C7KLw9mm+aSF/LGDYPbjhhM9FkwD5RdlquE/0mYQ==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=UofJTwzL74iEzfUvWcpiaJN+95B501sTmwFnWXrrrkQYXfJcDa3ktGqxWCdJd0rrsBsagLNcLzLiMm4dLHeVlrMNL4a6mQE6O0igO8Bz4+QdpvecqVpOld/64MdFGwx2/gDldTrPFBgrkpBM3pqGy/NL6tDwfi7T46DOEtukGh+G/LRPcRDeekcr2Nr2F3zihahJ1/X3tx1T7XQpUP4vMrITduGiJrpGsXzhFaXnhaf1P8VFNAHD1OvpjHr9UHMJsq3fRWLa2iRT6XmQ/NkV74NUAqFvO0ocNEym7X+zapWufCGZ3v6+suhSHD2Re188Ka+SdThBIYu3RljYybGS4A==
  • Cc: Huang Rui <ray.huang@xxxxxxx>, Jiqian Chen <Jiqian.Chen@xxxxxxx>, "Oleksii Kurochko" <oleksii.kurochko@xxxxxxxxx>, Anthony PERARD <anthony.perard@xxxxxxxxxx>, Juergen Gross <jgross@xxxxxxxx>
  • Delivery-date: Mon, 13 Oct 2025 07:46:01 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

64-bit BAR memory address is truncated when removing a passthrough
pci device from guest since it uses "unsigned int".

So, change to use 64-bit type to fix this problem.

Fixes: b0a1af61678b ("libxenlight: implement pci passthrough")
Signed-off-by: Jiqian Chen <Jiqian.Chen@xxxxxxx>
Release-Acked-by: Oleksii Kurochko <oleksii.kurochko@xxxxxxxxx>
---
cc: Anthony PERARD <anthony.perard@xxxxxxxxxx>
cc: Juergen Gross <jgross@xxxxxxxx>
---
v1->v2 changes:
* Change to use uint64_t instead of unsigned long long and change
  corresponding fscanf and LOGED parameters.
---
 tools/libs/light/libxl_pci.c | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/tools/libs/light/libxl_pci.c b/tools/libs/light/libxl_pci.c
index 2ea2caeb6624..ae416a162360 100644
--- a/tools/libs/light/libxl_pci.c
+++ b/tools/libs/light/libxl_pci.c
@@ -2001,7 +2001,8 @@ static void pci_remove_detached(libxl__egc *egc,
 {
     STATE_AO_GC(prs->aodev->ao);
     libxl_ctx *ctx = libxl__gc_owner(gc);
-    unsigned int start = 0, end = 0, flags = 0, size = 0, irq = 0;
+    uint64_t start = 0, end = 0, flags = 0, size = 0;
+    unsigned int irq = 0;
     int i, stubdomid = 0;
     const char *sysfs_path;
     FILE *f;
@@ -2031,7 +2032,7 @@ static void pci_remove_detached(libxl__egc *egc,
     }
 
     for (i = 0; i < PROC_PCI_NUM_RESOURCES; i++) {
-        if (fscanf(f, "0x%x 0x%x 0x%x\n", &start, &end, &flags) != 3)
+        if (fscanf(f, "0x%"SCNx64" 0x%"SCNx64" 0x%"SCNx64"\n", &start, &end, 
&flags) != 3)
             continue;
         size = end - start + 1;
         if (start) {
@@ -2040,7 +2041,7 @@ static void pci_remove_detached(libxl__egc *egc,
                                                  size, 0);
                 if (rc < 0)
                     LOGED(ERROR, domid,
-                          "xc_domain_ioport_permission error 0x%x/0x%x",
+                          "xc_domain_ioport_permission error 
%#"PRIx64"/%#"PRIx64,
                           start,
                           size);
             } else {
@@ -2050,7 +2051,7 @@ static void pci_remove_detached(libxl__egc *egc,
                                                 0);
                 if (rc < 0)
                     LOGED(ERROR, domid,
-                          "xc_domain_iomem_permission error 0x%x/0x%x",
+                          "xc_domain_ioport_permission error 
%#"PRIx64"/%#"PRIx64,
                           start,
                           size);
             }
-- 
2.34.1




 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.