[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/8] Support PCI based option rom loading
From: |
Blue Swirl |
Subject: |
Re: [Qemu-devel] [PATCH 1/8] Support PCI based option rom loading |
Date: |
Sun, 20 Dec 2009 17:02:05 +0000 |
On Sun, Dec 20, 2009 at 3:34 PM, Anthony Liguori <address@hidden> wrote:
> Blue Swirl wrote:
>>
>> On Fri, Dec 18, 2009 at 11:01 AM, Gerd Hoffmann <address@hidden> wrote:
>>
>>>
>>> From: Anthony Liguori <address@hidden>
>>>
>>> Currently, we preload option roms into the option rom space in memory.
>>> This
>>> prevents DDIM from functioning correctly which severely limits the number
>>> of roms we can support.
>>>
>>> This patch introduces a pci_add_option_rom() which registers the
>>> PCI_ROM_ADDRESS bar which points to our option rom. It also converts
>>> over
>>> the cirrus vga adapter, the rtl8139, virtio, and the e1000 to use this
>>> new mechanism.
>>>
>>
>> This means that the VGA roms are now visible for all targets, not just
>> when rom_enable_driver_roms is true. I'm not opposing this, but it may
>> create regressions.
>>
>
> This is pretty normal on bare metal though, no? I thought a lot of non-x86
> systems actually use x86 emulators specifically to enable support for these
> roms.
I think so too, but ROM loading was first introduced with
rom_enable_driver_roms mechanism just for cases like this.
But if there are no regressions, rom_enable_driver_roms mechanism
could be removed.
- [Qemu-devel] [PATCH 0/8] option rom loading overhaul., Gerd Hoffmann, 2009/12/18
- [Qemu-devel] [PATCH 3/8] fw_cfg: make calls typesafe, Gerd Hoffmann, 2009/12/18
- [Qemu-devel] [PATCH 1/8] Support PCI based option rom loading, Gerd Hoffmann, 2009/12/18
- Re: [Qemu-devel] [PATCH 1/8] Support PCI based option rom loading, Blue Swirl, 2009/12/19
- Re: [Qemu-devel] [PATCH 1/8] Support PCI based option rom loading, Anthony Liguori, 2009/12/20
- Re: [Qemu-devel] [PATCH 1/8] Support PCI based option rom loading,
Blue Swirl <=
- Re: [Qemu-devel] [PATCH 1/8] Support PCI based option rom loading, Alexander Graf, 2009/12/20
- Re: [Qemu-devel] [PATCH 1/8] Support PCI based option rom loading, Anthony Liguori, 2009/12/20
- Re: [Qemu-devel] [PATCH 1/8] Support PCI based option rom loading, Alexander Graf, 2009/12/20
- Re: [Qemu-devel] [PATCH 1/8] Support PCI based option rom loading, Andreas Färber, 2009/12/20
- Re: [Qemu-devel] [PATCH 1/8] Support PCI based option rom loading, Blue Swirl, 2009/12/20
- Re: [Qemu-devel] [PATCH 1/8] Support PCI based option rom loading, Benjamin Herrenschmidt, 2009/12/20
- Re: [Qemu-devel] [PATCH 1/8] Support PCI based option rom loading, Benjamin Herrenschmidt, 2009/12/20
[Qemu-devel] [PATCH 8/8] debug: enable bios messages, Gerd Hoffmann, 2009/12/18
[Qemu-devel] [PATCH 4/8] fw_cfg: add API for file transfer., Gerd Hoffmann, 2009/12/18