Hi all This is version 4 of this series. The first patch adds PcdPciDisableBusEnumeration in MdeModulePkg, which short-circuits full enumeration if set to true. We maintain some Xen specific functions at the moment, hopefully when QEMU / KVM figures out what to do with memory map we can refactor this part. The tree can be found at: http://xenbits.xen.org/git-http/people/liuw/ovmf.git v4 starting from 021b9538. Wei. Changes since V3: * make PcdPciDisableBusEnumeration PatchableInModule as well * fix copy right information in E820.h * do not embed E820 map to EFI_XEN_INFO * fix Pcd library inclusion in Ovmf*.dsc * some coding style fixes Changes since RFC v2: * PcdPciAllowFullEnumeration -> PcdPciDisableBusEnumeration * enable Xen path in dedicated patch * OVMF info moved to dedicated header Wei Liu (7): MdeModulePkg: introduce PcdPciDisableBusEnumeration OvmfPkg: introduce E820.h OvmfPkg: define EFI_XEN_OVMF_INFO OvmfPkg: detect Xen earlier OvmfPkg: introduce PublishPeiMemory OvmfPkg: introduce XenMemMapInitialization OvmfPkg: enable Xen specific path MdeModulePkg/Bus/Pci/PciBusDxe/PciBus.c | 5 +- MdeModulePkg/Bus/Pci/PciBusDxe/PciBusDxe.inf | 1 + MdeModulePkg/MdeModulePkg.dec | 3 + OvmfPkg/Include/IndustryStandard/E820.h | 46 +++++++++++++ OvmfPkg/OvmfPkgIa32.dsc | 6 +- OvmfPkg/OvmfPkgIa32X64.dsc | 6 +- OvmfPkg/OvmfPkgX64.dsc | 6 +- OvmfPkg/PlatformPei/MemDetect.c | 36 +++++++++- OvmfPkg/PlatformPei/Platform.c | 91 +++++++++++++++++++++++++- OvmfPkg/PlatformPei/Platform.h | 18 +++++ OvmfPkg/PlatformPei/PlatformPei.inf | 1 + OvmfPkg/PlatformPei/Xen.c | 38 +++++++---- OvmfPkg/PlatformPei/Xen.h | 45 +++++++++++++ 13 files changed, 283 insertions(+), 19 deletions(-) create mode 100644 OvmfPkg/Include/IndustryStandard/E820.h create mode 100644 OvmfPkg/PlatformPei/Xen.h -- 1.7.10.4
Series Reviewed-by: Jordan Justen <jordan.l.justen@intel.com> Some minor issues mentioned in a reply to patch 1. -Jordan On Fri, Nov 29, 2013 at 6:13 AM, Wei Liu <wei.liu2@citrix.com> wrote:> Hi all > > This is version 4 of this series. > > The first patch adds PcdPciDisableBusEnumeration in MdeModulePkg, which > short-circuits full enumeration if set to true. > > We maintain some Xen specific functions at the moment, hopefully when QEMU / > KVM figures out what to do with memory map we can refactor this part. > > The tree can be found at: > http://xenbits.xen.org/git-http/people/liuw/ovmf.git v4 > starting from 021b9538. > > Wei. > > Changes since V3: > * make PcdPciDisableBusEnumeration PatchableInModule as well > * fix copy right information in E820.h > * do not embed E820 map to EFI_XEN_INFO > * fix Pcd library inclusion in Ovmf*.dsc > * some coding style fixes > > Changes since RFC v2: > * PcdPciAllowFullEnumeration -> PcdPciDisableBusEnumeration > * enable Xen path in dedicated patch > * OVMF info moved to dedicated header > > Wei Liu (7): > MdeModulePkg: introduce PcdPciDisableBusEnumeration > OvmfPkg: introduce E820.h > OvmfPkg: define EFI_XEN_OVMF_INFO > OvmfPkg: detect Xen earlier > OvmfPkg: introduce PublishPeiMemory > OvmfPkg: introduce XenMemMapInitialization > OvmfPkg: enable Xen specific path > > MdeModulePkg/Bus/Pci/PciBusDxe/PciBus.c | 5 +- > MdeModulePkg/Bus/Pci/PciBusDxe/PciBusDxe.inf | 1 + > MdeModulePkg/MdeModulePkg.dec | 3 + > OvmfPkg/Include/IndustryStandard/E820.h | 46 +++++++++++++ > OvmfPkg/OvmfPkgIa32.dsc | 6 +- > OvmfPkg/OvmfPkgIa32X64.dsc | 6 +- > OvmfPkg/OvmfPkgX64.dsc | 6 +- > OvmfPkg/PlatformPei/MemDetect.c | 36 +++++++++- > OvmfPkg/PlatformPei/Platform.c | 91 +++++++++++++++++++++++++- > OvmfPkg/PlatformPei/Platform.h | 18 +++++ > OvmfPkg/PlatformPei/PlatformPei.inf | 1 + > OvmfPkg/PlatformPei/Xen.c | 38 +++++++---- > OvmfPkg/PlatformPei/Xen.h | 45 +++++++++++++ > 13 files changed, 283 insertions(+), 19 deletions(-) > create mode 100644 OvmfPkg/Include/IndustryStandard/E820.h > create mode 100644 OvmfPkg/PlatformPei/Xen.h > > -- > 1.7.10.4 > > > _______________________________________________ > Xen-devel mailing list > Xen-devel@lists.xen.org > http://lists.xen.org/xen-devel
On Sat, Nov 30, 2013 at 03:58:18PM -0800, Jordan Justen wrote:> Series Reviewed-by: Jordan Justen <jordan.l.justen@intel.com> >Thanks! Wei.> Some minor issues mentioned in a reply to patch 1. > > -Jordan >
On Sat, Nov 30, 2013 at 03:58:18PM -0800, Jordan Justen wrote:> Series Reviewed-by: Jordan Justen <jordan.l.justen@intel.com> > > Some minor issues mentioned in a reply to patch 1. >Hi Jordan Is there any more concerns regarding this series? All patches have reviewed-by tag now. I was quite clear about Ray''s question. Anything else I need to do to get this series merged? Wei.