On 03/09/16 12:07, Michał Zegan wrote: > Tried to test with the following qemu command line: > qemu-system-x86_64 -cdrom Pobrane/arch.iso -machine q35,accel=kvm -m 512 > -nographic -device ioh3420,port=1,chassis=1,id=pcieport1 -pflash OVMF.fd > > Tried to add a rtl8139 using device_add and then remove with device_del. > Results: hotplugging and hot unplugging works as expected with a new patch. > Tested-By: Michał Zegan <[email protected]>
Fantastic, thank you very much for the quick feedback! Laszlo > > W dniu 08.03.2016 o 15:19, Laszlo Ersek pisze: >> Version 2 of <http://thread.gmane.org/gmane.comp.bios.edk2.devel/8707>. >> >> News for v2: following Gerd's advice, I inserted a new patch (as #2), >> and reworked parts of patch #3. The changes are marked in detail on >> those patches. >> >> Review & testing feedback: Jordan should please review patches #2 and >> #3, and Gabriel and Michał should please retest the series in their >> usual environments. I also ran my own routine, successfully. >> >> Public branch: <https://github.com/lersek/edk2/commits/pcie_issue32_v2>. >> >> Cc: Gabriel Somlo <[email protected]> >> Cc: Gerd Hoffmann <[email protected]> >> Cc: Jordan Justen <[email protected]> >> Cc: Marcel Apfelbaum <[email protected]> >> Cc: Michał Zegan <[email protected]> >> >> Thanks >> Laszlo >> >> Laszlo Ersek (6): >> OvmfPkg: IndustryStandard/Q35MchIch9.h: add PCIEXBAR macros >> OvmfPkg: PlatformPei: lower the 32-bit PCI MMIO base to 2GB on Q35 >> OvmfPkg: PlatformPei: enable PCIEXBAR (aka MMCONFIG / ECAM) on Q35 >> OvmfPkg: add DxePciLibI440FxQ35 >> OvmfPkg: match PCI config access to machine type (if not >> USE_OLD_PCI_HOST) >> OvmfPkg: PciHostBridgeLib: permit access to the full extended config >> space >> >> OvmfPkg/OvmfPkgIa32.dsc >> | 29 ++++ >> OvmfPkg/OvmfPkgIa32X64.dsc >> | 29 ++++ >> OvmfPkg/OvmfPkgX64.dsc >> | 29 ++++ >> OvmfPkg/Library/DxePciLibI440FxQ35/DxePciLibI440FxQ35.inf >> | 47 ++++++ >> OvmfPkg/Library/PciHostBridgeLib/PciHostBridgeLib.inf >> | 1 + >> OvmfPkg/PlatformPei/PlatformPei.inf >> | 3 + >> OvmfPkg/Include/IndustryStandard/Q35MchIch9.h >> | 8 + >> {MdePkg/Library/BasePciLibCf8 => >> OvmfPkg/Library/DxePciLibI440FxQ35}/PciLib.c | 161 +++++++++++++++----- >> OvmfPkg/Library/PciHostBridgeLib/PciHostBridgeLib.c >> | 4 +- >> OvmfPkg/PlatformPei/Platform.c >> | 82 +++++++++- >> 10 files changed, 352 insertions(+), 41 deletions(-) >> create mode 100644 OvmfPkg/Library/DxePciLibI440FxQ35/DxePciLibI440FxQ35.inf >> copy {MdePkg/Library/BasePciLibCf8 => >> OvmfPkg/Library/DxePciLibI440FxQ35}/PciLib.c (85%) >> _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
