diff options
author | Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> | 2011-02-16 16:26:44 -0500 |
---|---|---|
committer | Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> | 2011-04-14 11:17:56 -0400 |
commit | 1eff1ad0285038e309a81da4a004f071608309fb (patch) | |
tree | 64c4708b40a95d5ce596859883610ba0b3efa76c /drivers/xen | |
parent | e6197acc726ab3baa60375a5891d58c2ee87e0f3 (diff) | |
download | op-kernel-dev-1eff1ad0285038e309a81da4a004f071608309fb.zip op-kernel-dev-1eff1ad0285038e309a81da4a004f071608309fb.tar.gz |
xen/irq: The Xen hypervisor cleans up the PIRQs if the other domain forgot.
And if the other domain forgot to clean up its PIRQs we don't need
to fail the operation. Just take a note of it and continue on.
Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Diffstat (limited to 'drivers/xen')
-rw-r--r-- | drivers/xen/events.c | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/drivers/xen/events.c b/drivers/xen/events.c index e4e8e9a..e51f3c5 100644 --- a/drivers/xen/events.c +++ b/drivers/xen/events.c @@ -728,7 +728,14 @@ int xen_destroy_irq(int irq) unmap_irq.pirq = info->u.pirq.pirq; unmap_irq.domid = info->u.pirq.domid; rc = HYPERVISOR_physdev_op(PHYSDEVOP_unmap_pirq, &unmap_irq); - if (rc) { + /* If another domain quits without making the pci_disable_msix + * call, the Xen hypervisor takes care of freeing the PIRQs + * (free_domain_pirqs). + */ + if ((rc == -ESRCH && info->u.pirq.domid != DOMID_SELF)) + printk(KERN_INFO "domain %d does not have %d anymore\n", + info->u.pirq.domid, info->u.pirq.pirq); + else if (rc) { printk(KERN_WARNING "unmap irq failed %d\n", rc); goto out; } |