On Fri, 23 Oct 2015 12:47:09 -0700, Pravin Shelar wrote: > This is not complete code. I found couple of issues with it.
That's likely, I didn't test it :-) > This code does not copy lwtunnel_state state into new dst. Not sure what state you mean. All the relevant state should be in struct ip_tunnel_info. > And it is converting lwtunnel dst into metadata dst. I did that intentionally. But your comment made me realize that I was too much focused on the tunnel implementation side. This function, should it be used outside of ovs, could be called earlier than the skb is handed to the tunnel. Then we of course need a valid dst->output and this approach (converting lwtunnel data to metadata_dst) would fail. This is the same problem we have with IPv6 ndisc replies. This needs more thinking and is out of scope of what you're trying to solve. Sorry for the noise. But at least it made me realize that the ndisc problem (that I still don't have a good solution for) is broader. Thanks, Jiri -- Jiri Benc -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
