File 7c98d1c1-nic-type.patch of Package libvirt.4601

commit 7c98d1c153da5810ed4dcaa6be177df369b7d4bd
Author: Jim Fehlig <jfehlig@suse.com>
Date:   Mon Jan 6 11:37:20 2014 -0700

    libxl: Fix initialization of nictype in libxl_device_nic
    
    As pointed out by the Xen folks [1], HVM nics should always be set
    to type LIBXL_NIC_TYPE_VIF_IOEMU unless the user explicity requests
    LIBXL_NIC_TYPE_VIF via model='netfront'.  The current logic in
    libxlMakeNic() only sets the nictype to LIBXL_NIC_TYPE_VIF_IOEMU if
    a model is specified that is not 'netfront', which breaks PXE booting
    configurations where no model is specified (i.e. use the hypervisor
    default).
    
      Reported-by: Stefan Bader <stefan.bader@canonical.com>
    
    [1] https://www.redhat.com/archives/libvir-list/2013-December/msg01156.html

Index: libvirt-1.1.2/src/libxl/libxl_conf.c
===================================================================
--- libvirt-1.1.2.orig/src/libxl/libxl_conf.c
+++ libvirt-1.1.2/src/libxl/libxl_conf.c
@@ -815,8 +815,12 @@ error:
 }
 
 int
-libxlMakeNic(virDomainNetDefPtr l_nic, libxl_device_nic *x_nic)
+libxlMakeNic(virDomainDefPtr def,
+             virDomainNetDefPtr l_nic,
+             libxl_device_nic *x_nic)
 {
+    bool ioemu_nic = STREQ(def->os.type, "hvm");
+
     /* TODO: Where is mtu stored?
      *
      * x_nics[i].mtu = 1492;
@@ -826,12 +830,16 @@ libxlMakeNic(virDomainNetDefPtr l_nic, l
 
     virMacAddrGetRaw(&l_nic->mac, x_nic->mac);
 
-    if (l_nic->model && !STREQ(l_nic->model, "netfront")) {
-        if (VIR_STRDUP(x_nic->model, l_nic->model) < 0)
-            return -1;
+    if (ioemu_nic)
         x_nic->nictype = LIBXL_NIC_TYPE_VIF_IOEMU;
-    } else {
+    else
         x_nic->nictype = LIBXL_NIC_TYPE_VIF;
+
+    if (l_nic->model) {
+        if (VIR_STRDUP(x_nic->model, l_nic->model) < 0)
+            return -1;
+        if (STREQ(l_nic->model, "netfront"))
+            x_nic->nictype = LIBXL_NIC_TYPE_VIF;
     }
 
     if (VIR_STRDUP(x_nic->ifname, l_nic->ifname) < 0)
@@ -868,7 +876,7 @@ libxlMakeNicList(virDomainDefPtr def,  l
         return -1;
 
     for (i = 0; i < nnics; i++) {
-        if (libxlMakeNic(l_nics[i], &x_nics[i]))
+        if (libxlMakeNic(def, l_nics[i], &x_nics[i]))
             goto error;
     }
 
Index: libvirt-1.1.2/src/libxl/libxl_conf.h
===================================================================
--- libvirt-1.1.2.orig/src/libxl/libxl_conf.h
+++ libvirt-1.1.2/src/libxl/libxl_conf.h
@@ -126,7 +126,9 @@ libxlMakeCapabilities(libxl_ctx *ctx);
 int
 libxlMakeDisk(virDomainDiskDefPtr l_dev, libxl_device_disk *x_dev);
 int
-libxlMakeNic(virDomainNetDefPtr l_nic, libxl_device_nic *x_nic);
+libxlMakeNic(virDomainDefPtr def,
+             virDomainNetDefPtr l_nic,
+             libxl_device_nic *x_nic);
 int
 libxlMakeVfb(libxlDriverPrivatePtr driver,
              virDomainGraphicsDefPtr l_vfb, libxl_device_vfb *x_vfb);