Skip to contentSkip to conversation

extboot.bin required in git

DISCUSSION Jan Kiszka 2 participants 10 messages Sep 2, 2009 KVM

lore
Share & tools
lore

Jan Kiszka

author
#0 lore raw

Hi Marcelo,

qemu-kvm's installation (0.11-rc as well as git head) is currently broken /wrt extboot.bin. Please check some proper binary in as pc-bios/extboot.bin.

Thanks,
Jan

Signature · 2 lines
Siemens AG, Corporate Technology, CT SE 2
Corporate Competence Center Embedded Linux
#1 lore raw
Quoted 6 lines · Jan Kiszka

Hi Marcelo,

qemu-kvm's installation (0.11-rc as well as git head) is currently broken /wrt extboot.bin. Please check some proper binary in as pc-bios/extboot.bin.

It's built at runtime. Why check in a binary?

Signature · 1 line
error compiling committee.c: too many arguments to function
author
#2 lore raw
Quoted 9 lines · Avi Kivity
Quoted 6 lines · Jan Kiszka

Hi Marcelo,

qemu-kvm's installation (0.11-rc as well as git head) is currently broken /wrt extboot.bin. Please check some proper binary in as pc-bios/extboot.bin.

It's built at runtime. Why check in a binary?

First of all, it's a qemu policy (AFAIK): try to build the rom, use shipped version if that doesn't work. And second, we (more precisely qemu) does not seem to use the optionrom output for installation yet. The latter is likely subject to change, but the former is expected to stay.

Jan

Signature · 2 lines
Siemens AG, Corporate Technology, CT SE 2
Corporate Competence Center Embedded Linux
#3 lore raw
Quoted 11 lines · Jan Kiszka
Quoted 9 lines · Avi Kivity
Quoted 7 lines · Jan Kiszka

Hi Marcelo,

qemu-kvm's installation (0.11-rc as well as git head) is currently broken /wrt extboot.bin. Please check some proper binary in as pc-bios/extboot.bin.

Quoted 4 lines
Quoted 2 lines

It's built at runtime. Why check in a binary?

First of all, it's a qemu policy (AFAIK): try to build the rom, use shipped version if that doesn't work.

Right, but this is qemu-kvm, which doesn't have the same issues (building an i386 target on non-i386). When extboot is merged into qemu.git, they can add the build-if-possible-otherwise-use-shipped thing.

Quoted 4 lines

And second, we (more precisely
qemu) does not seem to use the optionrom output for installation yet. The latter is likely subject to change, but the former is expected to stay.

We do, see 321b6ef91.

Signature · 1 line
error compiling committee.c: too many arguments to function
author
#4 lore raw
Quoted 12 lines · Avi Kivity
Quoted 11 lines · Jan Kiszka
Quoted 9 lines · Avi Kivity
Quoted 7 lines · Jan Kiszka

Hi Marcelo,

qemu-kvm's installation (0.11-rc as well as git head) is currently broken /wrt extboot.bin. Please check some proper binary in as pc-bios/extboot.bin.

Quoted 8 lines
Quoted 4 lines
Quoted 2 lines

It's built at runtime. Why check in a binary?

First of all, it's a qemu policy (AFAIK): try to build the rom, use shipped version if that doesn't work.

Right, but this is qemu-kvm, which doesn't have the same issues (building an i386 target on non-i386). When extboot is merged into qemu.git, they can add the build-if-possible-otherwise-use-shipped thing.

Well, people may expect qemu-kvm's source tree to build properly also for non-kvm usage. But this is not that important for now.

Quoted 7 lines
Quoted 5 lines

And second, we (more precisely
qemu) does not seem to use the optionrom output for installation yet. The latter is likely subject to change, but the former is expected to stay.

We do, see 321b6ef91.

Ah, I see: some out-of-tree build bug. Will fix.

Jan

Signature · 2 lines
Siemens AG, Corporate Technology, CT SE 2
Corporate Competence Center Embedded Linux
author
#5 lore raw
Quoted 15 lines · Jan Kiszka
Quoted 14 lines · Avi Kivity
Quoted 13 lines · Jan Kiszka
Quoted 11 lines · Avi Kivity
Quoted 7 lines · Jan Kiszka

Hi Marcelo,

qemu-kvm's installation (0.11-rc as well as git head) is currently broken /wrt extboot.bin. Please check some proper binary in as pc-bios/extboot.bin.

It's built at runtime. Why check in a binary?

Quoted 8 lines
Quoted 5 lines
Quoted 2 lines

First of all, it's a qemu policy (AFAIK): try to build the rom, use shipped version if that doesn't work.

Right, but this is qemu-kvm, which doesn't have the same issues (building an i386 target on non-i386). When extboot is merged into qemu.git, they can add the build-if-possible-otherwise-use-shipped thing.

Well, people may expect qemu-kvm's source tree to build properly also for non-kvm usage. But this is not that important for now.

... it just requires another patch to qemu's Makefile. Well.

Quoted 8 lines
Quoted 6 lines
Quoted 5 lines

And second, we (more precisely
qemu) does not seem to use the optionrom output for installation yet. The latter is likely subject to change, but the former is expected to stay.

We do, see 321b6ef91.

Ah, I see: some out-of-tree build bug. Will fix.

And this patch is missing in master, isn't it?

Jan

Signature · 2 lines
Siemens AG, Corporate Technology, CT SE 2
Corporate Competence Center Embedded Linux
#6 lore raw
Quoted 5 lines · Jan Kiszka
Quoted 3 lines

Ah, I see: some out-of-tree build bug. Will fix.

And this patch is missing in master, isn't it?

IIRC it works in master somehow.

Signature · 1 line
error compiling committee.c: too many arguments to function
author
#7 lore raw
Quoted 8 lines · Avi Kivity
Quoted 4 lines · Jan Kiszka
Quoted 2 lines

Ah, I see: some out-of-tree build bug. Will fix.

And this patch is missing in master, isn't it?

IIRC it works in master somehow.

Old extboot.bin still hanging around in your pc-bios folder...? :)

Jan

Signature · 2 lines
Siemens AG, Corporate Technology, CT SE 2
Corporate Competence Center Embedded Linux
#8 lore raw
Quoted 6 lines · Jan Kiszka
Quoted 3 lines

IIRC it works in master somehow.

Old extboot.bin still hanging around in your pc-bios folder...? :)

I'm sure I deleted them... but can't argue with reality.

Signature · 1 line
error compiling committee.c: too many arguments to function
#9 lore raw
In reply to Jan Kiszka
Quoted 3 lines · Jan Kiszka

Well, people may expect qemu-kvm's source tree to build properly also for non-kvm usage. But this is not that important for now.

It's not tested for non-kvm usage unfortunately.

Quoted 3 lines

Ah, I see: some out-of-tree build bug. Will fix.

That too is untested. Thanks.

Signature · 1 line
error compiling committee.c: too many arguments to function