[edk2-devel] [PATCH 0/4] OvmfPkg: CloudHv: Reduce PIO and MMIO accesses

Boeuf, Sebastien posted 4 patches 1 year, 11 months ago
Failed in applying to current master (apply log)
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
[edk2-devel] [PATCH 0/4] OvmfPkg: CloudHv: Reduce PIO and MMIO accesses
Posted by Boeuf, Sebastien 1 year, 11 months ago
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.



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#89655): https://edk2.groups.io/g/devel/message/89655
Mute This Topic: https://groups.io/mt/91011837/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH 0/4] OvmfPkg: CloudHv: Reduce PIO and MMIO accesses
Posted by Yao, Jiewen 1 year, 10 months ago
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?

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



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#90147): https://edk2.groups.io/g/devel/message/90147
Mute This Topic: https://groups.io/mt/91011837/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH 0/4] OvmfPkg: CloudHv: Reduce PIO and MMIO accesses
Posted by Boeuf, Sebastien 1 year, 10 months ago
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.

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.


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#90149): https://edk2.groups.io/g/devel/message/90149
Mute This Topic: https://groups.io/mt/91011837/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [edk2-devel] [PATCH 0/4] OvmfPkg: CloudHv: Reduce PIO and MMIO accesses
Posted by Boeuf, Sebastien 1 year, 10 months ago
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.


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#90152): https://edk2.groups.io/g/devel/message/90152
Mute This Topic: https://groups.io/mt/91011837/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [edk2-devel] [PATCH 0/4] OvmfPkg: CloudHv: Reduce PIO and MMIO accesses
Posted by Yao, Jiewen 1 year, 10 months ago
Merged https://github.com/tianocore/edk2/pull/2942

> -----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
> > >
> >



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#90176): https://edk2.groups.io/g/devel/message/90176
Mute This Topic: https://groups.io/mt/91011837/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [edk2-devel] [PATCH 0/4] OvmfPkg: CloudHv: Reduce PIO and MMIO accesses
Posted by Boeuf, Sebastien 1 year, 10 months ago
Thank you!
________________________________
From: Yao, Jiewen <jiewen.yao@intel.com>
Sent: Friday, June 3, 2022 12:52 PM
To: Boeuf, Sebastien <sebastien.boeuf@intel.com>; devel@edk2.groups.io <devel@edk2.groups.io>
Cc: kraxel@redhat.com <kraxel@redhat.com>; Justen, Jordan L <jordan.l.justen@intel.com>
Subject: RE: [PATCH 0/4] OvmfPkg: CloudHv: Reduce PIO and MMIO accesses

Merged https://github.com/tianocore/edk2/pull/2942

> -----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
> > >
> >

---------------------------------------------------------------------
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.


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#90177): https://edk2.groups.io/g/devel/message/90177
Mute This Topic: https://groups.io/mt/91011837/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [edk2-devel] [PATCH 0/4] OvmfPkg: CloudHv: Reduce PIO and MMIO accesses
Posted by Yao, Jiewen 1 year, 10 months ago
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
> > >
> >



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#90154): https://edk2.groups.io/g/devel/message/90154
Mute This Topic: https://groups.io/mt/91011837/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [edk2-devel] [PATCH 0/4] OvmfPkg: CloudHv: Reduce PIO and MMIO accesses
Posted by Boeuf, Sebastien 1 year, 11 months ago
Hi,

Sorry to bother you folks, but I was wondering if you had this patchset on your radar :)

Thanks for your time,
Sebastien
________________________________
From: Boeuf, Sebastien <sebastien.boeuf@intel.com>
Sent: Tuesday, May 10, 2022 2:50 PM
To: devel@edk2.groups.io <devel@edk2.groups.io>
Cc: Yao, Jiewen <jiewen.yao@intel.com>; Justen, Jordan L <jordan.l.justen@intel.com>; kraxel@redhat.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.


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#89965): https://edk2.groups.io/g/devel/message/89965
Mute This Topic: https://groups.io/mt/91011837/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-