Re: [Xen-devel] [PATCH] libxl: Auto-assign NIC devids in initiate_domain_create

[Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>]

Ian Campbell writes ("Re: [PATCH] libxl: Auto-assign NIC devids in 
initiate_domain_create"):
> On Thu, 2014-01-09 at 11:33 +0100, Stefan Bader wrote:
> > From bafc8f62ee3e3175ec4d978bceba4b5f891a597d Mon Sep 17 00:00:00 2001
> > From: Stefan Bader <stefan.bader@xxxxxxxxxxxxx>
> > Date: Wed, 8 Jan 2014 18:26:59 +0100
> > Subject: [PATCH] libxl: Auto-assign NIC devids in initiate_domain_create
> > 
> > This will change initiate_domain_create to walk through NIC definitions
> > and automatically assign devids to those which have not assigned one.
> > The devids are needed later in domcreate_launch_dm (for HVM domains
> > using emulated NICs). The command string for starting the device-model
> > has those ids as part of its arguments.
> > Assignment of devids in the hotplug case is handled by libxl_device_nic_add
> > but that would be called too late in the startup case.
> > I also moved the call to libxl__device_nic_setdefault here as this seems
> > to be the only path leading there and avoids doing the loop a third time.
> > The two loops are trying to handle a case where the caller sets some devids
> > (not sure that should be valid) but leaves some unset.

Thanks.  Thanks also for the careful and comprehensive explanation.

> > Signed-off-by: Stefan Bader <stefan.bader@xxxxxxxxxxxxx>
> 
> Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>

Acked-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>

> I think from a release point of view we should take this since it is a
> bug fix to the API which at least libvirt has tripped over (although
> libvirt has worked around it, others may not have done so).
 
> Ian J: Does that make sense?

I agree.

Thanks,
Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel