aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorWei Gang <gang.wei@intel.com>2011-01-26 09:05:53 +0000
committerWei Gang <gang.wei@intel.com>2011-01-26 09:05:53 +0000
commit70de4016b4fba4de5b02827a80c59a8f1671c0bf (patch)
tree37cc87c0cb670be10d898d97866ba4d893ad825c
parent5254df9090db75d2f90ba0167f94e56219f34815 (diff)
downloadxen-70de4016b4fba4de5b02827a80c59a8f1671c0bf.tar.gz
xen-70de4016b4fba4de5b02827a80c59a8f1671c0bf.tar.bz2
xen-70de4016b4fba4de5b02827a80c59a8f1671c0bf.zip
x86: Fix pirq teardown on domain destruction.
The privilege check in unmap_domain_pirq() fails since the teardown completes in RCU (idle domain) context. We can remove the check since it is covered in physdev_op() already, which is the only potentially unprivileged caller. Signed-off-by: Wei Gang <gang.wei@intel.com> xen-unstable changeset: 22816:e7f44fb4ecc3 xen-unstable date: Wed Jan 26 08:54:12 2011 +0000
-rw-r--r--xen/arch/x86/irq.c3
1 files changed, 0 insertions, 3 deletions
diff --git a/xen/arch/x86/irq.c b/xen/arch/x86/irq.c
index 5458cedaa3..e0c7c95957 100644
--- a/xen/arch/x86/irq.c
+++ b/xen/arch/x86/irq.c
@@ -1549,9 +1549,6 @@ int unmap_domain_pirq(struct domain *d, int pirq)
if ( (pirq < 0) || (pirq >= d->nr_pirqs) )
return -EINVAL;
- if ( !IS_PRIV_FOR(current->domain, d) )
- return -EINVAL;
-
ASSERT(spin_is_locked(&pcidevs_lock));
ASSERT(spin_is_locked(&d->event_lock));