Comment 3 for bug 948323

Revision history for this message
Stefan Bader (smb) wrote : Re: Rom images for e1000 and ne2k missign vendor and device id

The bug, if to say so, is that for kvm there have been additional rom image targets added. By the way the build system for ipxe works, I believe all the required rom images have been build already by the makeall (whatever it is called again). Just the names are different for some of them (for above reasons).
So a fix may indeed be:
1. remove the additional build target for allqemu (not 100% sure about virtio-net) or
    at least all beside of virtio-net
2. just copy them to different names (ne->ne2k_isa, 8086100e->e1000_82540).
The other two roms should be already present under their name (pcnet32 and rtl8139).

On the other hand I checked and it seems that kvm does not need the rom header to include the correct vendor and device id. So the change is not strictly required. For Xen we now have a patch to not include the kvm rom name but the auto build one. So it could just be a build optimization (not to build those two twice, probably just a linking step)