[dpdk-dev] [PATCH] net/mlx5: fix TCP flag field modification

Subsystems: networking drivers, the rest

3 messages, 3 authors, 2021-02-21 · open the first message on its own page

[dpdk-dev] [PATCH] net/mlx5: fix TCP flag field modification

From: Alexander Kozyrev <hidden>
Date: 2021-02-05 03:03:11

Wrong modification field ID is used for TCP flags: the inner
TCP flag field is changing instead of the outermost one.
Use the proper field ID to match the expectations.

Fixes: 641dbe4fb0 ("net/mlx5: support modify field flow action")
Cc: stable@dpdk.org

Signed-off-by: Alexander Kozyrev <redacted>
---
 drivers/net/mlx5/mlx5_flow_dv.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/mlx5/mlx5_flow_dv.c b/drivers/net/mlx5/mlx5_flow_dv.c
index 76696b7b62..a44291a3c9 100644
--- a/drivers/net/mlx5/mlx5_flow_dv.c
+++ b/drivers/net/mlx5/mlx5_flow_dv.c
@@ -1653,7 +1653,7 @@ mlx5_flow_field_id_to_modify_info
 		break;
 	case RTE_FLOW_FIELD_TCP_FLAGS:
 		info[idx] = (struct field_modify_info){1, 0,
-					MLX5_MODI_IN_TCP_FLAGS};
+					MLX5_MODI_OUT_TCP_FLAGS};
 		if (mask) {
 			mask[idx] = 0x0000003f;
 			if (width < 6)
-- 
2.24.1

Re: [dpdk-dev] [PATCH] net/mlx5: fix TCP flag field modification

From: Slava Ovsiienko <hidden>
Date: 2021-02-05 16:27:20

-----Original Message-----
From: Alexander Kozyrev <redacted>
Sent: Friday, February 5, 2021 5:03
To: dev@dpdk.org
Cc: stable@dpdk.org; Raslan Darawsheh <redacted>; Slava
Ovsiienko [off-list ref]; Matan Azrad [off-list ref];
Ori Kam [off-list ref]
Subject: [PATCH] net/mlx5: fix TCP flag field modification

Wrong modification field ID is used for TCP flags: the inner TCP flag field is
changing instead of the outermost one.
Use the proper field ID to match the expectations.

Fixes: 641dbe4fb0 ("net/mlx5: support modify field flow action")
Cc: stable@dpdk.org

Signed-off-by: Alexander Kozyrev <redacted>
Acked-by: Viacheslav Ovsiienko <redacted>
quoted hunk
---
 drivers/net/mlx5/mlx5_flow_dv.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/mlx5/mlx5_flow_dv.c
b/drivers/net/mlx5/mlx5_flow_dv.c index 76696b7b62..a44291a3c9 100644
--- a/drivers/net/mlx5/mlx5_flow_dv.c
+++ b/drivers/net/mlx5/mlx5_flow_dv.c
@@ -1653,7 +1653,7 @@ mlx5_flow_field_id_to_modify_info
 		break;
 	case RTE_FLOW_FIELD_TCP_FLAGS:
 		info[idx] = (struct field_modify_info){1, 0,
-					MLX5_MODI_IN_TCP_FLAGS};
+					MLX5_MODI_OUT_TCP_FLAGS};
 		if (mask) {
 			mask[idx] = 0x0000003f;
 			if (width < 6)
--
2.24.1

Re: [dpdk-dev] [PATCH] net/mlx5: fix TCP flag field modification

From: Raslan Darawsheh <hidden>
Date: 2021-02-21 09:32:16

Hi,
-----Original Message-----
From: Alexander Kozyrev <redacted>
Sent: Friday, February 5, 2021 5:03 AM
To: dev@dpdk.org
Cc: stable@dpdk.org; Raslan Darawsheh <redacted>; Slava
Ovsiienko [off-list ref]; Matan Azrad [off-list ref];
Ori Kam [off-list ref]
Subject: [PATCH] net/mlx5: fix TCP flag field modification

Wrong modification field ID is used for TCP flags: the inner
TCP flag field is changing instead of the outermost one.
Use the proper field ID to match the expectations.

Fixes: 641dbe4fb0 ("net/mlx5: support modify field flow action")
Cc: stable@dpdk.org

Signed-off-by: Alexander Kozyrev <redacted>
Patch applied to next-net-mlx,

Kindest regards,
Raslan Darawsheh
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help