From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: citrix.com, ip: 162.221.156.55, mailfrom: prvs=01609d005=anthony.perard@citrix.com) Received: from SMTP03.CITRIX.COM (SMTP03.CITRIX.COM [162.221.156.55]) by groups.io with SMTP; Tue, 30 Apr 2019 09:36:26 -0700 X-IronPort-AV: E=Sophos;i="5.60,414,1549929600"; d="scan'208";a="84777940" Date: Tue, 30 Apr 2019 17:35:51 +0100 From: "Anthony PERARD" To: Laszlo Ersek CC: Igor Druzhinin , , , , , Subject: Re: [PATCH v3 3/3] OvmfPkg/XenSupport: turn off address decoding before BAR sizing Message-ID: <20190430163551.GA1264@perard.uk.xensource.com> References: <1556223838-5176-1-git-send-email-igor.druzhinin@citrix.com> <1556223838-5176-4-git-send-email-igor.druzhinin@citrix.com> MIME-Version: 1.0 In-Reply-To: User-Agent: Mutt/1.11.4 (2019-03-13) Return-Path: anthony.perard@citrix.com Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline On Fri, Apr 26, 2019 at 08:15:07PM +0200, Laszlo Ersek wrote: > On 04/25/19 22:23, Igor Druzhinin wrote: > > On Xen, hvmloader firmware leaves address decoding enabled for > > enumerated PCI device before jumping into OVMF. OVMF seems to > > expect it to be disabled and tries to size PCI BARs in several places > > without disabling it which causes BAR64, for example, being > > incorrectly placed by QEMU. > > > > Fix it by disabling PCI address decoding explicitly before the > > first attempt to size BARs on Xen. > > > > Signed-off-by: Igor Druzhinin > > --- > > Changes in v3: > > - dropped unused arguments and rename PcatPciRootBridgeDecoding function > > - make mask application more clear as suggested > > --- > > OvmfPkg/Library/PciHostBridgeLib/XenSupport.c | 21 +++++++++++++++++++++ > > 1 file changed, 21 insertions(+) > > > > diff --git a/OvmfPkg/Library/PciHostBridgeLib/XenSupport.c b/OvmfPkg/Library/PciHostBridgeLib/XenSupport.c > > index 76fca53..b41bd6b 100644 > > --- a/OvmfPkg/Library/PciHostBridgeLib/XenSupport.c > > +++ b/OvmfPkg/Library/PciHostBridgeLib/XenSupport.c > > @@ -49,6 +49,22 @@ PcatPciRootBridgeBarExisted ( > > EnableInterrupts (); > > } > > > > +#define PCI_COMMAND_DECODE ((UINT16)(EFI_PCI_COMMAND_IO_SPACE | \ > > + EFI_PCI_COMMAND_MEMORY_SPACE)) > > +STATIC > > +VOID > > +PcatPciRootBridgeDecodingDisable ( > > + IN UINTN Address > > + ) > > +{ > > + UINT16 Value; > > + > > + Value = PciRead16 (Address); > > + if (Value & PCI_COMMAND_DECODE) { > > + PciWrite16 (Address, Value & ~(UINT32)PCI_COMMAND_DECODE); > > + } > > +} > > + > > STATIC > > VOID > > PcatPciRootBridgeParseBars ( > > @@ -75,6 +91,11 @@ PcatPciRootBridgeParseBars ( > > UINT64 Limit; > > PCI_ROOT_BRIDGE_APERTURE *MemAperture; > > > > + // Disable address decoding for every device before OVMF starts sizing it > > + PcatPciRootBridgeDecodingDisable ( > > + PCI_LIB_ADDRESS (Bus, Device, Function, PCI_COMMAND_OFFSET) > > + ); > > + > > for (Offset = BarOffsetBase; Offset < BarOffsetEnd; Offset += sizeof (UINT32)) { > > PcatPciRootBridgeBarExisted ( > > PCI_LIB_ADDRESS (Bus, Device, Function, Offset), > > > > Acked-by: Laszlo Ersek > > Someone from the Xen community please ACK this too, and then we can push > the series. Acked-by: Anthony PERARD Thanks, -- Anthony PERARD