diff options
author | Wei Gang <gang.wei@intel.com> | 2011-01-26 08:54:12 +0000 |
---|---|---|
committer | Wei Gang <gang.wei@intel.com> | 2011-01-26 08:54:12 +0000 |
commit | 56bcfc196f19d9667c5ae8f6dc40d7e32a59a00e (patch) | |
tree | e11a62ef3f386919a4ceb719fd2d74e071029184 | |
parent | 16b2a6cf7fb0585fcfd51b0f4febac3be91a91a1 (diff) | |
download | xen-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.c | 3 |
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)); |