>>> On 16.03.18 at 14:30, <roger....@citrix.com> wrote: > --- a/xen/drivers/vpci/vpci.c > +++ b/xen/drivers/vpci/vpci.c > @@ -47,6 +47,7 @@ void vpci_remove_device(struct pci_dev *pdev) > xfree(r); > } > spin_unlock(&pdev->vpci->lock); > + xfree(pdev->vpci->msi); > xfree(pdev->vpci);
Okay, the #ifdef here is gone as agreed. But something is still fishy here for the test utility: You don't change the test code here at all, which made me go look why that is. As it looks you use a single static struct vpci instance there. That'll make the xfree() you add here correct, but the other one above then is wrong (you're freeing static data). Or am I missing something? Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
