On Wed, Nov 8, 2017 at 9:05 PM, Anton Gerasimov <an...@advancedtelematic.com> wrote: > Adding Igor Mammedov to the loop. >
Really add Igor Mammedov. Igor, can you help look at this? > On 11/08/2017 01:59 PM, Anton Gerasimov wrote: >> To whoever might be interested: I've bisected qemu and the breaking >> commit is 208fa0e43645edd0b0d8f838857dfc79daff40a8 (pc: make 'pc.rom' >> readonly when machine has PCI enabled). It's just three lines added, >> I'll paste the whole patch here. Not quite sure what can we do here though. >> >> >> diff --git a/hw/i386/pc.c b/hw/i386/pc.c >> index 22e16031b0..59435390ba 100644 >> --- a/hw/i386/pc.c >> +++ b/hw/i386/pc.c >> @@ -1443,6 +1443,9 @@ void pc_memory_init(PCMachineState *pcms, >> option_rom_mr = g_malloc(sizeof(*option_rom_mr)); >> memory_region_init_ram(option_rom_mr, NULL, "pc.rom", PC_ROM_SIZE, >> &error_fatal); >> + if (pcmc->pci_enabled) { >> + memory_region_set_readonly(option_rom_mr, true); >> + } >> memory_region_add_subregion_overlap(rom_memory, >> PC_ROM_MIN_VGA, >> option_rom_mr, >> >> Regards, Bin