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

Re: [Xen-devel] [PATCH RFC 14/23] hotplug: add FreeBSD vif-bridge



On Mon, 2014-06-02 at 16:46 +0200, Roger Pau Monnà wrote:
> On 02/06/14 15:52, Ian Campbell wrote:
> > On Mon, 2014-06-02 at 13:56 +0200, Roger Pau Monnà wrote:
> >> On 28/04/14 16:39, Ian Campbell wrote:
> >>> On Wed, 2014-04-16 at 16:13 +0200, Roger Pau Monne wrote:
> >>>> Add a simple vif-bridge script, that takes care of adding network
> >>>> backends (tap or xnb) to a pre-configured bridge.
> >>>
> >>> Compared with the Linux one it lacks the handling of the vifname
> >>> property and the attempt to provide a default if no bridge was
> >>> specified. This is probably acceptable I suppose.
> >>
> >> I would like to explicitly avoid providing a default bridge if no bridge
> >> was specified, I think this should probably be done in the global xl
> >> configuration file.
> > 
> > Is that what NetBSD does? I know Linux provides one, so FreeBSD should
> > either be the same as NetBSD or Linux (so if NetBSD provides a default
> > so should FreeBSD).
> 
> NetBSD doesn't seem to provide a default bridge, here is what NetBSD
> does in vif-bridge:
> 
> xbridge=$(xenstore-read "$xpath/bridge")
> [...]
> brconfig $xbridge add $iface

OK, then I'm fine with FreeBSD not having a default either.

> 
> >>> OOI why the x prefixes everywhere?
> >>
> >> Because they were there on the NetBSD version, but I will remove them in
> >> the next version of the series.
> > 
> > Feel free to leave them if you would rather not diverge (with the aim of
> > eventually merging). Up to you though.
> 
> I've removed the "x" prefixes, since I think they are not helpful. Later
> on we can remove them from the NetBSD script if there's interest in
> merging them.

Sure.

Ian.



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

 


Rackspace

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