netdev
[Top] [All Lists]

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

To: Thomas Graf <tgraf@xxxxxxx>
Subject: Re: [PATCH][RFC] etherip: Ethernet-in-IPv4 tunneling
From: Lennert Buytenhek <buytenh@xxxxxxxxxxxxxx>
Date: Thu, 13 Jan 2005 01:18:37 +0100
Cc: Ben Greear <greearb@xxxxxxxxxxxxxxx>, netdev@xxxxxxxxxxx, shemminger@xxxxxxxx, shollenbeck@xxxxxxxxxxxx
In-reply-to: <20050112234344.GM26856@xxxxxxxxxxxxxx>
References: <20050112222437.GC14280@xxxxxxxxxxxxxxxxx> <41E5A7E9.4030101@xxxxxxxxxxxxxxx> <20050112224810.GE14280@xxxxxxxxxxxxxxxxx> <41E5AEAC.8060706@xxxxxxxxxxxxxxx> <20050112231615.GF14280@xxxxxxxxxxxxxxxxx> <20050112234344.GM26856@xxxxxxxxxxxxxx>
Sender: netdev-bounce@xxxxxxxxxxx
User-agent: Mutt/1.4.1i
On Thu, Jan 13, 2005 at 12:43:44AM +0100, Thomas Graf 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.

What's that?  I can't find it in my kernel tree nor on google.


> 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.

Indeed, it would be good to have this cleaned up.


cheers,
Lennert

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