efi: Disable interrupts around EFI calls, not in the epilog/prolog calls

Browse source

Tapasweni Pathak reported that we do a kmalloc() in efi_call_phys_prolog()
on x86-64 while having interrupts disabled, which is a big no-no, as
kmalloc() can sleep.

Solve this by removing the irq disabling from the prolog/epilog calls
around EFI calls: it's unnecessary, as in this stage we are single
threaded in the boot thread, and we don't ever execute this from
interrupt contexts.

Reported-by: Tapasweni Pathak <tapaswenipathak@gmail.com>
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Signed-off-by: Matt Fleming <matt.fleming@intel.com>

...

This commit is contained in:

Ingo Molnar 2015-03-03 07:34:33 +01:00 • committed by Matt Fleming

parent fed6cefe3b

commit 23a0d4e8fa

3 changed files with 10 additions and 11 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download patch file Download diff file Expand all files Collapse all files

7

arch/x86/platform/efi/efi.c

View file

11

arch/x86/platform/efi/efi_32.c

View file

3

arch/x86/platform/efi/efi_64.c

View file