Andrew Morton wrote:
Parts of the 16-bit real mode loader code are now being compiled as C code
with gcc in 32 bit mode passing the .code16gcc directive to the assembler to
correct the stack frames to 16 bit. This kludge won't work unless all the
16-bit segment registers are set to the same value. Gcc only manipulates
the offset of the address and doesn't know anything about segment registers
or segment override prefixes. My lilo was setting SS=0x8000, DS=0x9000, and
SP=0xB000 before entering the kernel loader. This makes stack automatics
unreachable from the data segment without segment override prefixes.
I was tempted to patch the kernel code, but instead decided to try
"upgrading" lilo to grub-0.97 and found that grub works just fine. This
also has the significant advantage that we won't need those nasty as86 and
ld86 things any more since lilo was the last package on our systems that
used them.
However, it would probably be a good idea to modify the kernel loader to
lock out interrupts and explicitly set up the stack in its assembly startup
code to insure that the stack is located correctly above the code in the
same segment, rather than relying on the boot loader to do the right thing.
The existing setup code already insures that the other segment registers are
equal but omits the stack segment register. Also, because lilo (and
others?) loads the data/code segment at 0X90000, the stack pointer would
have to be set no higher than 0XA000 to avoid potential overwrites of the
EBDA. But I believe from my look at the code that the data/code sits below
0X8000 in the segment, so this should be fine.
If others think this is a good thing, I will test and submit a patch.
I think this is a good thing ;)
Not quite so fast. The entry value of SS:SP is actually part of the
protocol (an upper memory boundary), although for 2.01+ one could argue
it is redundant with the heap_end field in the header.
I'm rather confused which particular LILO this would possibly be,
especially given the oddball version number. The boot protocol was
pretty much formalized by Werner Amsberger (sp?), the original LILO
author, with contributions from Hans Lermen and myself. It hasn't
changed in this area.
If this was a LILO that someone "cleverly broke" I'd like to understand
the nature of it, so we can work around it properly. I see a couple of
options:
- If protocol >= 2.01, force (e)sp to match the heap_end field of the
setup structure. For < 2.01, what to do?
- Pray and hope the value of SP is sane to start out with in the correct SS.
- Declare the "cleverly broken" version of LILO not so cleverly broken.
For what it's worth, the old code, for protocol < 2.02, the boot code
would simply overwrite %ss, leaving %sp unchanged (alternative #2.) So
this configuration was always buggy. There is a comment in the old code
(setup.S, line 655) that "after this the stack should not be used", but
we then go right into the A20 code which does a bunch of subroutine calls.
I think at this point that if protocol >= 2.01 and CAN_USE_HEAP, we
should set %ss:%sp to that, otherwise fall back to simply setting %ss
and hope that %sp is set to something sane. I don't like it, but I
don't see any better alternative.
-hpa
-hpa
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
