Re: More E820 brokenness

2007-09-28 Thread Rafael J. Wysocki
On Friday, 28 September 2007 02:12, H. Peter Anvin wrote: > Jordan Crouse wrote: > > > > Worked, but that just raises more questions. Why didn't more x86 boxes > > break or, alternatively, why did a new version of the BIOS fix the problem? > > I guess we shouldn't look a gift horse in the

AW: More E820 brokenness

2007-09-28 Thread Joerg Pommnitz
<[EMAIL PROTECTED]> An: H. Peter Anvin <[EMAIL PROTECTED]> CC: [EMAIL PROTECTED]; Joerg Pommnitz <[EMAIL PROTECTED]>; Chuck Ebbert <[EMAIL PROTECTED]>; Linux Kernel Mailing List ; Andi Kleen <[EMAIL PROTECTED]> Gesendet: Freitag, den 28. September 2007, 01:15:52 Uhr Bet

AW: More E820 brokenness

2007-09-28 Thread Joerg Pommnitz
E820 brokenness On 27/09/07 15:47 -0700, H. Peter Anvin wrote: Jordan Crouse wrote: Breaks on the Geode - original behavior. I think that having boot_prams.e820_entries != 0 makes the kernel assume the e820 data is correct. Okay, now I'm utterly baffled how 2.6.22 ever worked

Re: More E820 brokenness

2007-09-28 Thread Rafael J. Wysocki
On Friday, 28 September 2007 02:12, H. Peter Anvin wrote: Jordan Crouse wrote: Worked, but that just raises more questions. Why didn't more x86 boxes break or, alternatively, why did a new version of the BIOS fix the problem? I guess we shouldn't look a gift horse in the mouth. Or

Re: More E820 brokenness

2007-09-27 Thread H. Peter Anvin
Jordan Crouse wrote: > > Worked, but that just raises more questions. Why didn't more x86 boxes > break or, alternatively, why did a new version of the BIOS fix the problem? > I guess we shouldn't look a gift horse in the mouth. Or something. > Why didn't more x86 boxes break... well, it's

Re: More E820 brokenness

2007-09-27 Thread Jordan Crouse
On 27/09/07 16:36 -0700, H. Peter Anvin wrote: > Jordan Crouse wrote: > >>> > >> Oh bugger, looks like this one might be genuinely my fault after all. > >> The ID check in the new code is buggy. > >> > >> Can you please test this revised patch out (against current -git)? > > > > > > That looks

Re: More E820 brokenness

2007-09-27 Thread H. Peter Anvin
Jordan Crouse wrote: >>> >> Oh bugger, looks like this one might be genuinely my fault after all. >> The ID check in the new code is buggy. >> >> Can you please test this revised patch out (against current -git)? > > > That looks the same as the previous patch you sent? > Sorry, this is the

Re: More E820 brokenness

2007-09-27 Thread Jordan Crouse
On 27/09/07 16:27 -0700, H. Peter Anvin wrote: > Jordan Crouse wrote: > > On 27/09/07 15:47 -0700, H. Peter Anvin wrote: > >> Jordan Crouse wrote: > >>> Breaks on the Geode - original behavior. > >>> > >>> I think that having boot_prams.e820_entries != 0 makes the kernel > >>> assume the e820 data

Re: More E820 brokenness

2007-09-27 Thread H. Peter Anvin
Jordan Crouse wrote: > On 27/09/07 15:47 -0700, H. Peter Anvin wrote: >> Jordan Crouse wrote: >>> Breaks on the Geode - original behavior. >>> >>> I think that having boot_prams.e820_entries != 0 makes the kernel >>> assume the e820 data is correct. >>> >> Okay, now I'm utterly baffled how 2.6.22

Re: More E820 brokenness

2007-09-27 Thread H. Peter Anvin
Jordan Crouse wrote: > > I copied in a 2.6.22 kernel to see that it really did work, and it did. > But here's the crazy part - I did a dmesg, and it looks like it > *is* using e820 data, and it looks complete (I see the entire map - > including the ACPI and reserved blocks way up high). > > So

Re: More E820 brokenness

2007-09-27 Thread Jordan Crouse
On 27/09/07 15:47 -0700, H. Peter Anvin wrote: > Jordan Crouse wrote: > > > > Breaks on the Geode - original behavior. > > > > I think that having boot_prams.e820_entries != 0 makes the kernel > > assume the e820 data is correct. > > > > Okay, now I'm utterly baffled how 2.6.22 ever worked on

Re: More E820 brokenness

