[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] network: Added option to disable NIC option rom
From: |
Gerd Hoffmann |
Subject: |
Re: [Qemu-devel] [PATCH] network: Added option to disable NIC option roms |
Date: |
Thu, 12 Jan 2012 08:58:39 +0100 |
User-agent: |
Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.24) Gecko/20111104 Red Hat/3.1.16-2.el6_1 Thunderbird/3.1.16 |
Hi,
> I'm not sure about the consequences (hotplugging feature, etc.) when
> changing it to romfile as in other PCI devices.
There should be no noticable difference.
> Also the patch is more
> generic and supports static and dynamic devices (hotplugable and
> possible future devices).
Hotplug works just fine for the other pci devices.
> Patch supports without hotplugging both
> options to disable the romfile:
> ,romfile=
> ,romfile=disabled
That should be a separate patch. And it should update
pci_add_option_rom() so it works equally for all pci devices.
> And the patch has already been tested ...
Well, it adds a bunch of code which would not be needed in the first
place if you would simply make use of the romfile support of the pci layer.
cheers,
Gerd