Revert "pc: Kill the "use flash device for BIOS unless KVM" misfeature"
This reverts commit 9953f8822c
.
While Markus's analysis is entirely correct, there are 1.6 patches
that fix the bug for real and without requiring machine type hacks.
Let's think of the children who will have to read this code, and
avoid a complicated mess of semantics that differ between <1.5,
1.5, and >1.5.
Conflicts:
hw/i386/pc_piix.c
hw/i386/pc_q35.c
include/hw/i386/pc.h
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Acked-by: Anthony Liguori <aliguori@us.ibm.com>
Message-id: 1368189483-7915-1-git-send-email-pbonzini@redhat.com
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
This commit is contained in:
parent
3556c233d9
commit
9e1c2ec8fd
|
@ -209,7 +209,7 @@ void pc_system_firmware_init(MemoryRegion *rom_memory)
|
||||||
* TODO This device exists only so that users can switch between
|
* TODO This device exists only so that users can switch between
|
||||||
* use of flash and ROM for the BIOS. The ability to switch was
|
* use of flash and ROM for the BIOS. The ability to switch was
|
||||||
* created because flash doesn't work with KVM. Once it does, we
|
* created because flash doesn't work with KVM. Once it does, we
|
||||||
* should drop this device for new machine types.
|
* should drop this device.
|
||||||
*/
|
*/
|
||||||
sysfw_dev = (PcSysFwDevice*) qdev_create(NULL, "pc-sysfw");
|
sysfw_dev = (PcSysFwDevice*) qdev_create(NULL, "pc-sysfw");
|
||||||
|
|
||||||
|
@ -226,9 +226,9 @@ void pc_system_firmware_init(MemoryRegion *rom_memory)
|
||||||
Use old rom based firmware initialization for KVM. */
|
Use old rom based firmware initialization for KVM. */
|
||||||
/*
|
/*
|
||||||
* This is a Bad Idea, because it makes enabling/disabling KVM
|
* This is a Bad Idea, because it makes enabling/disabling KVM
|
||||||
* guest-visible. Do it only in bug-compatibility mode.
|
* guest-visible. Let's fix it for real in QEMU 1.6.
|
||||||
*/
|
*/
|
||||||
if (pc_sysfw_flash_vs_rom_bug_compatible && kvm_enabled()) {
|
if (kvm_enabled()) {
|
||||||
if (pflash_drv != NULL) {
|
if (pflash_drv != NULL) {
|
||||||
fprintf(stderr, "qemu: pflash cannot be used with kvm enabled\n");
|
fprintf(stderr, "qemu: pflash cannot be used with kvm enabled\n");
|
||||||
exit(1);
|
exit(1);
|
||||||
|
@ -255,7 +255,7 @@ void pc_system_firmware_init(MemoryRegion *rom_memory)
|
||||||
}
|
}
|
||||||
|
|
||||||
static Property pcsysfw_properties[] = {
|
static Property pcsysfw_properties[] = {
|
||||||
DEFINE_PROP_UINT8("rom_only", PcSysFwDevice, rom_only, 1),
|
DEFINE_PROP_UINT8("rom_only", PcSysFwDevice, rom_only, 0),
|
||||||
DEFINE_PROP_END_OF_LIST(),
|
DEFINE_PROP_END_OF_LIST(),
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|
|
@ -248,7 +248,6 @@ static void pc_init_pci(QEMUMachineInitArgs *args)
|
||||||
|
|
||||||
static void pc_init_pci_1_4(QEMUMachineInitArgs *args)
|
static void pc_init_pci_1_4(QEMUMachineInitArgs *args)
|
||||||
{
|
{
|
||||||
pc_sysfw_flash_vs_rom_bug_compatible = true;
|
|
||||||
has_pvpanic = false;
|
has_pvpanic = false;
|
||||||
x86_cpu_compat_set_features("n270", FEAT_1_ECX, 0, CPUID_EXT_MOVBE);
|
x86_cpu_compat_set_features("n270", FEAT_1_ECX, 0, CPUID_EXT_MOVBE);
|
||||||
pc_init_pci(args);
|
pc_init_pci(args);
|
||||||
|
@ -257,7 +256,6 @@ static void pc_init_pci_1_4(QEMUMachineInitArgs *args)
|
||||||
static void pc_init_pci_1_3(QEMUMachineInitArgs *args)
|
static void pc_init_pci_1_3(QEMUMachineInitArgs *args)
|
||||||
{
|
{
|
||||||
enable_compat_apic_id_mode();
|
enable_compat_apic_id_mode();
|
||||||
pc_sysfw_flash_vs_rom_bug_compatible = true;
|
|
||||||
has_pvpanic = false;
|
has_pvpanic = false;
|
||||||
pc_init_pci(args);
|
pc_init_pci(args);
|
||||||
}
|
}
|
||||||
|
@ -267,7 +265,6 @@ static void pc_init_pci_1_2(QEMUMachineInitArgs *args)
|
||||||
{
|
{
|
||||||
disable_kvm_pv_eoi();
|
disable_kvm_pv_eoi();
|
||||||
enable_compat_apic_id_mode();
|
enable_compat_apic_id_mode();
|
||||||
pc_sysfw_flash_vs_rom_bug_compatible = true;
|
|
||||||
has_pvpanic = false;
|
has_pvpanic = false;
|
||||||
pc_init_pci(args);
|
pc_init_pci(args);
|
||||||
}
|
}
|
||||||
|
|
|
@ -210,7 +210,6 @@ static void pc_q35_init(QEMUMachineInitArgs *args)
|
||||||
|
|
||||||
static void pc_q35_init_1_4(QEMUMachineInitArgs *args)
|
static void pc_q35_init_1_4(QEMUMachineInitArgs *args)
|
||||||
{
|
{
|
||||||
pc_sysfw_flash_vs_rom_bug_compatible = true;
|
|
||||||
has_pvpanic = false;
|
has_pvpanic = false;
|
||||||
x86_cpu_compat_set_features("n270", FEAT_1_ECX, 0, CPUID_EXT_MOVBE);
|
x86_cpu_compat_set_features("n270", FEAT_1_ECX, 0, CPUID_EXT_MOVBE);
|
||||||
pc_q35_init(args);
|
pc_q35_init(args);
|
||||||
|
|
|
@ -169,7 +169,6 @@ static inline bool isa_ne2000_init(ISABus *bus, int base, int irq, NICInfo *nd)
|
||||||
}
|
}
|
||||||
|
|
||||||
/* pc_sysfw.c */
|
/* pc_sysfw.c */
|
||||||
extern bool pc_sysfw_flash_vs_rom_bug_compatible;
|
|
||||||
void pc_system_firmware_init(MemoryRegion *rom_memory);
|
void pc_system_firmware_init(MemoryRegion *rom_memory);
|
||||||
|
|
||||||
/* pvpanic.c */
|
/* pvpanic.c */
|
||||||
|
@ -238,10 +237,6 @@ int e820_add_entry(uint64_t, uint64_t, uint32_t);
|
||||||
.driver = "virtio-net-pci",\
|
.driver = "virtio-net-pci",\
|
||||||
.property = "romfile",\
|
.property = "romfile",\
|
||||||
.value = "pxe-virtio.rom",\
|
.value = "pxe-virtio.rom",\
|
||||||
},{\
|
|
||||||
.driver = "pc-sysfw",\
|
|
||||||
.property = "rom_only",\
|
|
||||||
.value = stringify(0),\
|
|
||||||
},{\
|
},{\
|
||||||
.driver = "486-" TYPE_X86_CPU,\
|
.driver = "486-" TYPE_X86_CPU,\
|
||||||
.property = "model",\
|
.property = "model",\
|
||||||
|
|
Loading…
Reference in a new issue