linux-kernel - Re: Suspected regression in "x86: extend e820 ealy

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <48349B9D.1000501@goop.org>
Date:	Wed, 21 May 2008 23:01:01 +0100
From:	Jeremy Fitzhardinge <jeremy@...p.org>
To:	Yinghai Lu <yhlu.kernel@...il.com>
CC:	Ingo Molnar <mingo@...e.hu>,
	kernel list <linux-kernel@...r.kernel.org>,
	Thomas Gleixner <tglx@...utronix.de>,
	"H. Peter Anvin" <hpa@...or.com>
Subject: Re: Suspected regression in "x86: extend e820 ealy_res support 32bit"

Yinghai Lu wrote:
> On Wed, May 21, 2008 at 11:19 AM, Jeremy Fitzhardinge <jeremy@...p.org> wrote:
>   
>> I'm seeing a crash in current x86.git tip/auto-latest when booting under
>> Xen.  The crash is rather early, but it's in __alloc_bootmem_core() in the
>> final memset clear.  Apparently the allocator is returning a bad page.
>>
>> This points to changes in the setup of the bootmem allocator, and the
>> changes "x86: extend e820 ealy_res support 32bit" make to
>> arch/x86/kernel/setup_32.c:setup_bootmem_allocator() looks like the most
>> likely suspect.  Unfortunately its a rather large patch which is not easy to
>> revert, so I haven't actually confirmed this yet.
>>     
>
>
> thanks. please check the attached patch
>   

Thanks for the quick response.  I definitely confirmed that "x86: extend 
e820 ealy_res support 32bit" is the source of the regression, and 
reverting the change makes the system bootable under Xen. (My previous 
reversion failed because I didn't actually have the x86.git changes 
applied :-/)

This patch generates the warning:

      CC      arch/x86/kernel/setup_32.o
    /home/jeremy/hg/xen/paravirt/linux/arch/x86/kernel/setup_32.c: In
    function ‘setup_arch’:
    /home/jeremy/hg/xen/paravirt/linux/arch/x86/kernel/setup_32.c:580:
    warning: ‘bootmap_size’ is used uninitialized in this function

but I think it's just that the panic() message still uses it before its 
assigned.

Hm, and it also fails to fix the problem.  I'm still seeing the virtual 
machine crash in the same way.

Thanks,
    J
--
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/