From mboxrd@z Thu Jan 1 00:00:00 1970 From: Roger Pau Monne Subject: Re: [PATCH 11/13] libxl: set nic type to VIF by default Date: Wed, 30 May 2012 13:03:52 +0100 Message-ID: <4FC60CA8.4090404@citrix.com> References: <1337184716-49276-1-git-send-email-roger.pau@citrix.com> <1337184716-49276-12-git-send-email-roger.pau@citrix.com> <20406.31676.266501.253510@mariner.uk.xensource.com> <4FBA6D52.9050809@citrix.com> <20420.57290.841831.831419@mariner.uk.xensource.com> <1338302817.14158.120.camel@zakaz.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1338302817.14158.120.camel@zakaz.uk.xensource.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Ian Campbell Cc: Ian Jackson , "xen-devel@lists.xen.org" List-Id: xen-devel@lists.xenproject.org Ian Campbell wrote: > On Tue, 2012-05-29 at 15:40 +0100, Ian Jackson wrote: >> Roger Pau Monne writes ("Re: [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"): >>>>> - 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. >> Sorry. I seem to have failed to reply to this. >> >> I'm a bit confused about all this, I confess. What does xend do ? >> >> ATM it seems that this change would mean that a guest config file >> specified in the most obvious way wouldn't get an emulated nic at >> all. That can't be right, can it ? > > It's confusingly named. > > IOEMU -> emulated only on HVM, meaningless on PV. My understanding was that IOEMU meant both TAP and PV, and was only valid for HVM guests, and VIF meant PV only and was valid for both domain types? I don't think this is right, this will mean VIF has a different meaning depending on the type of domain, which is quite annoying. If this is right, I have to change my hotplug patches, but this will be a mess, because I have no way to tell if a domain is PV or HVM when plugging in the devices, so if VIF is passed for both PV or PV+TAP I will have no way of knowing that. Should we create three different nic types? VIF (PV), IOEMU (TAP), HYBRID (PV+TAP)? > VIF -> both emulated and PV (on HVM) or just PV (on PV) > > So a default of "VIF" is what you normally want. > > Ian. >