|
 |
357786 |
From 95f87089d1bdd881775a5d09c59363f5460f7ce2 Mon Sep 17 00:00:00 2001
|
|
|
357786 |
From: David Gibson <dgibson@redhat.com>
|
|
|
357786 |
Date: Thu, 14 Jun 2018 01:31:36 +0200
|
|
|
357786 |
Subject: [PATCH 5/9] spapr_pci: Remove unhelpful pagesize warning
|
|
|
357786 |
|
|
|
357786 |
RH-Author: David Gibson <dgibson@redhat.com>
|
|
|
357786 |
Message-id: <20180614013136.3504-1-dgibson@redhat.com>
|
|
|
357786 |
Patchwork-id: 80681
|
|
|
357786 |
O-Subject: [RHEL-7.6 qemu-kvm-rhev PATCH] spapr_pci: Remove unhelpful pagesize warning
|
|
|
357786 |
Bugzilla: 1505664
|
|
|
357786 |
RH-Acked-by: Laurent Vivier <lvivier@redhat.com>
|
|
|
357786 |
RH-Acked-by: Thomas Huth <thuth@redhat.com>
|
|
|
357786 |
RH-Acked-by: Serhii Popovych <spopovyc@redhat.com>
|
|
|
357786 |
|
|
|
357786 |
From: David Gibson <david@gibson.dropbear.id.au>
|
|
|
357786 |
|
|
|
357786 |
By default, the IOMMU model built into the spapr virtual PCI host bridge
|
|
|
357786 |
supports 4kiB and 64kiB IOMMU page sizes. However this can be overridden
|
|
|
357786 |
which may be desirable to allow larger IOMMU page sizes when running a
|
|
|
357786 |
guest with hugepage backing and passthrough devices. For that reason a
|
|
|
357786 |
warning was printed when the device wasn't configured to allow the pagesize
|
|
|
357786 |
with which guest RAM is backed.
|
|
|
357786 |
|
|
|
357786 |
Experience has proven, however, that this message is more confusing than
|
|
|
357786 |
useful. Worse it sometimes makes little sense when the host-available page
|
|
|
357786 |
sizes don't match those available on the guest, which can happen with
|
|
|
357786 |
a POWER8 guest running on a POWER9 KVM host.
|
|
|
357786 |
|
|
|
357786 |
Long term we do want better handling to allow large IOMMU page sizes to be
|
|
|
357786 |
used, but for now this parameter and warning don't really accomplish it.
|
|
|
357786 |
So, remove the message, pending a better solution.
|
|
|
357786 |
|
|
|
357786 |
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
|
|
|
357786 |
|
|
|
357786 |
Signed-off-by: David Gibson <dgibson@redhat.com>
|
|
|
357786 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
357786 |
---
|
|
|
357786 |
hw/ppc/spapr_pci.c | 7 -------
|
|
|
357786 |
1 file changed, 7 deletions(-)
|
|
|
357786 |
|
|
|
357786 |
diff --git a/hw/ppc/spapr_pci.c b/hw/ppc/spapr_pci.c
|
|
|
357786 |
index 39a1498..f936ce6 100644
|
|
|
357786 |
--- a/hw/ppc/spapr_pci.c
|
|
|
357786 |
+++ b/hw/ppc/spapr_pci.c
|
|
|
357786 |
@@ -1717,13 +1717,6 @@ static void spapr_phb_realize(DeviceState *dev, Error **errp)
|
|
|
357786 |
}
|
|
|
357786 |
|
|
|
357786 |
/* DMA setup */
|
|
|
357786 |
- if (((sphb->page_size_mask & qemu_getrampagesize()) == 0)
|
|
|
357786 |
- && kvm_enabled()) {
|
|
|
357786 |
- warn_report("System page size 0x%lx is not enabled in page_size_mask "
|
|
|
357786 |
- "(0x%"PRIx64"). Performance may be slow",
|
|
|
357786 |
- qemu_getrampagesize(), sphb->page_size_mask);
|
|
|
357786 |
- }
|
|
|
357786 |
-
|
|
|
357786 |
for (i = 0; i < windows_supported; ++i) {
|
|
|
357786 |
tcet = spapr_tce_new_table(DEVICE(sphb), sphb->dma_liobn[i]);
|
|
|
357786 |
if (!tcet) {
|
|
|
357786 |
--
|
|
|
357786 |
1.8.3.1
|
|
|
357786 |
|