>>> On 14.03.18 at 15:04, <[email protected]> wrote:
> --- a/xen/drivers/vpci/vpci.c
> +++ b/xen/drivers/vpci/vpci.c
> @@ -47,6 +47,10 @@ void vpci_remove_device(struct pci_dev *pdev)
>          xfree(r);
>      }
>      spin_unlock(&pdev->vpci->lock);
> +#ifdef __XEN__
> +    /* NB: fields below are not exposed to the user-space test harness. */
> +    xfree(pdev->vpci->msi);
> +#endif

Would it maybe be better to add such dummy field(s), to avoid the
#ifdef here? Anyway, with or without that
Reviewed-by: Jan Beulich <[email protected]>

Jan


_______________________________________________
Xen-devel mailing list
[email protected]
https://lists.xenproject.org/mailman/listinfo/xen-devel

Reply via email to