]> git.droids-corp.org - dpdk.git/commit
app/testpmd: fix raw encap of GENEVE option
authorBing Zhao <bingz@nvidia.com>
Thu, 24 Feb 2022 07:02:14 +0000 (09:02 +0200)
committerFerruh Yigit <ferruh.yigit@intel.com>
Fri, 25 Feb 2022 14:48:07 +0000 (15:48 +0100)
commitd9e597141fd3a69278c31b1daf6da7ce54de3b8e
treeb8b9fcf670912f1ed9e77f40ebe9dd7ef019d6dd
parentdf5807658026e7b18493a6c8ebf0e47adee26b4d
app/testpmd: fix raw encap of GENEVE option

The structure "rte_flow_item_geneve_opt" is not a protocol header of
geneve tunnel option from rfc8926. The field "data" is a pointer
which points to the actual variable-length option data. So the
structure is not packed.

There is 4 bytes hole before the pointer in a 64-bit system. The
option header is just 4 bytes. When using offsetof() to get the
fixed part's size of option header, the wrong value 8 was got. When
constructing the encap header, a wrong size and offset was used due
to this hole.

With this commit, the fixed part's size is calculated explicitly
based on all fields.

Fixes: 55c074f3ba1d ("app/testpmd: support GENEVE option item")
Cc: stable@dpdk.org
Signed-off-by: Bing Zhao <bingz@nvidia.com>
Reviewed-by: Viacheslav Ovsiienko <viacheslavo@nvidia.com>
app/test-pmd/cmdline_flow.c