[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAFaHj6HxV=YgZiUp-WkuGVJBa2nv-g86yUaa_9XxnhqTex=zOg@mail.gmail.com>
Date: Thu, 25 Oct 2012 11:12:58 -0400
From: rsa <ravi.mlists@...il.com>
To: Benjamin LaHaise <bcrl@...ck.org>
Cc: "Eric W. Biederman" <ebiederm@...ssion.com>, netdev@...r.kernel.org
Subject: Re: switching network namespace midway
Thanks Eric. That makes sense. I can remember both network namespaces
and switch to the other prior to route lookup if necessary.
Any gotchas on receiving tunneled packed on the tunnel interface? Looking at
the code there is no change necessary. Packet arrives on tunnel
interface which is
decapsulated and given to net_if_rx() for inner packet processing. Am I mssing
something?
thank you
rsa
On Wed, Oct 24, 2012 at 5:21 PM, Benjamin LaHaise <bcrl@...ck.org> wrote:
> On Wed, Oct 24, 2012 at 02:11:14PM -0700, Eric W. Biederman wrote:
>> rsa <ravi.mlists@...il.com> writes:
>>
>> > Assuming I have a tunnel interface where two route lookups are done --
>> > one for innter
>> > packet and the other for outer -- do you see any issues in switching
>> > the network
>> > namespace prior to second route lookup (and restore to the original namespace
>> > after the second lookup is done)?
>> >
>> > If so, are there any other calls other than sk_change_net() needed?
>>
>> In general sk_change_net is a bad idea.
>>
>> Most likely what you want to do is simply memorize both struct net's
>> that you care about and perform the routing lookup as appropriate.
>>
>> Certainly you don't want to be calling sk_change_net for every packet
>> that goes through your tunnel.
>
> I've actually done this with L2TP. The packets coming into the system from
> the tunnel are received on one UDP socket in one "struct net", but the
> decapsulated packets are received on a "struct net_device" that is in a
> different "struct net". No special coding is required -- just move the
> tunnel's net_device into another namespace after creation and it works as
> expected. Using sk_change_net() would be full of races and is really not
> required for the vast majority of use cases.
>
> -ben
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists