[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <49764890.5010403@katalix.com>
Date: Tue, 20 Jan 2009 21:56:32 +0000
From: James Chapman <jchapman@...alix.com>
To: Cyrill Gorcunov <gorcunov@...il.com>
CC: davem@...emloft.net, linux-kernel@...r.kernel.org,
netdev@...r.kernel.org, devel@...nvz.org, xemul@...nvz.org,
Michal Ostrowski <mostrows@...thlink.net>,
Cyrill Gorcunov <gorcunov@...nvz.org>
Subject: Re: [PATCH 5/5] net: pppoe,pppol2tp - register channels with explicit
net
Cyrill Gorcunov wrote:
> In PPPo[E|L2TP] we could explicitly point which net namespace
> we're going to use for channels - make it so.
>
> CC: James Chapman <jchapman@...alix.com>
> CC: Michal Ostrowski <mostrows@...thlink.net>
> Signed-off-by: Cyrill Gorcunov <gorcunov@...nvz.org>
This one's ok by me too for pppol2tp.
Signed-off-by: James Chapman <jchapman@...alix.com>
> ---
> drivers/net/pppoe.c | 2 +-
> drivers/net/pppol2tp.c | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> Index: linux-2.6.git/drivers/net/pppoe.c
> ===================================================================
> --- linux-2.6.git.orig/drivers/net/pppoe.c
> +++ linux-2.6.git/drivers/net/pppoe.c
> @@ -695,7 +695,7 @@ static int pppoe_connect(struct socket *
> po->chan.private = sk;
> po->chan.ops = &pppoe_chan_ops;
>
> - error = ppp_register_channel(&po->chan);
> + error = ppp_register_net_channel(dev_net(dev), &po->chan);
> if (error)
> goto err_put;
>
> Index: linux-2.6.git/drivers/net/pppol2tp.c
> ===================================================================
> --- linux-2.6.git.orig/drivers/net/pppol2tp.c
> +++ linux-2.6.git/drivers/net/pppol2tp.c
> @@ -1749,7 +1749,7 @@ static int pppol2tp_connect(struct socke
> po->chan.ops = &pppol2tp_chan_ops;
> po->chan.mtu = session->mtu;
>
> - error = ppp_register_channel(&po->chan);
> + error = ppp_register_net_channel(sock_net(sk), &po->chan);
> if (error)
> goto end_put_tun;
>
>
--
James Chapman
Katalix Systems Ltd
http://www.katalix.com
Catalysts for your Embedded Linux software development
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists