|
 |
357786 |
From 3483f36dbf965a4ca01c4a2d3bce1132340596d1 Mon Sep 17 00:00:00 2001
|
|
|
357786 |
From: "plai@redhat.com" <plai@redhat.com>
|
|
|
357786 |
Date: Fri, 31 Aug 2018 16:25:52 +0200
|
|
|
357786 |
Subject: [PATCH 10/29] vfio/pci: do not set the PCIDevice 'has_rom' attribute
|
|
|
357786 |
MIME-Version: 1.0
|
|
|
357786 |
Content-Type: text/plain; charset=UTF-8
|
|
|
357786 |
Content-Transfer-Encoding: 8bit
|
|
|
357786 |
|
|
|
357786 |
RH-Author: plai@redhat.com
|
|
|
357786 |
Message-id: <1535732759-22481-3-git-send-email-plai@redhat.com>
|
|
|
357786 |
Patchwork-id: 82004
|
|
|
357786 |
O-Subject: [RHEL7.6 PATCH BZ 1539280 2/9] vfio/pci: do not set the PCIDevice 'has_rom' attribute
|
|
|
357786 |
Bugzilla: 1539280
|
|
|
357786 |
RH-Acked-by: Michael S. Tsirkin <mst@redhat.com>
|
|
|
357786 |
RH-Acked-by: Pankaj Gupta <pagupta@redhat.com>
|
|
|
357786 |
RH-Acked-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
357786 |
|
|
|
357786 |
From: Cédric Le Goater <clg@kaod.org>
|
|
|
357786 |
|
|
|
357786 |
PCI devices needing a ROM allocate an optional MemoryRegion with
|
|
|
357786 |
pci_add_option_rom(). pci_del_option_rom() does the cleanup when the
|
|
|
357786 |
device is destroyed. The only action taken by this routine is to call
|
|
|
357786 |
vmstate_unregister_ram() which clears the id string of the optional
|
|
|
357786 |
ROM RAMBlock and now, also flags the RAMBlock as non-migratable. This
|
|
|
357786 |
was recently added by commit b895de502717 ("migration: discard
|
|
|
357786 |
non-migratable RAMBlocks"), .
|
|
|
357786 |
|
|
|
357786 |
VFIO devices do their own loading of the PCI option ROM in
|
|
|
357786 |
vfio_pci_size_rom(). The memory region is switched to an I/O region
|
|
|
357786 |
and the PCI attribute 'has_rom' is set but the RAMBlock of the ROM
|
|
|
357786 |
region is not allocated. When the associated PCI device is deleted,
|
|
|
357786 |
pci_del_option_rom() calls vmstate_unregister_ram() which tries to
|
|
|
357786 |
flag a NULL RAMBlock, leading to a SEGV.
|
|
|
357786 |
|
|
|
357786 |
It seems that 'has_rom' was set to have memory_region_destroy()
|
|
|
357786 |
called, but since commit 469b046ead06 ("memory: remove
|
|
|
357786 |
memory_region_destroy") this is not necessary anymore as the
|
|
|
357786 |
MemoryRegion is freed automagically.
|
|
|
357786 |
|
|
|
357786 |
Remove the PCIDevice 'has_rom' attribute setting in vfio.
|
|
|
357786 |
|
|
|
357786 |
Fixes: b895de502717 ("migration: discard non-migratable RAMBlocks")
|
|
|
357786 |
Signed-off-by: Cédric Le Goater <clg@kaod.org>
|
|
|
357786 |
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
|
|
|
357786 |
Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
|
|
|
357786 |
(cherry picked from commit 26c0ae56386edacc8b0da40264748f59afedb1bb)
|
|
|
357786 |
Signed-off-by: Paul Lai <plai@redhat.com>
|
|
|
357786 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
357786 |
---
|
|
|
357786 |
hw/vfio/pci.c | 1 -
|
|
|
357786 |
1 file changed, 1 deletion(-)
|
|
|
357786 |
|
|
|
357786 |
diff --git a/hw/vfio/pci.c b/hw/vfio/pci.c
|
|
|
357786 |
index 4683eb4..c00b91c 100644
|
|
|
357786 |
--- a/hw/vfio/pci.c
|
|
|
357786 |
+++ b/hw/vfio/pci.c
|
|
|
357786 |
@@ -990,7 +990,6 @@ static void vfio_pci_size_rom(VFIOPCIDevice *vdev)
|
|
|
357786 |
pci_register_bar(&vdev->pdev, PCI_ROM_SLOT,
|
|
|
357786 |
PCI_BASE_ADDRESS_SPACE_MEMORY, &vdev->pdev.rom);
|
|
|
357786 |
|
|
|
357786 |
- vdev->pdev.has_rom = true;
|
|
|
357786 |
vdev->rom_read_failed = false;
|
|
|
357786 |
}
|
|
|
357786 |
|
|
|
357786 |
--
|
|
|
357786 |
1.8.3.1
|
|
|
357786 |
|