netdev
[Top] [All Lists]

Re: [PATCH][RFC] etherip: Ethernet-in-IPv4 tunneling

To: Lennert Buytenhek <buytenh@xxxxxxxxxxxxxx>
Subject: Re: [PATCH][RFC] etherip: Ethernet-in-IPv4 tunneling
From: Thomas Graf <tgraf@xxxxxxx>
Date: Thu, 13 Jan 2005 00:43:44 +0100
Cc: Ben Greear <greearb@xxxxxxxxxxxxxxx>, netdev@xxxxxxxxxxx, shemminger@xxxxxxxx, shollenbeck@xxxxxxxxxxxx
In-reply-to: <20050112231615.GF14280@xi.wantstofly.org>
References: <20050112222437.GC14280@xi.wantstofly.org> <41E5A7E9.4030101@candelatech.com> <20050112224810.GE14280@xi.wantstofly.org> <41E5AEAC.8060706@candelatech.com> <20050112231615.GF14280@xi.wantstofly.org>
Sender: netdev-bounce@xxxxxxxxxxx
* Lennert Buytenhek <20050112231615.GF14280@xxxxxxxxxxxxxxxxx> 2005-01-13 00:16
> On Wed, Jan 12, 2005 at 03:11:40PM -0800, Ben Greear wrote:
> 
> > >>Also, could you add an ioctl that allowed one to query whether or not
> > >>a particular device is an etherip device?  I had always wished I had added
> > >>this earlier to the VLAN code :)
> > >
> > >Hmmm.  Bridge devices don't have this either, do they?  Can you name
> > >an advantage of having this?
> > 
> > I got the request several times with regard to VLANs.  Lots of people
> > (and applications) will want to know the interface type for various
> > reasons.  If you don't give them a nice programatic thing like an
> > IOCTL to call, they will undoubtedly start making assumptions based
> > off of the device name...
> 
> Makes sense..
> 
> Unfortunately SIOCGETTUNNEL is (SIOCDEVPRIVATE + 3), otherwise we could
> just say something like "If an ARPHRD_ETHER device supports SIOCGETTUNNEL,
> it's an ether/ip tunnel."
> 
> Any better ideas?  I hate adding more ioctls.

I think it should go into ip_tunnel_param, unforunately there are no
unused fields. Maybe schedule this for 2.7 together with a clean up
of all the tunnels so they share redundant code? Lots of gre related
code and comments spread over non-gre tunnels that should go away.

<Prev in Thread] Current Thread [Next in Thread>