[PATCH 0/2] Remove Xen PVH dependency on PCI

Jason Andryuk posted 2 patches 3 years, 5 months ago
Failed in applying to current master (apply log)
arch/x86/xen/Kconfig | 38 ++++++++++++++++++++++----------------
drivers/xen/Makefile |  2 +-
2 files changed, 23 insertions(+), 17 deletions(-)
[PATCH 0/2] Remove Xen PVH dependency on PCI
Posted by Jason Andryuk 3 years, 5 months ago
A Xen PVH domain doesn't have a PCI bus or devices, so it doesn't need
PCI support built in.  Currently, XEN_PVH depends on XEN_PVHVM which
depends on PCI.

The first patch introduces XEN_PVHVM_GUEST as a toplevel item and
changes XEN_PVHVM to a hidden variable.  This allows XEN_PVH to depend
on XEN_PVHVM without PCI while XEN_PVHVM_GUEST depends on PCI.

The second patch moves XEN_512GB to clean up the option nesting.

Jason Andryuk (2):
  xen: Remove Xen PVH/PVHVM dependency on PCI
  xen: Kconfig: nest Xen guest options

 arch/x86/xen/Kconfig | 38 ++++++++++++++++++++++----------------
 drivers/xen/Makefile |  2 +-
 2 files changed, 23 insertions(+), 17 deletions(-)

-- 
2.26.2


Re: [PATCH 0/2] Remove Xen PVH dependency on PCI
Posted by Jürgen Groß 3 years, 3 months ago
On 14.10.20 19:53, Jason Andryuk wrote:
> A Xen PVH domain doesn't have a PCI bus or devices, so it doesn't need
> PCI support built in.  Currently, XEN_PVH depends on XEN_PVHVM which
> depends on PCI.
> 
> The first patch introduces XEN_PVHVM_GUEST as a toplevel item and
> changes XEN_PVHVM to a hidden variable.  This allows XEN_PVH to depend
> on XEN_PVHVM without PCI while XEN_PVHVM_GUEST depends on PCI.
> 
> The second patch moves XEN_512GB to clean up the option nesting.
> 
> Jason Andryuk (2):
>    xen: Remove Xen PVH/PVHVM dependency on PCI
>    xen: Kconfig: nest Xen guest options
> 
>   arch/x86/xen/Kconfig | 38 ++++++++++++++++++++++----------------
>   drivers/xen/Makefile |  2 +-
>   2 files changed, 23 insertions(+), 17 deletions(-)
> 

Series applied to: xen/tip.git for-linus-5.11


Juergen
Re: [SUSPECTED SPAM][PATCH 0/2] Remove Xen PVH dependency on PCI
Posted by Andrew Cooper 3 years, 5 months ago
On 14/10/2020 18:53, Jason Andryuk wrote:
> A Xen PVH domain doesn't have a PCI bus or devices,

[*] Yet.

> so it doesn't need PCI support built in.

Untangling the dependences is a good thing, but eventually we plan to
put an optional PCI bus back in, e.g. for SRIOV usecases.

~Andrew

Re: [SUSPECTED SPAM][PATCH 0/2] Remove Xen PVH dependency on PCI
Posted by Jason Andryuk 3 years, 5 months ago
On Wed, Oct 14, 2020 at 2:04 PM Andrew Cooper <andrew.cooper3@citrix.com> wrote:
>
> On 14/10/2020 18:53, Jason Andryuk wrote:
> > A Xen PVH domain doesn't have a PCI bus or devices,
>
> [*] Yet.

:)

> > so it doesn't need PCI support built in.
>
> Untangling the dependences is a good thing, but eventually we plan to
> put an optional PCI bus back in, e.g. for SRIOV usecases.

Yes, and to be clear this change doesn't preclude including the PCI
code.  I was just looking to remove code from my VMs that aren't using
PCI devices.

Regards,
Jason