Merge branch 'vxlan-cleanups'
authorDavid S. Miller <davem@davemloft.net>
Thu, 18 Feb 2016 04:52:12 +0000 (23:52 -0500)
committerDavid S. Miller <davem@davemloft.net>
Thu, 18 Feb 2016 04:52:12 +0000 (23:52 -0500)
commit5270c4dade091681313141a75f8401c61b72e82c
tree7208c2dd2669bfe11a132652c27ac8406293cbdf
parentd8ef034730350f98c622eb7560dea28b258e9daf
parentb9167b2e775999d267874b5a1a3a79020f54753a
Merge branch 'vxlan-cleanups'

Jiri Benc says:

====================
vxlan: clean up rx path, consolidating extension handling

The rx path of VXLAN turned over time into kind of spaghetti code. The rx
processing is split between vxlan_udp_encap_recv and vxlan_rcv but in an
artificial way: vxlan_rcv is just called at the end of vxlan_udp_encap_recv,
continuing the rx processing where vxlan_udp_encap_recv left it. There's no
clear border between those two functions.

It makes sense to combine those functions into one; this will be actually
needed for VXLAN-GPE where we'll need to skip part of the processing which
is hard to do with the current code.

However, both functions are too long already. This patchset is shortening
them, consolidating extension handling that is spread all around together
and moving it to separate functions. (Later patchsets will do more
consolidation in other parts of the functions with the final goal of merging
vxlan_udp_encap_recv and vxlan_rcv.)

In process of consolidation of the extension handling, I needed to deal with
vni field in a generic way, as its lower 8 bits mean different things for
different extensions. While cleaning up the code to strictly distinguish
between "vni" and "vni field" (which contains vni plus an additional byte),
I also converted the code not to convert endianess back and forth.

The full picture can be seen at:
https://github.com/jbenc/linux-vxlan/commits/master
====================

Signed-off-by: David S. Miller <davem@davemloft.net>