diff options
author | Jeremy Fitzhardinge <jeremy@goop.org> | 2009-02-06 16:38:56 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2009-02-16 02:56:41 -0500 |
commit | c99608637eac8834d830496c462c054137772122 (patch) | |
tree | 2ed6f93a8e733a61a78d61f5181c79246f5458f9 /arch/x86/xen | |
parent | 3d39e9d07b576ee72f2c94cfad9e618fe21763b2 (diff) |
x86, xen: do multicall callbacks with interrupts disabled
We can't call the callbacks after enabling interrupts, as we may get a
nested multicall call, which would cause a great deal of havok.
Signed-off-by: Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'arch/x86/xen')
-rw-r--r-- | arch/x86/xen/multicalls.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/arch/x86/xen/multicalls.c b/arch/x86/xen/multicalls.c index 6cffd5532b91..8bff7e7c290b 100644 --- a/arch/x86/xen/multicalls.c +++ b/arch/x86/xen/multicalls.c | |||
@@ -170,8 +170,6 @@ void xen_mc_flush(void) | |||
170 | } else | 170 | } else |
171 | BUG_ON(b->argidx != 0); | 171 | BUG_ON(b->argidx != 0); |
172 | 172 | ||
173 | local_irq_restore(flags); | ||
174 | |||
175 | for (i = 0; i < b->cbidx; i++) { | 173 | for (i = 0; i < b->cbidx; i++) { |
176 | struct callback *cb = &b->callbacks[i]; | 174 | struct callback *cb = &b->callbacks[i]; |
177 | 175 | ||
@@ -179,6 +177,8 @@ void xen_mc_flush(void) | |||
179 | } | 177 | } |
180 | b->cbidx = 0; | 178 | b->cbidx = 0; |
181 | 179 | ||
180 | local_irq_restore(flags); | ||
181 | |||
182 | WARN_ON(ret); | 182 | WARN_ON(ret); |
183 | } | 183 | } |
184 | 184 | ||