Re: [Xen-devel] [PATCH for-4.5] reset PCI devices on force removal even when QEMU returns error

[Sander Eikelenboom <linux@xxxxxxxxxxxxxx>]

Hi Konrad,

This doesn't seem to be applied yet, nor does it seem to have a release-(N)ACK 
from you ?

--
Sander



Friday, November 28, 2014, 5:53:09 PM, you wrote:

> On do_pci_remove when QEMU returns error, we just bail out early without
> resetting the device. On domain shutdown we are racing with QEMU exiting
> and most often QEMU closes the QMP connection before executing the
> requested command.

> In these cases if force=1, it makes sense to go ahead with rest of the
> PCI device removal, that includes resetting the device and calling
> xc_deassign_device. Otherwise we risk not resetting the device properly
> on domain shutdown.

> Signed-off-by: Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>

> diff --git a/tools/libxl/libxl_pci.c b/tools/libxl/libxl_pci.c
> index 316643c..0ac0b93 100644
> --- a/tools/libxl/libxl_pci.c
> +++ b/tools/libxl/libxl_pci.c
> @@ -1243,7 +1245,7 @@ static int do_pci_remove(libxl__gc *gc, uint32_t domid,
>              rc = ERROR_INVAL;
>              goto out_fail;
>          }
> -        if (rc) {
> +        if (rc && !force) {
>              rc = ERROR_FAIL;
>              goto out_fail;
>          }



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