On 2025-04-24 02:18, Jan Beulich wrote:
On 23.04.2025 23:22, Jason Andryuk wrote:--- a/xen/drivers/vpci/msix.c +++ b/xen/drivers/vpci/msix.c @@ -666,7 +666,7 @@ int vpci_make_msix_hole(const struct pci_dev *pdev) put_gfn(d, start); gprintk(XENLOG_WARNING, "%pp: existing mapping (mfn: %" PRI_mfn - "type: %d) at %#lx clobbers MSIX MMIO area\n", + " type: %d) at %#lx clobbers MSIX MMIO area\n",Imo this is a good indication that the format string better wouldn't be extending across multiple lines. Then it would also be possible to grep for it (in the sources) using e.g. "existing mapping .* clobbers".
Sure, a single line works for me. Thanks, Jason
