[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 11/13] libxl: set nic type to VIF by default

Ian Jackson wrote:
Roger Pau Monne writes ("[PATCH 11/13] libxl: set nic type to VIF by default"):
Set the default value for nic interfaces to VIF, since it used to be
IOEMU, even for PV guests.

How odd.

diff --git a/tools/libxl/libxl.c b/tools/libxl/libxl.c
index 2490138..631de15 100644
--- a/tools/libxl/libxl.c
+++ b/tools/libxl/libxl.c
@@ -1637,7 +1637,7 @@ int libxl__device_nic_setdefault(libxl__gc *gc, 
libxl_device_nic *nic)
                                    libxl__xen_script_dir_path())<  0 )
          return ERROR_FAIL;
      if (!nic->nictype)
-        nic->nictype = LIBXL_NIC_TYPE_IOEMU;
+        nic->nictype = LIBXL_NIC_TYPE_VIF;

But doesn't this set the default type to VIF even for HVM guests ?

Shouldn't HVM guests use the "ioemu" parameter if they want an emulated card? If no parameter is provided then the default type should be VIF, because there's no other way to specifically set the type to VIF.

Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.