aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorWei Gang <gang.wei@intel.com>2011-01-26 08:54:12 +0000
committerWei Gang <gang.wei@intel.com>2011-01-26 08:54:12 +0000
commit56bcfc196f19d9667c5ae8f6dc40d7e32a59a00e (patch)
treee11a62ef3f386919a4ceb719fd2d74e071029184
parent16b2a6cf7fb0585fcfd51b0f4febac3be91a91a1 (diff)
downloadxen-56bcfc196f19d9667c5ae8f6dc40d7e32a59a00e.tar.gz
xen-56bcfc196f19d9667c5ae8f6dc40d7e32a59a00e.tar.bz2
xen-56bcfc196f19d9667c5ae8f6dc40d7e32a59a00e.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>
-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 11bd233794..3275b2d07c 100644
--- a/xen/arch/x86/irq.c
+++ b/xen/arch/x86/irq.c
@@ -1567,9 +1567,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));