Re: [Xen-devel] [PATCH for-4.5] libxl: don't use a qemu-ifup script on FreeBSD

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Mon, 2014-09-29 at 17:45 +0200, Roger Pau Monnà wrote:
> El 29/09/14 a les 17.37, Ian Campbell ha escrit:
> > On Mon, 2014-09-29 at 16:05 +0200, Roger Pau Monnà wrote:
> >> El 29/09/14 a les 15.35, Ian Campbell ha escrit:
> >>> On Fri, 2014-09-26 at 14:07 -0400, Konrad Rzeszutek Wilk wrote:
> >>>> On Fri, Sep 26, 2014 at 06:14:12PM +0200, Roger Pau Monne wrote:
> >>>>> FreeBSD doesn't use any qemu-ifup script in order to setup the network, 
> >>>>> it
> >>>>> is all done on the hotplug script like Linux.
> >>>>>
> >>>>> Signed-off-by: Roger Pau Monnà <roger.pau@xxxxxxxxxx>
> >>>>> Cc: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
> >>>>> Cc: Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>
> >>>>> Cc: Ian Campbell <ian.campbell@xxxxxxxxxx>
> >>>>> Cc: Wei Liu <wei.liu2@xxxxxxxxxx>
> >>>>> Cc: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
> >>>>> ---
> >>>>> I request a code freeze exception on the basis that this is a bugfix 
> >>>>> and it
> >>>>> doesn't change the behaviour on platforms different than FreeBSD.
> >>>>
> >>>> <nods>
> >>>>
> >>>> Release-Acked-by: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
> >>>
> >>> Applied.
> >>>
> >>> I'm wondering if anyone should be using qemu-ifup these days?
> >>
> >> AFAIR, NetBSD uses it for qemu-trad at least. I was tempted to invert
> >> the ifdef and do something like:
> >>
> >> #ifdef __NetBSD__
> >>    ... use qemu-ifup
> >> #else
> >>    ... no ifup script
> >> #endif
> >>
> >> But I have no idea what Solaris does, or if we still try to support it.
> > 
> > Do we/libxl not call the scripts too in those cases?
> 
> Yes, the vif hotplug script is also called, but on NetBSD it just takes
> care of adding the vif, not the tap interface.
> 
> I think it's done this way on NetBSD because tap interfaces cannot be
> renamed, so only Qemu knows the actual unit/name of the tap interface
> and calls the qemu-ifup script passing that name as the first argument.

OK.



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