On Fri, Sep 27, 2013 at 12:56 AM, Steffen Klassert
[off-list ref] wrote:

On Thu, Sep 26, 2013 at 11:24:07AM -0700, Pravin Shelar wrote:

quoted

On Thu, Sep 26, 2013 at 1:13 AM, Steffen Klassert
[off-list ref] wrote:

quoted

On Wed, Sep 25, 2013 at 09:03:11AM -0700, Pravin Shelar wrote:

quoted

fallback tunnel s not required to be in hash table, Its is returned if
none of hashed tunnels are matched, ref ip_tunnel_lookup().
Can you post command to reproduce this issue?

Something like

ip tunnel change tunl0 mode ipip remote 0.0.0.0 local 0.0.0.0 ttl 0 tos 1

worked until v3.9 and stopped working with v3.10.

OK, I see the bug, tunnel exact match lookup does not check fb tunnel.
There are two options.
1. Fix ip_tunnel_find() to check for fb tunnel.
2. Add fb tunnel to hash table, which is what ur patch does.
I think your patch is better solution as it get rid of special case.
But patch is not complete. It needs to remove fb tunnel checks on
netdev unregister.

It looks like this is another bug that requires an additional patch.
We add the fallback tunnel to the unregister list when we iterate over
all netdevices in the namespace at the beginning of ip_tunnel_destroy()
and then again explicitly at the end of ip_tunnel_destroy().

right.
And if the special case is removed, code will become simple.