enum rte_flow_item_type states that items matching protocol headers must be stacked in the same order as the protocol layers to match. As the result the patch changes ABI since Geneve is added just after VXLAN (the closest protocol).
In fact as far as I can see many items do not follow the requirement already. May be the comment/requirement should be removed and GENEVE should be added at the end of the list. If so, should be keep it just after VXLAN in all other places or move after ESP as well? Roman Zhukov (2): ethdev: add GENEVE flow pattern item app/testpmd: support GENEVE pattern item in flow rules app/test-pmd/cmdline_flow.c | 31 +++++++++++++++++++++++++++++ app/test-pmd/config.c | 1 + doc/guides/prog_guide/rte_flow.rst | 12 +++++++++++ doc/guides/testpmd_app_ug/testpmd_funcs.rst | 5 +++++ lib/librte_ether/rte_flow.c | 1 + lib/librte_ether/rte_flow.h | 31 +++++++++++++++++++++++++++++ 6 files changed, 81 insertions(+) -- 2.7.4
