]> git.proxmox.com Git - qemu.git/commit
Fix multiboot compilation
authorAlexander Graf <agraf@suse.de>
Tue, 1 Jun 2010 23:56:50 +0000 (01:56 +0200)
committerAurelien Jarno <aurelien@aurel32.net>
Wed, 9 Jun 2010 10:36:11 +0000 (12:36 +0200)
commit07442ab4a1ce0bb5939db5b724e26d6da2759875
treecb99f26ab4d6a46500f17c813a47ab41840bbe33
parentdbe6a18d821f6edd29010f6d23e37e2d90961634
Fix multiboot compilation

Commit dd4239d6574ca41c94fc0d0f77ddc728510ffc57 broke multiboot. It replaced the
instruction "rep insb (%dx), %es:(%edi)" by the binary output of
"addr32 rep insb (%dx), %es:(%di)".

Linuxboot calls the respective helper function in a code16 section. So the
original instruction was automatically translated to its "addr32" equivalent.
For multiboot, we're running in code32 so gcc didn't add the "addr32" which
breaks the instruction.

This patch splits that helper function in one which uses addr32 and one which
does not, so everyone's happy.

The good news is that nobody probably cared so far. The bundled multiboot.bin
binary was built before the change and is thus correct.

Please also put this patch into -stable.

Signed-off-by: Alexander Graf <agraf@suse.de>
Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>
(cherry picked from commit 590bf491a49670843ee902c47f7ab1de5e9acd06)
pc-bios/optionrom/linuxboot.S
pc-bios/optionrom/optionrom.h