Re: [PATCH 0/4] OvmfPkg: CloudHv: Reduce PIO and MMIO accesses

["Boeuf, Sebastien" <sebastien.boeuf@intel.com>]

On Thu, 2022-06-02 at 11:04 +0200, Sebastien Boeuf wrote:
> On Thu, 2022-06-02 at 08:57 +0000, Yao, Jiewen wrote:
> > Acked-by: Jiewen Yao <Jiewen.yao@intel.com>
> > 
> > I notice the PR failed -
> > https://github.com/tianocore/edk2/pull/2875
> > 
> > I just want to double confirm, do you have a success PR for this
> > patch set?
> 
> Let me repush the PR once rebased on latest master. I think the
> failures were only related to the Windows CI issues.

Ok the CI is passing fine after I rebased on latest master. See
https://github.com/tianocore/edk2/pull/2875

Thanks,
Sebastien

> 
> Thanks,
> Sebastien
> 
> > 
> > Thank you
> > Yao Jiewen
> > 
> > > -----Original Message-----
> > > From: Boeuf, Sebastien <sebastien.boeuf@intel.com>
> > > Sent: Tuesday, May 10, 2022 8:51 PM
> > > To: devel@edk2.groups.io
> > > Cc: Yao, Jiewen <jiewen.yao@intel.com>; Justen, Jordan L
> > > <jordan.l.justen@intel.com>; kraxel@redhat.com; Boeuf, Sebastien
> > > <sebastien.boeuf@intel.com>
> > > Subject: [PATCH 0/4] OvmfPkg: CloudHv: Reduce PIO and MMIO
> > > accesses
> > > 
> > > From: Sebastien Boeuf <sebastien.boeuf@intel.com>
> > > 
> > > The goal of this series is to reduce the amount of inappropriate
> > > PIO
> > > and MMIO accesses generated by the firmware when running on Cloud
> > > Hypervisor.
> > > 
> > > For MMIO accesses, it is about providing the right base address
> > > where
> > > the firmware will be loaded by the VMM in order to avoid
> > > unexpected
> > > accesses to some address ranges.
> > > 
> > > For PIO accesses, it is about preventing some read/write to be
> > > performed
> > > in the first place, as we know the underlying device is not
> > > emulated by
> > > Cloud Hypervisor.
> > > 
> > > Signed-off-by: Sebastien Boeuf <sebastien.boeuf@intel.com>
> > > 
> > > Sebastien Boeuf (4):
> > >   OvmfPkg: CloudHv: Fix FW_BASE_ADDRESS
> > >   OvmfPkg: Check for QemuFwCfg availability before accessing it
> > >   OvmfPkg: CloudHv: Rely on QemuFwCfgLibNull implementation
> > >   OvmfPkg: Don't access A20 gate register on Cloud Hypervisor
> > > 
> > >  OvmfPkg/CloudHv/CloudHvDefines.fdf.inc        | 65
> > > +++++++++++++++++++
> > >  OvmfPkg/CloudHv/CloudHvX64.dsc                |  4 +-
> > >  OvmfPkg/CloudHv/CloudHvX64.fdf                |  2 +-
> > >  OvmfPkg/Library/PlatformInitLib/Platform.c    | 13 ++--
> > >  .../QemuBootOrderLib/QemuBootOrderLib.c       |  8 ++-
> > >  5 files changed, 82 insertions(+), 10 deletions(-)
> > >  create mode 100644 OvmfPkg/CloudHv/CloudHvDefines.fdf.inc
> > > 
> > > --
> > > 2.32.0
> > 
> 

---------------------------------------------------------------------
Intel Corporation SAS (French simplified joint stock company)
Registered headquarters: "Les Montalets"- 2, rue de Paris, 
92196 Meudon Cedex, France
Registration Number:  302 456 199 R.C.S. NANTERRE
Capital: 5 208 026.16 Euros

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.