Re: [Qemu-devel] Rewritten Linux kernel loader

2007-05-18 Thread Adam Lackorzynski
Hi, On Thu May 17, 2007 at 14:06:57 -0700, H. Peter Anvin wrote: +/* Generate an initial boot sector which sets state and jump to + a specified vector */ +static int generate_bootsect(uint32_t gpr[8], uint16_t segs[6], uint16_t ip) +{ +uint8_t bootsect[512], *p; +int i; + +

Re: [Qemu-devel] Rewritten Linux kernel loader

2007-05-18 Thread H. Peter Anvin
Adam Lackorzynski wrote: + +/* Make sure we have a partition signature */ +bootsect[0x510] = 0x55; +bootsect[0x511] = 0xaa; These two should be decimal instead of hex. Oh, yes, duh. 510 and 511 or 0x1fe and 0x1ff. -hpa

[Qemu-devel] Rewritten Linux kernel loader

2007-05-17 Thread H. Peter Anvin
I got a bug report today that my recent changes to the Linux kernel setup broke the Qemu kernel loader. I implemented a workaround, but found a number of serious bugs in the loader itself. As a consequence, I have rewritten it; here is a patch against qemu-0.9.0. As rewritten, it should follow

Re: [Qemu-devel] Rewritten Linux kernel loader

2007-05-17 Thread Ed Swierk
On Wednesday 16 May 2007 14:31:38 H. Peter Anvin wrote: As rewritten, it should follow the current version of the Linux boot protocol specification and recommendations. As a side benefit, it no longer relies on load_linux.S; instead I have a small code generator which can be used to set up an

Re: [Qemu-devel] Rewritten Linux kernel loader

2007-05-17 Thread Paul Brook
On Wednesday 16 May 2007, H. Peter Anvin wrote: I got a bug report today that my recent changes to the Linux kernel setup broke the Qemu kernel loader. I implemented a workaround, but found a number of serious bugs in the loader itself. As a consequence, I have rewritten it; here is a patch

Re: [Qemu-devel] Rewritten Linux kernel loader

2007-05-17 Thread H. Peter Anvin
H. Peter Anvin wrote: I got a bug report today that my recent changes to the Linux kernel setup broke the Qemu kernel loader. I implemented a workaround, but found a number of serious bugs in the loader itself. As a consequence, I have rewritten it; here is a patch against qemu-0.9.0. As