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

["Yao, Jiewen" <jiewen.yao@intel.com>]

Sounds good, thanks!

> -----Original Message-----
> From: Boeuf, Sebastien <sebastien.boeuf@intel.com>
> Sent: Thursday, June 2, 2022 5:29 PM
> To: Yao, Jiewen <jiewen.yao@intel.com>; devel@edk2.groups.io
> Cc: kraxel@redhat.com; Justen, Jordan L <jordan.l.justen@intel.com>
> Subject: Re: [PATCH 0/4] OvmfPkg: CloudHv: Reduce PIO and MMIO accesses
> 
> 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
> > >
> >