From 888cdbd9f1efc35bae9cba26e2d6466e1c78de47 Mon Sep 17 00:00:00 2001 From: Jason Andryuk Date: Thu, 24 Apr 2025 17:23:26 -0400 Subject: [PATCH] xen/vpci: Fix msix existing mapping printk MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit The format string lacks a space, so mfn and type run together: (XEN) d0v0 0000:06:00.7: existing mapping (mfn: 753037type: 0) at 0x1 clobbers MSIX MMIO area Add a space. Additionally, move the format string to a single long line to improve grep-ability. Fixes: 677053fac17a ("vpci/msix: carve p2m hole for MSIX MMIO regions") Signed-off-by: Jason Andryuk Reviewed-by: Roger Pau Monné --- xen/drivers/vpci/msix.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/xen/drivers/vpci/msix.c b/xen/drivers/vpci/msix.c index f3804ce047..3568f2a651 100644 --- a/xen/drivers/vpci/msix.c +++ b/xen/drivers/vpci/msix.c @@ -665,8 +665,7 @@ int vpci_make_msix_hole(const struct pci_dev *pdev) default: put_gfn(d, start); gprintk(XENLOG_WARNING, - "%pp: existing mapping (mfn: %" PRI_mfn - "type: %d) at %#lx clobbers MSIX MMIO area\n", + "%pp: existing mapping (mfn: %" PRI_mfn " type: %d) at %#lx clobbers MSIX MMIO area\n", &pdev->sbdf, mfn_x(mfn), t, start); return -EEXIST; } -- 2.39.5