On 03/04/16 20:42, Michał Zegan wrote: > Confirmed, PCI hotplug seems to work now.
Thank you, I'll take that as Tested-by: Michał Zegan <webczat_...@poczta.onet.pl> Cheers Laszlo > W dniu 04.03.2016 o 20:13, Laszlo Ersek pisze: >> On 03/04/16 19:56, Michał Zegan wrote: >>> How are you supposed to apply those patches? git am says the first one >>> is empty. >> >> It's best to fetch the public branch I linked in the blurb. >> >> Thanks! >> Laszlo >> >>> W dniu 04.03.2016 o 15:46, Laszlo Ersek pisze: >>>> The first two patches should fix >>>> <https://github.com/tianocore/edk2/issues/32>, based on Marcel's >>>> analysis in that item. (Thanks a lot for that again!) These patches are >>>> (and should be) independent of -D USE_OLD_PCI_HOST; that is, they are >>>> active regardless of the fallback that we're temporarily retaining. >>>> >>>> Michał, can you please test if the series solves your issue? >>>> >>>> Then, based on the first two patches, the other three expose ECAM to the >>>> firmware modules proper (DXE and later). These do depend on >>>> USE_OLD_PCI_HOST==FALSE (which is the default now, and the only >>>> supported build in the mid term). >>>> >>>> I tested & regression tested the series in a bunch of scenarios; the >>>> most interesting is probably the usual physical GPU assignment. On Q35, >>>> the core PciHostBridgeDxe now permits access to the full extended config >>>> space, and it works all right with the PCIe GTX750. >>>> >>>> Gabriel, can you please check if your Q35 OSX guest(s) continue working >>>> with this? >>>> >>>> Public branch: <https://github.com/lersek/edk2/commits/pcie_issue32_v1>. >>>> >>>> Cc: Gabriel Somlo <so...@cmu.edu> >>>> Cc: Jordan Justen <jordan.l.jus...@intel.com> >>>> Cc: Marcel Apfelbaum <mar...@redhat.com> >>>> Cc: Michał Zegan <webczat_...@poczta.onet.pl> >>>> Cc: Ruiyu Ni <ruiyu...@intel.com> >>>> >>>> Thanks! >>>> Laszlo >>>> >>>> Laszlo Ersek (5): >>>> OvmfPkg: IndustryStandard/Q35MchIch9.h: add PCIEXBAR macros >>>> 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/Include/IndustryStandard/Q35MchIch9.h >>>> | 8 + >>>> OvmfPkg/Library/DxePciLibI440FxQ35/DxePciLibI440FxQ35.inf >>>> | 47 ++++++ >>>> {MdePkg/Library/BasePciLibCf8 => >>>> OvmfPkg/Library/DxePciLibI440FxQ35}/PciLib.c | 161 +++++++++++++++----- >>>> OvmfPkg/Library/PciHostBridgeLib/PciHostBridgeLib.c >>>> | 4 +- >>>> OvmfPkg/Library/PciHostBridgeLib/PciHostBridgeLib.inf >>>> | 1 + >>>> OvmfPkg/OvmfPkgIa32.dsc >>>> | 29 ++++ >>>> OvmfPkg/OvmfPkgIa32X64.dsc >>>> | 29 ++++ >>>> OvmfPkg/OvmfPkgX64.dsc >>>> | 29 ++++ >>>> OvmfPkg/PlatformPei/Platform.c >>>> | 81 ++++++++++ >>>> OvmfPkg/PlatformPei/PlatformPei.inf >>>> | 3 + >>>> 10 files changed, 356 insertions(+), 36 deletions(-) >>>> create mode 100644 >>>> OvmfPkg/Library/DxePciLibI440FxQ35/DxePciLibI440FxQ35.inf >>>> copy {MdePkg/Library/BasePciLibCf8 => >>>> OvmfPkg/Library/DxePciLibI440FxQ35}/PciLib.c (85%) >>>> >> _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
