diff options
| author |   Alexey Kodanev <alexey.kodanev@oracle.com> | 2015-03-13 19:13:53 +0300 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2015-03-13 13:08:07 -0400 |
| commit | 40fb70f3aa0a67d28a30c854d4e7aa10b0511db9 (patch) | |
| tree | a6aa7450acf5919c7ff1eabca584881e70819672 /include/net | |
| parent | tulip_core.c : out-of-bounds check. (diff) | |
| download | linux-dev-40fb70f3aa0a67d28a30c854d4e7aa10b0511db9.tar.xz linux-dev-40fb70f3aa0a67d28a30c854d4e7aa10b0511db9.zip | |
vxlan: fix wrong usage of VXLAN_VID_MASK
commit dfd8645ea1bd9127 wrongly assumes that VXLAN_VDI_MASK includes
eight lower order reserved bits of VNI field that are using for remote
checksum offload.
Right now, when VNI number greater then 0xffff, vxlan_udp_encap_recv()
will always return with 'bad_flag' error, reducing the usable vni range
from 0..16777215 to 0..65535. Also, it doesn't really check whether RCO
bits processed or not.
Fix it by adding new VNI mask which has all 32 bits of VNI field:
24 bits for id and 8 bits for other usage.
Signed-off-by: Alexey Kodanev <alexey.kodanev@oracle.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/net')
| -rw-r--r-- | include/net/vxlan.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/include/net/vxlan.h b/include/net/vxlan.h index eabd3a038674..c73e7abbbaa5 100644 --- a/include/net/vxlan.h +++ b/include/net/vxlan.h @@ -91,6 +91,7 @@ struct vxlanhdr { #define VXLAN_N_VID (1u << 24) #define VXLAN_VID_MASK (VXLAN_N_VID - 1) +#define VXLAN_VNI_MASK (VXLAN_VID_MASK << 8) #define VXLAN_HLEN (sizeof(struct udphdr) + sizeof(struct vxlanhdr)) struct vxlan_metadata { |