[dpdk-stable] patch 'net/mlx5: fix mask used for IPv6 item validation' has been queued to LTS release 18.11.9

[Kevin Traynor <ktraynor@redhat.com>]

Hi,

FYI, your patch has been queued to LTS release 18.11.9

Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 06/03/20. So please
shout if anyone has objections.

Also note that after the patch there's a diff of the upstream commit vs the
patch applied to the branch. This will indicate if there was any rebasing
needed to apply to the stable branch. If there were code changes for rebasing
(ie: not only metadata diffs), please double check that the rebase was
correctly done.

Queued patches are on a temporary branch at:
https://github.com/kevintraynor/dpdk-stable-queue

This queued commit can be viewed at:
https://github.com/kevintraynor/dpdk-stable-queue/commit/a7e766dd7824d50489820192298700e67150cd93

Thanks.

Kevin.

---
From a7e766dd7824d50489820192298700e67150cd93 Mon Sep 17 00:00:00 2001
From: Dekel Peled <dekelp@mellanox.com>
Date: Tue, 3 Mar 2020 14:37:30 +0200
Subject: [PATCH] net/mlx5: fix mask used for IPv6 item validation

[ upstream commit 6bd5efb23db9e419dec51fbea7f71c2fd3703d75 ]

The nic_mask struct in function mlx5_flow_validate_item_ipv6()
includes hop_limits field.
This is redundant since matching on this item is not supported using
Verbs flow engine (dv_flow_en=0).

(Separate patch will be sent to handle this issue for DV flow engine.)

Setting a rule including match on this field will fail:
testpmd> flow create 0 ingress pattern eth / ipv6 hop is 1 / end actions
drop / end
port_flow_complain(): Caught PMD error type 1 (cause unspecified):
hardware refuses to create flow: Operation not supported

This patch removes the redundant field from nic_mask, to ensure that
such flow rules will be rejected by PMD validation:
testpmd> flow create 0 ingress pattern eth / ipv6 hop is 1 / end actions
drop / end
port_flow_complain(): Caught PMD error type 13 (specific pattern item):
cause: 0x7fffffffa2c8, mask enables non supported bits: Operation not
supported

The related lines in function flow_verbs_translate_item_ipv6() are
removed as well, since they are redundant.

Fixes: 23c1d42c7138 ("net/mlx5: split flow validation to dedicated function")
Fixes: 84c406e74524 ("net/mlx5: add flow translate function")

Signed-off-by: Dekel Peled <dekelp@mellanox.com>
Acked-by: Matan Azrad <matan@mellanox.com>
---
 drivers/net/mlx5/mlx5_flow.c       | 1 -
 drivers/net/mlx5/mlx5_flow_verbs.c | 3 ---
 2 files changed, 4 deletions(-)

diff --git a/drivers/net/mlx5/mlx5_flow.c b/drivers/net/mlx5/mlx5_flow.c
index 7f518fcd26..362fae56c4 100644
--- a/drivers/net/mlx5/mlx5_flow.c
+++ b/drivers/net/mlx5/mlx5_flow.c
@@ -1267,5 +1267,4 @@ mlx5_flow_validate_item_ipv6(const struct rte_flow_item *item,
 			.vtc_flow = RTE_BE32(0xffffffff),
 			.proto = 0xff,
-			.hop_limits = 0xff,
 		},
 	};
diff --git a/drivers/net/mlx5/mlx5_flow_verbs.c b/drivers/net/mlx5/mlx5_flow_verbs.c
index 1a8a9e63ca..05a3db501f 100644
--- a/drivers/net/mlx5/mlx5_flow_verbs.c
+++ b/drivers/net/mlx5/mlx5_flow_verbs.c
@@ -480,5 +480,4 @@ flow_verbs_translate_item_ipv6(struct mlx5_flow *dev_flow,
 					 IPV6_HDR_TC_SHIFT;
 		ipv6.val.next_hdr = spec->hdr.proto;
-		ipv6.val.hop_limit = spec->hdr.hop_limits;
 		ipv6.mask.flow_label =
 			rte_cpu_to_be_32((vtc_flow_mask & IPV6_HDR_FL_MASK) >>
@@ -487,5 +486,4 @@ flow_verbs_translate_item_ipv6(struct mlx5_flow *dev_flow,
 					  IPV6_HDR_TC_SHIFT;
 		ipv6.mask.next_hdr = mask->hdr.proto;
-		ipv6.mask.hop_limit = mask->hdr.hop_limits;
 		/* Remove unwanted bits from values. */
 		for (i = 0; i < RTE_DIM(ipv6.val.src_ip); ++i) {
@@ -496,5 +494,4 @@ flow_verbs_translate_item_ipv6(struct mlx5_flow *dev_flow,
 		ipv6.val.traffic_class &= ipv6.mask.traffic_class;
 		ipv6.val.next_hdr &= ipv6.mask.next_hdr;
-		ipv6.val.hop_limit &= ipv6.mask.hop_limit;
 	}
 	flow_verbs_spec_add(&dev_flow->verbs, &ipv6, size);
-- 
2.21.3

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2020-05-28 17:12:59.897711990 +0100
+++ 0014-net-mlx5-fix-mask-used-for-IPv6-item-validation.patch	2020-05-28 17:12:59.056557419 +0100
@@ -1 +1 @@
-From 6bd5efb23db9e419dec51fbea7f71c2fd3703d75 Mon Sep 17 00:00:00 2001
+From a7e766dd7824d50489820192298700e67150cd93 Mon Sep 17 00:00:00 2001
@@ -5,0 +6,2 @@
+[ upstream commit 6bd5efb23db9e419dec51fbea7f71c2fd3703d75 ]
+
@@ -32 +33,0 @@
-Cc: stable@dpdk.org
@@ -42 +43 @@
-index 16c1ef3532..41072da6df 100644
+index 7f518fcd26..362fae56c4 100644
@@ -45 +46 @@
-@@ -1641,5 +1641,4 @@ mlx5_flow_validate_item_ipv6(const struct rte_flow_item *item,
+@@ -1267,5 +1267,4 @@ mlx5_flow_validate_item_ipv6(const struct rte_flow_item *item,
@@ -52 +53 @@
-index 04845f8f4c..459e7b6483 100644
+index 1a8a9e63ca..05a3db501f 100644
@@ -55,2 +56,2 @@
-@@ -495,5 +495,4 @@ flow_verbs_translate_item_ipv6(struct mlx5_flow *dev_flow,
- 					 RTE_IPV6_HDR_TC_SHIFT;
+@@ -480,5 +480,4 @@ flow_verbs_translate_item_ipv6(struct mlx5_flow *dev_flow,
+ 					 IPV6_HDR_TC_SHIFT;
@@ -60,3 +61,3 @@
- 			rte_cpu_to_be_32((vtc_flow_mask & RTE_IPV6_HDR_FL_MASK) >>
-@@ -502,5 +501,4 @@ flow_verbs_translate_item_ipv6(struct mlx5_flow *dev_flow,
- 					  RTE_IPV6_HDR_TC_SHIFT;
+ 			rte_cpu_to_be_32((vtc_flow_mask & IPV6_HDR_FL_MASK) >>
+@@ -487,5 +486,4 @@ flow_verbs_translate_item_ipv6(struct mlx5_flow *dev_flow,
+ 					  IPV6_HDR_TC_SHIFT;
@@ -67 +68 @@
-@@ -511,5 +509,4 @@ flow_verbs_translate_item_ipv6(struct mlx5_flow *dev_flow,
+@@ -496,5 +494,4 @@ flow_verbs_translate_item_ipv6(struct mlx5_flow *dev_flow,