Op 24-10-12 17:21, Olof Johansson schreef:
> Hi,
>
> On Wed, Oct 24, 2012 at 1:40 AM, Maarten Lankhorst
> <maarten.lankho...@canonical.com> wrote:
>> Op 24-10-12 08:24, Olof Johansson schreef:
>>> When 32-bit EFI is used with 64-bit kernel (or vice versa), turn off
>>> efi_enabled once setup is done. Beyond setup, it is normally used to
>>> determine if runtime services are available and we will have none.
>>>
>>> This will resolve issues stemming from efivars modprobe panicking on a
>>> 32/64-bit setup, as well as some reboot issues on similar setups.
>>>
>>> Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=45991
>>>
>>> Reported-by: Marko Kohtala <marko.koht...@gmail.com>
>>> Reported-by: Maxim Kammerer <m...@dee.su>
>>> Signed-off-by: Olof Johansson <o...@lixom.net>
>>> Cc: sta...@kernel.org # 3.4 - 3.6
>>> Cc: Matthew Garrett <m...@redhat.com>
>>> Cc: Maarten Lankhorst <maarten.lankho...@canonical.com>
>>> ---
>>>
>>> v2: rebase due to context diffs, and simplified efi_is_native() logic.
>>>
>>> arch/x86/kernel/setup.c | 11 +++++++++++
>>> arch/x86/platform/efi/efi.c | 16 +++++++++-------
>>> 2 files changed, 20 insertions(+), 7 deletions(-)
>>>
>>> diff --git a/arch/x86/kernel/setup.c b/arch/x86/kernel/setup.c
>>> index 468e98d..ea2c587 100644
>>> --- a/arch/x86/kernel/setup.c
>>> +++ b/arch/x86/kernel/setup.c
>>> @@ -1048,6 +1048,17 @@ void __init setup_arch(char **cmdline_p)
>>> arch_init_ideal_nops();
>>>
>>> register_refined_jiffies(CLOCK_TICK_RATE);
>>> +
>>> +#ifdef CONFIG_EFI
>>> + /* Once setup is done above, disable efi_enabled on mismatched
>>> + * firmware/kernel archtectures since there is no support for
>>> + * runtime services.
>>> + */
>>> + if (IS_ENABLED(CONFIG_X86_64) != efi_64bit) {
>>> + pr_info("efi: Setup done, disabling due to 32/64-bit
>>> mismatch\n");
>>> + efi_enabled = 0;
>>> + }
>>> +#endif
>>> }
>> Won't this give a spurious warning if it's already disabled?
> Ah, of course, my bad. v3 forthcoming.
>
>> And it should probably be moved to before the vga con setup, else it seems
>> you
>> might not get a vga console when efifb is not used. Unless that's
>> intentional,
>> but in that case please change the commit message to reflect that. :-)
> Hmm. I think the current logic and flow is valid -- it's likely a bad
> idea to enable VGA console if the memory isn't set aside in the memory
> map, since that possibly means that EFI didn't POST graphics for VGA
> text mode in the first place.
>
>> With those issues fixed in next version.
>>
>> Acked-by: Maarten Lankhorst <maarten.lankho...@canonical.com>
> Please confirm if you agree with my reasoning above, I'll post v3
> right after if so.
>
Ack
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
