[PATCH v2 03/26] net/hinic3: fix VXLAN VNI spec and mask write

[Jiacheng Ye]

The VXLAN parser writes the flow item spec into key_mask and never
populates the mask, so VNI matches against a wrong value. Store the spec
into key_spec and the mask into key_mask.

Fixes: efa3b9b36fc7 ("net/hinic3: add flow control and filters")
Cc: sta...@dpdk.org

Signed-off-by: Jiacheng Ye <yejiache...@h-partners.com>
---
 drivers/net/hinic3/hinic3_flow.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/drivers/net/hinic3/hinic3_flow.c b/drivers/net/hinic3/hinic3_flow.c
index 453b4369b3..1f686d4675 100644
--- a/drivers/net/hinic3/hinic3_flow.c
+++ b/drivers/net/hinic3/hinic3_flow.c
@@ -967,6 +967,7 @@ hinic3_flow_fdir_vxlan(struct rte_flow_error *error,
 {
        const struct rte_flow_item_vxlan *spec_vxlan, *mask_vxlan;
        uint32_t vxlan_vni_id = 0;
+       uint32_t vxlan_vni_id_mask = 0;
 
        spec_vxlan = (const struct rte_flow_item_vxlan *)flow_item->spec;
        mask_vxlan = (const struct rte_flow_item_vxlan *)flow_item->mask;
@@ -990,8 +991,11 @@ hinic3_flow_fdir_vxlan(struct rte_flow_error *error,
        }
 
        memcpy(((uint8_t *)&vxlan_vni_id + 1), spec_vxlan->vni, 3);
-       filter->fdir_filter.key_mask.tunnel.tunnel_id =
+       filter->fdir_filter.key_spec.tunnel.tunnel_id =
                rte_be_to_cpu_32(vxlan_vni_id);
+       memcpy(((uint8_t *)&vxlan_vni_id_mask + 1), mask_vxlan->vni, 3);
+       filter->fdir_filter.key_mask.tunnel.tunnel_id =
+               rte_be_to_cpu_32(vxlan_vni_id_mask);
        return 0;
 }
 
-- 
2.33.0