linux-kernel - Re: [Bug 14658] Regression in efi.c

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Mon, 30 Nov 2009 03:13:32 +1030
From:	indexer <indexer@...ernode.on.net>
To:	"Tang, Feng" <feng.tang@...el.com>
CC:	LKML <linux-kernel@...r.kernel.org>,
	"H. Peter Anvin" <hpa@...or.com>
Subject: Re: [Bug 14658] Regression in efi.c

Feng, Peter

I decided to go back a few versions and check out if there any previous 
bugs, and upon bisecting i found this one

commit 37ba7ab5e33cebc25c68fffe33e9f21e7c2014e8
Author: H. Peter Anvin <hpa@...or.com>
Date: Mon May 11 15:56:08 2009 -0700

x86, boot: make kernel_alignment adjustable; new bzImage fields

Make the kernel_alignment field adjustable; this allows us to set it
to a large value (intended to be 16 MB to avoid ZONE_DMA contention,
memory holes and other weirdness) while a smart bootloader can still
force a loading at a lesser alignment if absolutely necessary.

Also export pref_address (preferred loading address, corresponding to
the link-time address) and init_size, the total amount of linear
memory the kernel will require during initialization.

[ Impact: allows better kernel placement, gives bootloader more info ]

Signed-off-by: H. Peter Anvin <hpa@...or.com>

arch/x86/boot/compressed/head_32.S | 7 +++++--
arch/x86/boot/compressed/head_64.S | 14 ++++++++++----
arch/x86/boot/header.S | 15 +++++++++++++--
arch/x86/include/asm/boot.h | 15 +++++++++++++++
arch/x86/kernel/asm-offsets_32.c | 1 +
arch/x86/kernel/asm-offsets_64.c | 1 +
6 files changed, 45 insertions(+), 8 deletions(-)


This seems to stop efi booting as well on x86_64. I have also been able 
to recently test this on a gen 4 macbook pro as well as my own gen 5 to 
make sure it is not an issue specific to this model.

William

Tang, Feng wrote:
> William,
>
> You've confirmed that apply commit 772be899b "x86: Make EFI RTC function depend
> on 32bit again" right after 7bd867dfb "x86: Move get/set_wallclock to 
> x86_platform_ops" will get a bootable kernel.
>
> Then, one debug method may be use "git-rebase -i" to move commit 772be899b right after 
> 7bd867dfb, and do the bisect from there on. 
>
> Thanks,
> Feng
>   
>> -----Original Message-----
>> From: indexer [mailto:indexer@...ernode.on.net]
>> Sent: 2009年11月25日 17:43
>> To: Tang, Feng
>> Cc: LKML
>> Subject: Re: [Bug 14658] Regression in efi.c
>>
>> Feng tang
>>
>> I have just tested and applied that correction, and it resulted in a
>> working system. I dont really know where to go from here so any advice
>> would be appreciated.
>>
>> William
>>     
> N嫥叉靣笡y氊b瞂千v豝�)藓{.n�+壏{睉赙zXФ.洝塄}财爖�&j:+v墾�.珣赙zZ+€�+zf＂穐殘啳嗃i�z�.畐ア�?櫒璀�&�)撷.f旟^j谦y呩@A玜囤�.0鹅h�.鍜i

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/