> > $ git-grep pc-0 > > hw/display/vga-pci.c: /* compatibility with pc-0.13 and older */ > > hw/display/vga.c: /* With pc-0.12 and below we map both the PCI BAR and the fixed VBE region, > > hw/display/vmware_vga.c: /* compatibility with pc-0.13 and older */ > > These are the "rombar" hacks that I've mentioned above. The question is > whether we want to remove them or whether I should just adjust the comments? Hmm. All that cruft ... vga maps the framebuffer @ 0xe0000000 with rombar=off. It's an alias of the pci memory bar. rombar=off is basically a flag for "really old firmware" here, vgabios used to have that address hardcoded, a decade ago. We fixed that roughly the same timeframe where we switched to seabios, which in turn allowed us to place the vgabios in the pci rom bar (instead of copying it to 0xa000 in guest ram). Which is probably the reason why we have only one switch for both. I don't expect anyone actually sets the rombar property for vga devices (it's more common for NICs, for network boot tweaks), so I guess we can get away with simply dropping the hacks in vga-pci.c and vmware_vga.c. The comment in vga.c is not fully correct though, isa-vga needs that too, so we have to keep vga_init_vbe for the time being ... cheers, Gerd -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list