[libvirt] [Xen-devel] Setting devid for emulated NICs (Xen 4.3.1 / libvirt 1.2.0) using libxl driver
Ian Campbell
Ian.Campbell at citrix.com
Wed Dec 18 12:27:26 UTC 2013
On Tue, 2013-12-17 at 18:32 +0100, Stefan Bader wrote:
> >
> > Might this libxl fix be relevant:
> > commit 5420f26507fc5c9853eb1076401a8658d72669da
> > Author: Jim Fehlig <jfehlig at suse.com>
> > Date: Fri Jan 11 12:22:26 2013 +0000
> >
> > libxl: Set vfb and vkb devid if not done so by the caller
> >
> > Other devices set a sensible devid if the caller has not done so.
> > Do the same for vfb and vkb. While at it, factor out the common code
> > used to determine a sensible devid, so it can be used by other
> > libxl__device_*_add functions.
> >
> > Signed-off-by: Jim Fehlig <jfehlig at suse.com>
> > Acked-by: Ian Campbell <ian.campbell at citrix.com>
> > Committed-by: Ian Campbell <ian.campbell at citrix.com>
> >
> > and a follow up in dfeccbeaa. Although the comment implies that nic's
> > were already correctly assigning a devid if the caller specified -1, so
> > I don't know why it doesn't work for you :-(
>
> Ok, yes, the commit above indeed changes libxl__device_nic_add to call
> libxl__device_nextid for the devid... Just how is this actually called.
> Maybe not sufficient but "git grep libxl__device_nic_add" in the xen code only
> shows the definition and a declaration in libxl_internal.h to me...
I have a feeling a macro might be involved...
Here we go, look for DEFINE_DEVICE_REMOVE in libxl.c. We should really
add the eventual function names in comments to provide grep fodder....
Ian.
More information about the libvir-list
mailing list