Re: [Xen-devel] [PATCH v3] if xen_platform_pci=0 is set don't blow up.

[Bjorn Helgaas <bhelgaas@xxxxxxxxxx>]

On Mon, Dec 16, 2013 at 8:04 AM, Konrad Rzeszutek Wilk
<konrad.wilk@xxxxxxxxxx> wrote:
> The first patch:
>  [PATCH v3 1/2] xen/pvhvm: If xen_platform_pci=0 is set don't blow up
>
> I would like to commit to stable as it is fixing an eggregious bug -
> where we blow up if the guest config has: "xen_platform_pci=0" setup.
> This bug has been in existence for years and it is time to stamp it out.
>
> The second patch is a cleanup - not a stable candidate.
>
> It touches all of the Xen frontend drivers and adds the logic of:
> "if user disabled us, don't init" - with variations. As you can
> specify exactly which ones you want to init and which ones
> not (Linux runtime parameter 'xen_emul_unplug'). But for the majority
> of drivers - it is just an on/off switch.
>
> Since it touches a lot of maintainers I figured I would send it
> to Linus on Wednesday or Thursday.
>
> Thank you!
>
>  arch/x86/xen/platform-pci-unplug.c         | 79 
> ++++++++++++++++++++++++++++--
>  drivers/block/xen-blkfront.c               |  4 +-
>  drivers/char/tpm/xen-tpmfront.c            |  4 ++
>  drivers/input/misc/xen-kbdfront.c          |  4 ++
>  drivers/net/xen-netfront.c                 |  2 +-
>  drivers/pci/xen-pcifront.c                 |  4 ++
>  drivers/video/xen-fbfront.c                |  4 ++
>  drivers/xen/xenbus/xenbus_probe_frontend.c |  2 +-
>  include/xen/platform_pci.h                 | 25 +++++++++-
>  9 files changed, 119 insertions(+), 9 deletions(-)

If you want it:

Acked-by: Bjorn Helgaas <bhelgaas@xxxxxxxxxx>

for the PCI parts.

>
> Konrad Rzeszutek Wilk (2):
>       xen/pvhvm: If xen_platform_pci=0 is set don't blow up (v3).
>       xen/pvhvm: Remove the xen_platform_pci int.
>

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel