Re: [INET]: Introduce tunnel4/tunnel6

2006-03-28 Thread Herbert Xu
On Tue, Mar 28, 2006 at 01:13:11AM -0800, David S. Miller wrote: > > Ok, this seems reasonable, applied. Thanks. > BTW, can you sanity check your patches with something like: > > git apply --check --whitespace=error-all diff > > This patch here had a bunch of trailing whitespace I had > t

Re: [INET]: Introduce tunnel4/tunnel6

2006-03-28 Thread David S. Miller
From: Herbert Xu <[EMAIL PROTECTED]> Date: Mon, 27 Mar 2006 21:17:59 +1100 > Basically this patch moves the generic tunnel protocol stuff out of > xfrm4_tunnel/xfrm6_tunnel and moves it into the new files of tunnel4.c > and tunnel6 respectively. Ok, this seems reasonable, applied. BTW, can you s

Re: [INET]: Introduce tunnel4/tunnel6

2006-03-27 Thread Herbert Xu
On Mon, Mar 27, 2006 at 12:01:41PM +0100, Hugo Santos wrote: > >One small issue with the patch is tunnel6_rcv's return of 0 when no > handler handles the packet which will increase IPSTATS_MIB_INDELIVERS. > I guess -1 should be returned on error. I was checking ip_input and the > expected b

Re: [INET]: Introduce tunnel4/tunnel6

2006-03-27 Thread Hugo Santos
Hi Herbert, I have a couple of comments, please see below. > The reason for this is that the problem that Hugo uncovered is only > the tip of the iceberg. The real problem is that when we removed the > dependency of ipip on xfrm4_tunnel we didn't really consider the module > case at all. > >

[INET]: Introduce tunnel4/tunnel6

2006-03-27 Thread Herbert Xu
Hi Dave: I've finally finished this stuff. Basically this patch moves the generic tunnel protocol stuff out of xfrm4_tunnel/xfrm6_tunnel and moves it into the new files of tunnel4.c and tunnel6 respectively. The reason for this is that the problem that Hugo uncovered is only the tip of the icebe