On Thu, Nov 10, 2016 at 1:56 AM, Jiri Benc <jb...@redhat.com> wrote: > On Wed, 9 Nov 2016 19:34:06 -0800, Pravin Shelar wrote: >> Why it would not help in non-ovs vxlan egress path? It avoids checking >> (if condition) for device loop. > > I may be missing something but I count the same number of conditions > for each packet, they're just at a different place after the patch. > I am specifically talking about cached routes. If the dst entry is cached, this patch avoids checking for device loop.
> E.g. for IPv4, the "if (!sock4)" is moved from vxlan_xmit_one into > vxlan_get_route and the "rt" error handling stays logically the same > (three if conditions in the non-error path) but is moved into > vxlan_get_route. Similarly for IPv6. > > Jiri