diff options
author | Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com> | 2009-03-04 20:36:57 -0500 |
---|---|---|
committer | Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com> | 2009-03-30 12:25:34 -0400 |
commit | 8de07bbdede03598801cf33ab23dcbcd28a918d2 (patch) | |
tree | 9caf7228d87561b411e0057c0a4b9107fc416ee2 | |
parent | 4185f35404dc96f8525298c7c548aee419f3b3f4 (diff) |
xen/mmu: weaken flush_tlb_other test
Impact: fixes crashing bug
There's no particular problem with getting an empty cpu mask,
so just shortcut-return if we get one.
Avoids crash reported by Christophe Saout <christophe@saout.de>
Signed-off-by: Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com>
-rw-r--r-- | arch/x86/xen/mmu.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/arch/x86/xen/mmu.c b/arch/x86/xen/mmu.c index df87c803cecc..e425a32e0a90 100644 --- a/arch/x86/xen/mmu.c +++ b/arch/x86/xen/mmu.c | |||
@@ -1293,8 +1293,8 @@ static void xen_flush_tlb_others(const struct cpumask *cpus, | |||
1293 | } *args; | 1293 | } *args; |
1294 | struct multicall_space mcs; | 1294 | struct multicall_space mcs; |
1295 | 1295 | ||
1296 | BUG_ON(cpumask_empty(cpus)); | 1296 | if (cpumask_empty(cpus)) |
1297 | BUG_ON(!mm); | 1297 | return; /* nothing to do */ |
1298 | 1298 | ||
1299 | mcs = xen_mc_entry(sizeof(*args)); | 1299 | mcs = xen_mc_entry(sizeof(*args)); |
1300 | args = mcs.args; | 1300 | args = mcs.args; |