Re: [Xen-devel] [Qemu-devel] [PATCH v2 2/3] tap-bsd: implement a FreeBSD only version of tap_open

[Stefan Hajnoczi <stefanha@xxxxxxxxx>]

On Tue, Jul 22, 2014 at 01:26:00PM +0100, Stefano Stabellini wrote:
> On Tue, 22 Jul 2014, Roger Pau Monné wrote:
> > On 27/05/14 15:29, Stefan Hajnoczi wrote:
> > > On Fri, May 23, 2014 at 05:57:48PM +0200, Roger Pau Monne wrote:
> > >> The current behaviour of tap_open for BSD systems differ greatly from
> > >> it's Linux counterpart. Since FreeBSD supports interface renaming and
> > >> tap device cloning by opening /dev/tap, implement a FreeBSD specific
> > >> version of tap_open that behaves like it's Linux counterpart.
> > >>
> > >> This is specially important for toolstacks that use Qemu (like Xen
> > >> libxl), in order to have a unified behaviour across suported
> > >> platforms.
> > >>
> > >> Signed-off-by: Roger Pau Monné <roger.pau@xxxxxxxxxx>
> > >> Cc: xen-devel@xxxxxxxxxxxxxxxxxxxx
> > >> Cc: Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>
> > >> Cc: Anthony Liguori <aliguori@xxxxxxxxxx>
> > >> Cc: Stefan Hajnoczi <stefanha@xxxxxxxxxx>
> > >> ---
> > >>  net/tap-bsd.c |   70 
> > >> ++++++++++++++++++++++++++++++++++++++++++++++++++++++++-
> > >>  1 files changed, 69 insertions(+), 1 deletions(-)
> > > 
> > > Reviewed-by: Stefan Hajnoczi <stefanha@xxxxxxxxxx>
> > 
> > I still don't see this committed to the repository, should I ping someone?
> 
> I was assuming that this patch would go via some other tree.  But if
> Stefan is OK I could pick it up and submit a pull request for both patch
> 1 and 2 of this series.

I'm fine with that.  I only reviewed this patch since it affects the net
subsystem and left the series as a whole for someone to merge.

Stefan

Attachment: pgpgLuHpTxm6N.pgp
Description: PGP signature

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