Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-09 Thread H. Peter Anvin
Alexander van Heukelum wrote: > On Wed, May 09, 2007 at 09:51:36AM -0700, H. Peter Anvin wrote: >> Alexander van Heukelum wrote: >>> Hi! >>> >>> I'm making coffee now. I just don't see what I missed? Maybe you were >>> led astray by the new PARAM_VESA_PAD I added? >>> >> Again, I object to

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-09 Thread Alexander van Heukelum
On Wed, May 09, 2007 at 09:51:36AM -0700, H. Peter Anvin wrote: > Alexander van Heukelum wrote: > > > > Hi! > > > > I'm making coffee now. I just don't see what I missed? Maybe you were > > led astray by the new PARAM_VESA_PAD I added? > > > > Again, I object to changing a documented interface

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-09 Thread H. Peter Anvin
Alexander van Heukelum wrote: > > Hi! > > I'm making coffee now. I just don't see what I missed? Maybe you were > led astray by the new PARAM_VESA_PAD I added? > Again, I object to changing a documented interface when it's not necessary to do so. Just declare the structure packed.

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-09 Thread Antonino A. Daplas
On Wed, 2007-05-09 at 10:30 -0400, Lennart Sorensen wrote: > On Wed, May 09, 2007 at 08:04:07AM +0800, Antonino A. Daplas wrote: > > On Tue, 2007-05-08 at 20:32 +0200, Alexander van Heukelum wrote: > > > On Tue, May 08, 2007 at 03:28:17AM -0700, Andrew Morton wrote: > > > > On Sat, 5 May 2007

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-09 Thread Lennart Sorensen
On Wed, May 09, 2007 at 08:04:07AM +0800, Antonino A. Daplas wrote: > On Tue, 2007-05-08 at 20:32 +0200, Alexander van Heukelum wrote: > > On Tue, May 08, 2007 at 03:28:17AM -0700, Andrew Morton wrote: > > > On Sat, 5 May 2007 12:44:52 +0200 Alexander van Heukelum <[EMAIL > > > PROTECTED]> wrote:

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-09 Thread Antonino A. Daplas
On Wed, 2007-05-09 at 10:54 +0200, Alexander van Heukelum wrote: > On Wed, May 09, 2007 at 08:04:07AM +0800, Antonino A. Daplas wrote: > > On Tue, 2007-05-08 at 20:32 +0200, Alexander van Heukelum wrote: > > > diff --git a/arch/i386/boot/bootsect.S b/arch/i386/boot/bootsect.S > > > diff --git

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-09 Thread Alexander van Heukelum
On Wed, May 09, 2007 at 08:04:07AM +0800, Antonino A. Daplas wrote: > On Tue, 2007-05-08 at 20:32 +0200, Alexander van Heukelum wrote: > > diff --git a/arch/i386/boot/bootsect.S b/arch/i386/boot/bootsect.S > > diff --git a/arch/i386/boot/video.S b/arch/i386/boot/video.S > > index 8143c95..8e404cb

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-09 Thread Alexander van Heukelum
On Wed, May 09, 2007 at 08:04:07AM +0800, Antonino A. Daplas wrote: On Tue, 2007-05-08 at 20:32 +0200, Alexander van Heukelum wrote: diff --git a/arch/i386/boot/bootsect.S b/arch/i386/boot/bootsect.S diff --git a/arch/i386/boot/video.S b/arch/i386/boot/video.S index 8143c95..8e404cb 100644

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-09 Thread Antonino A. Daplas
On Wed, 2007-05-09 at 10:54 +0200, Alexander van Heukelum wrote: On Wed, May 09, 2007 at 08:04:07AM +0800, Antonino A. Daplas wrote: On Tue, 2007-05-08 at 20:32 +0200, Alexander van Heukelum wrote: diff --git a/arch/i386/boot/bootsect.S b/arch/i386/boot/bootsect.S diff --git

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-09 Thread Lennart Sorensen
On Wed, May 09, 2007 at 08:04:07AM +0800, Antonino A. Daplas wrote: On Tue, 2007-05-08 at 20:32 +0200, Alexander van Heukelum wrote: On Tue, May 08, 2007 at 03:28:17AM -0700, Andrew Morton wrote: On Sat, 5 May 2007 12:44:52 +0200 Alexander van Heukelum [EMAIL PROTECTED] wrote: ---

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-09 Thread Antonino A. Daplas
On Wed, 2007-05-09 at 10:30 -0400, Lennart Sorensen wrote: On Wed, May 09, 2007 at 08:04:07AM +0800, Antonino A. Daplas wrote: On Tue, 2007-05-08 at 20:32 +0200, Alexander van Heukelum wrote: On Tue, May 08, 2007 at 03:28:17AM -0700, Andrew Morton wrote: On Sat, 5 May 2007 12:44:52 +0200

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-09 Thread H. Peter Anvin
Alexander van Heukelum wrote: Hi! I'm making coffee now. I just don't see what I missed? Maybe you were led astray by the new PARAM_VESA_PAD I added? Again, I object to changing a documented interface when it's not necessary to do so. Just declare the structure packed. -hpa -

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-09 Thread Alexander van Heukelum
On Wed, May 09, 2007 at 09:51:36AM -0700, H. Peter Anvin wrote: Alexander van Heukelum wrote: Hi! I'm making coffee now. I just don't see what I missed? Maybe you were led astray by the new PARAM_VESA_PAD I added? Again, I object to changing a documented interface when it's not

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-09 Thread H. Peter Anvin
Alexander van Heukelum wrote: On Wed, May 09, 2007 at 09:51:36AM -0700, H. Peter Anvin wrote: Alexander van Heukelum wrote: Hi! I'm making coffee now. I just don't see what I missed? Maybe you were led astray by the new PARAM_VESA_PAD I added? Again, I object to changing a documented

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-08 Thread Antonino A. Daplas
On Tue, 2007-05-08 at 20:32 +0200, Alexander van Heukelum wrote: > On Tue, May 08, 2007 at 03:28:17AM -0700, Andrew Morton wrote: > > On Sat, 5 May 2007 12:44:52 +0200 Alexander van Heukelum <[EMAIL > > PROTECTED]> wrote: > > > --- a/arch/i386/boot/bootsect.S > > > +++ b/arch/i386/boot/bootsect.S

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-08 Thread H. Peter Anvin
Alexander van Heukelum wrote: > On Tue, May 08, 2007 at 11:45:47AM -0700, H. Peter Anvin wrote: >> Alexander van Heukelum wrote: >>> Oh! A padding hole in a struct! That could be a problem. If the freeze >>> is after decompression, could you test if this makes it work again? >>> >> The correct fix

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-08 Thread Andrew Morton
On Tue, 8 May 2007 20:32:32 +0200 Alexander van Heukelum <[EMAIL PROTECTED]> wrote: > On Tue, May 08, 2007 at 03:28:17AM -0700, Andrew Morton wrote: > > On Sat, 5 May 2007 12:44:52 +0200 Alexander van Heukelum <[EMAIL > > PROTECTED]> wrote: > > > --- a/arch/i386/boot/bootsect.S > > > +++

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-08 Thread Alexander van Heukelum
On Tue, May 08, 2007 at 11:45:47AM -0700, H. Peter Anvin wrote: > Alexander van Heukelum wrote: > > > > Oh! A padding hole in a struct! That could be a problem. If the freeze > > is after decompression, could you test if this makes it work again? > > > > The correct fix is to apply

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-08 Thread H. Peter Anvin
Alexander van Heukelum wrote: > > Oh! A padding hole in a struct! That could be a problem. If the freeze > is after decompression, could you test if this makes it work again? > The correct fix is to apply __attribute__((packed)) to this structure. Of course, changing the boot sector will cause

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-08 Thread Alexander van Heukelum
On Tue, May 08, 2007 at 03:28:17AM -0700, Andrew Morton wrote: > On Sat, 5 May 2007 12:44:52 +0200 Alexander van Heukelum <[EMAIL PROTECTED]> > wrote: > > --- a/arch/i386/boot/bootsect.S > > +++ b/arch/i386/boot/bootsect.S > > @@ -44,7 +44,7 @@ #endif > > _start: > > > > # Normalize the

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-08 Thread H. Peter Anvin
Andrew Morton wrote: > > Without patch: > > (gdb) x/20i _start > 0x0 <_start>: ljmpw $0x0,$0x8 > 0x6 <_start+6>: rolb $0x8c,(%edi) > 0x9 : enter $0xd88e,$0x8e > 0xd : rorb $0xfb,0x7c00bcd0(%esi) > 0x14 : cld > 0x15 : mov$0x20ac0031,%esi > 0x1a : (bad) > 0x1b :

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-08 Thread Andrew Morton
On Tue, 08 May 2007 04:25:00 -0700 "H. Peter Anvin" <[EMAIL PROTECTED]> wrote: > Andrew Morton wrote: > > >> > >># Normalize the start address > >> - jmpl$BOOTSEG, $start2 > >> + jmpw$BOOTSEG, $start2 > > > > Sigh, another blow struck in the ongoing struggle between my Vaio and

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-08 Thread H. Peter Anvin
Andi Kleen wrote: > > You should have put that into a different mail (is there a electron shortage > somewhere now? @) Ok, thanks for the well researched bug report. Will fix > in > my tree. > Nah. Just a brain shortage at 5 in the morning. -hpa - To unsubscribe from this list:

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-08 Thread Andi Kleen
On Tuesday 08 May 2007 13:25, H. Peter Anvin wrote: > Andrew Morton wrote: > >># Normalize the start address > >> - jmpl$BOOTSEG, $start2 > >> + jmpw$BOOTSEG, $start2 > > > > Sigh, another blow struck in the ongoing struggle between my Vaio and the > > rest of the world. > > > >

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-08 Thread H. Peter Anvin
Andrew Morton wrote: >> >> # Normalize the start address >> -jmpl$BOOTSEG, $start2 >> +jmpw$BOOTSEG, $start2 > > Sigh, another blow struck in the ongoing struggle between my Vaio and the > rest of the world. > > Stone-cold black-screen lockup immediately upon boot. > >

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-08 Thread Andrew Morton
On Sat, 5 May 2007 12:44:52 +0200 Alexander van Heukelum <[EMAIL PROTECTED]> wrote: > The x86 bzImage contains a stub to inform people that it is not possible > any more to run a Linux kernel by catting it to a floppy and then > booting from it. This was meant to be all 16-bit code. The first >

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-08 Thread Andrew Morton
On Sat, 5 May 2007 12:44:52 +0200 Alexander van Heukelum [EMAIL PROTECTED] wrote: The x86 bzImage contains a stub to inform people that it is not possible any more to run a Linux kernel by catting it to a floppy and then booting from it. This was meant to be all 16-bit code. The first

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-08 Thread H. Peter Anvin
Andrew Morton wrote: # Normalize the start address -jmpl$BOOTSEG, $start2 +jmpw$BOOTSEG, $start2 Sigh, another blow struck in the ongoing struggle between my Vaio and the rest of the world. Stone-cold black-screen lockup immediately upon boot. Stock FC5

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-08 Thread Andi Kleen
On Tuesday 08 May 2007 13:25, H. Peter Anvin wrote: Andrew Morton wrote: # Normalize the start address - jmpl$BOOTSEG, $start2 + jmpw$BOOTSEG, $start2 Sigh, another blow struck in the ongoing struggle between my Vaio and the rest of the world. Stone-cold black-screen

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-08 Thread H. Peter Anvin
Andi Kleen wrote: You should have put that into a different mail (is there a electron shortage somewhere now? @) Ok, thanks for the well researched bug report. Will fix in my tree. Nah. Just a brain shortage at 5 in the morning. -hpa - To unsubscribe from this list: send the

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-08 Thread Andrew Morton
On Tue, 08 May 2007 04:25:00 -0700 H. Peter Anvin [EMAIL PROTECTED] wrote: Andrew Morton wrote: # Normalize the start address - jmpl$BOOTSEG, $start2 + jmpw$BOOTSEG, $start2 Sigh, another blow struck in the ongoing struggle between my Vaio and the rest of the

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-08 Thread H. Peter Anvin
Andrew Morton wrote: Without patch: (gdb) x/20i _start 0x0 _start: ljmpw $0x0,$0x8 0x6 _start+6: rolb $0x8c,(%edi) 0x9 start2+1: enter $0xd88e,$0x8e 0xd start2+5: rorb $0xfb,0x7c00bcd0(%esi) 0x14 start2+12: cld 0x15 start2+13: mov$0x20ac0031,%esi 0x1a

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-08 Thread Alexander van Heukelum
On Tue, May 08, 2007 at 03:28:17AM -0700, Andrew Morton wrote: On Sat, 5 May 2007 12:44:52 +0200 Alexander van Heukelum [EMAIL PROTECTED] wrote: --- a/arch/i386/boot/bootsect.S +++ b/arch/i386/boot/bootsect.S @@ -44,7 +44,7 @@ #endif _start: # Normalize the start address -

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-08 Thread H. Peter Anvin
Alexander van Heukelum wrote: Oh! A padding hole in a struct! That could be a problem. If the freeze is after decompression, could you test if this makes it work again? The correct fix is to apply __attribute__((packed)) to this structure. Of course, changing the boot sector will cause

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-08 Thread Alexander van Heukelum
On Tue, May 08, 2007 at 11:45:47AM -0700, H. Peter Anvin wrote: Alexander van Heukelum wrote: Oh! A padding hole in a struct! That could be a problem. If the freeze is after decompression, could you test if this makes it work again? The correct fix is to apply __attribute__((packed))

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-08 Thread Andrew Morton
On Tue, 8 May 2007 20:32:32 +0200 Alexander van Heukelum [EMAIL PROTECTED] wrote: On Tue, May 08, 2007 at 03:28:17AM -0700, Andrew Morton wrote: On Sat, 5 May 2007 12:44:52 +0200 Alexander van Heukelum [EMAIL PROTECTED] wrote: --- a/arch/i386/boot/bootsect.S +++

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-08 Thread H. Peter Anvin
Alexander van Heukelum wrote: On Tue, May 08, 2007 at 11:45:47AM -0700, H. Peter Anvin wrote: Alexander van Heukelum wrote: Oh! A padding hole in a struct! That could be a problem. If the freeze is after decompression, could you test if this makes it work again? The correct fix is to apply

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-08 Thread Antonino A. Daplas
On Tue, 2007-05-08 at 20:32 +0200, Alexander van Heukelum wrote: On Tue, May 08, 2007 at 03:28:17AM -0700, Andrew Morton wrote: On Sat, 5 May 2007 12:44:52 +0200 Alexander van Heukelum [EMAIL PROTECTED] wrote: --- a/arch/i386/boot/bootsect.S +++ b/arch/i386/boot/bootsect.S @@ -44,7

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-05 Thread H. Peter Anvin
Acked-by: H. Peter Anvin <[EMAIL PROTECTED]> Alexander van Heukelum wrote: > Hi! > > The x86 bzImage contains a stub to inform people that it is not possible > any more to run a Linux kernel by catting it to a floppy and then > booting from it. This was meant to be all 16-bit code. The first >

[PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-05 Thread Alexander van Heukelum
Hi! The x86 bzImage contains a stub to inform people that it is not possible any more to run a Linux kernel by catting it to a floppy and then booting from it. This was meant to be all 16-bit code. The first instruction, however, ended up as being coded as a 16:32-bit far jump. I assume the

[PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-05 Thread Alexander van Heukelum
Hi! The x86 bzImage contains a stub to inform people that it is not possible any more to run a Linux kernel by catting it to a floppy and then booting from it. This was meant to be all 16-bit code. The first instruction, however, ended up as being coded as a 16:32-bit far jump. I assume the

Re: [PATCH] Make bootsector stub 16-bit-only (i386)

2007-05-05 Thread H. Peter Anvin
Acked-by: H. Peter Anvin [EMAIL PROTECTED] Alexander van Heukelum wrote: Hi! The x86 bzImage contains a stub to inform people that it is not possible any more to run a Linux kernel by catting it to a floppy and then booting from it. This was meant to be all 16-bit code. The first