2007-09-27 Thread H. Peter Anvin
Jordan Crouse wrote: > > Breaks on the Geode - original behavior. > > I think that having boot_prams.e820_entries != 0 makes the kernel > assume the e820 data is correct. > Okay, now I'm utterly baffled how 2.6.22 ever worked on this Geode, because this, to the best of my reading, mimics the

Re: More E820 brokenness

2007-09-27 Thread Jordan Crouse
On 27/09/07 15:17 -0700, H. Peter Anvin wrote: > As luck would have it, it's not just an obscure Geode system which has a > broken E820 implementation. Today I received a bug report about a Dell > system (XPS M1330) with broken E820. > > Unfortunately, the workaround for the Geode breaks this

More E820 brokenness

2007-09-27 Thread H. Peter Anvin
As luck would have it, it's not just an obscure Geode system which has a broken E820 implementation. Today I received a bug report about a Dell system (XPS M1330) with broken E820. Unfortunately, the workaround for the Geode breaks this system, because x86-64 doesn't fall back to the e801/88

More E820 brokenness

2007-09-27 Thread H. Peter Anvin
As luck would have it, it's not just an obscure Geode system which has a broken E820 implementation. Today I received a bug report about a Dell system (XPS M1330) with broken E820. Unfortunately, the workaround for the Geode breaks this system, because x86-64 doesn't fall back to the e801/88

Re: More E820 brokenness

2007-09-27 Thread H. Peter Anvin
Jordan Crouse wrote: I copied in a 2.6.22 kernel to see that it really did work, and it did. But here's the crazy part - I did a dmesg, and it looks like it *is* using e820 data, and it looks complete (I see the entire map - including the ACPI and reserved blocks way up high). So

Re: More E820 brokenness

2007-09-27 Thread Jordan Crouse
On 27/09/07 15:47 -0700, H. Peter Anvin wrote: Jordan Crouse wrote: Breaks on the Geode - original behavior. I think that having boot_prams.e820_entries != 0 makes the kernel assume the e820 data is correct. Okay, now I'm utterly baffled how 2.6.22 ever worked on this Geode,

Re: More E820 brokenness

2007-09-27 Thread H. Peter Anvin
Jordan Crouse wrote: On 27/09/07 15:47 -0700, H. Peter Anvin wrote: Jordan Crouse wrote: Breaks on the Geode - original behavior. I think that having boot_prams.e820_entries != 0 makes the kernel assume the e820 data is correct. Okay, now I'm utterly baffled how 2.6.22 ever worked on this

Re: More E820 brokenness

2007-09-27 Thread H. Peter Anvin
Jordan Crouse wrote: Breaks on the Geode - original behavior. I think that having boot_prams.e820_entries != 0 makes the kernel assume the e820 data is correct. Okay, now I'm utterly baffled how 2.6.22 ever worked on this Geode, because this, to the best of my reading, mimics the 2.6.22

Re: More E820 brokenness

2007-09-27 Thread Jordan Crouse
On 27/09/07 15:17 -0700, H. Peter Anvin wrote: As luck would have it, it's not just an obscure Geode system which has a broken E820 implementation. Today I received a bug report about a Dell system (XPS M1330) with broken E820. Unfortunately, the workaround for the Geode breaks this system,

Re: More E820 brokenness

2007-09-27 Thread Jordan Crouse
On 27/09/07 16:27 -0700, H. Peter Anvin wrote: Jordan Crouse wrote: On 27/09/07 15:47 -0700, H. Peter Anvin wrote: Jordan Crouse wrote: Breaks on the Geode - original behavior. I think that having boot_prams.e820_entries != 0 makes the kernel assume the e820 data is correct. Okay,

Re: More E820 brokenness

2007-09-27 Thread H. Peter Anvin
Jordan Crouse wrote: Oh bugger, looks like this one might be genuinely my fault after all. The ID check in the new code is buggy. Can you please test this revised patch out (against current -git)? That looks the same as the previous patch you sent? Sorry, this is the right one...

Re: More E820 brokenness

2007-09-27 Thread Jordan Crouse
On 27/09/07 16:36 -0700, H. Peter Anvin wrote: Jordan Crouse wrote: Oh bugger, looks like this one might be genuinely my fault after all. The ID check in the new code is buggy. Can you please test this revised patch out (against current -git)? That looks the same as the previous

Re: More E820 brokenness

2007-09-27 Thread H. Peter Anvin
Jordan Crouse wrote: Worked, but that just raises more questions. Why didn't more x86 boxes break or, alternatively, why did a new version of the BIOS fix the problem? I guess we shouldn't look a gift horse in the mouth. Or something. Why didn't more x86 boxes break... well, it's pretty