From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id D23D7A0032 for ; Wed, 16 Mar 2022 16:16:10 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id CD969410EC; Wed, 16 Mar 2022 16:16:10 +0100 (CET) Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by mails.dpdk.org (Postfix) with ESMTP id 32A7141148 for ; Wed, 16 Mar 2022 16:16:09 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1647443768; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=g1Y8NuKaUTrZxREVF/xc/+QJIA+TW6T2FONyJSuYPM4=; b=HFLZRFqMGcp4niQoMNjAhWYFYcZY0Q+CY+tNClosJpuAd3/KZtkPCPUqZmF0uylLZw7tz6 GZYYhgx5eCavddjevQ5B6TaXABu2sLD2de9Hbonjv19a12Kp3fc8mdFjykNvEGfJ6bdASq zyNP6Yp82ozN92RZQbNgz08wOPA+ZNs= Received: from mimecast-mx02.redhat.com (mimecast-mx02.redhat.com [66.187.233.88]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-391-xM6rsgKTMaarVlQ5jcudkw-1; Wed, 16 Mar 2022 11:16:05 -0400 X-MC-Unique: xM6rsgKTMaarVlQ5jcudkw-1 Received: from smtp.corp.redhat.com (int-mx10.intmail.prod.int.rdu2.redhat.com [10.11.54.10]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id BFF6F1044564; Wed, 16 Mar 2022 15:16:04 +0000 (UTC) Received: from rh.redhat.com (unknown [10.39.195.131]) by smtp.corp.redhat.com (Postfix) with ESMTP id B68C8401053; Wed, 16 Mar 2022 15:16:03 +0000 (UTC) From: Kevin Traynor To: Jiawei Wang Cc: Viacheslav Ovsiienko , dpdk stable Subject: patch 'net/mlx5: fix port matching in sample flow rule' has been queued to stable release 21.11.1 Date: Wed, 16 Mar 2022 15:15:23 +0000 Message-Id: <20220316151524.1242199-22-ktraynor@redhat.com> In-Reply-To: <20220316151524.1242199-1-ktraynor@redhat.com> References: <20220316151524.1242199-1-ktraynor@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.85 on 10.11.54.10 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=ktraynor@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset="US-ASCII" X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Hi, FYI, your patch has been queued to stable release 21.11.1 Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet. It will be pushed if I get no objections before 03/21/22. 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 This queued commit can be viewed at: https://github.com/kevintraynor/dpdk-stable/commit/22ed8b668425bd042aa095bdffa307940fd6acc2 Thanks. Kevin --- >From 22ed8b668425bd042aa095bdffa307940fd6acc2 Mon Sep 17 00:00:00 2001 From: Jiawei Wang Date: Mon, 7 Mar 2022 04:10:42 +0200 Subject: [PATCH] net/mlx5: fix port matching in sample flow rule [ upstream commit 04c0d3f20f54618c2b62f353b31b5a2fb20d687a ] If there are an explicit port match and sample action in the same flow, mlx5 PMD pushes the explicit port match in the prefix subflow, and uses the tag item match in the suffix subflow. The explicit port match was translated into source vport match so the sample suffix subflow lost this match after flow split. This patch copies the explicit port match to the sample suffix subflow, and the latter gets the correct source vport value in the flow matcher. Fixes: b4c0ddbfcc58 ("net/mlx5: split sample flow into two sub-flows") Signed-off-by: Jiawei Wang Acked-by: Viacheslav Ovsiienko --- drivers/net/mlx5/mlx5_flow.c | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/drivers/net/mlx5/mlx5_flow.c b/drivers/net/mlx5/mlx5_flow.c index 10afd96fea..cf4646e187 100644 --- a/drivers/net/mlx5/mlx5_flow.c +++ b/drivers/net/mlx5/mlx5_flow.c @@ -5555,5 +5555,5 @@ flow_check_match_action(const struct rte_flow_action actions[], } -#define SAMPLE_SUFFIX_ITEM 2 +#define SAMPLE_SUFFIX_ITEM 3 /** @@ -5596,4 +5596,5 @@ static int flow_sample_split_prep(struct rte_eth_dev *dev, int add_tag, + const struct rte_flow_item items[], struct rte_flow_item sfx_items[], const struct rte_flow_action actions[], @@ -5723,4 +5724,10 @@ flow_sample_split_prep(struct rte_eth_dev *dev, }; /* Prepare the suffix subflow items. */ + for (; items->type != RTE_FLOW_ITEM_TYPE_END; items++) { + if (items->type == RTE_FLOW_ITEM_TYPE_PORT_ID) { + memcpy(sfx_items, items, sizeof(*sfx_items)); + sfx_items++; + } + } tag_spec = (void *)(sfx_items + SAMPLE_SUFFIX_ITEM); tag_spec->data = tag_id; @@ -6336,5 +6343,5 @@ flow_create_split_sample(struct rte_eth_dev *dev, next_ft_step; pre_actions = sfx_actions + actions_n; - tag_id = flow_sample_split_prep(dev, add_tag, sfx_items, + tag_id = flow_sample_split_prep(dev, add_tag, items, sfx_items, actions, sfx_actions, pre_actions, actions_n, -- 2.34.1 --- Diff of the applied patch vs upstream commit (please double-check if non-empty: --- --- - 2022-03-16 15:14:12.633334075 +0000 +++ 0022-net-mlx5-fix-port-matching-in-sample-flow-rule.patch 2022-03-16 15:14:12.134847689 +0000 @@ -1 +1 @@ -From 04c0d3f20f54618c2b62f353b31b5a2fb20d687a Mon Sep 17 00:00:00 2001 +From 22ed8b668425bd042aa095bdffa307940fd6acc2 Mon Sep 17 00:00:00 2001 @@ -5,0 +6,2 @@ +[ upstream commit 04c0d3f20f54618c2b62f353b31b5a2fb20d687a ] + @@ -17 +18,0 @@ -Cc: stable@dpdk.org @@ -26 +27 @@ -index cd55bdc2c0..78cb38d42b 100644 +index 10afd96fea..cf4646e187 100644 @@ -29 +30 @@ -@@ -5784,5 +5784,5 @@ flow_check_match_action(const struct rte_flow_action actions[], +@@ -5555,5 +5555,5 @@ flow_check_match_action(const struct rte_flow_action actions[], @@ -36 +37 @@ -@@ -5825,4 +5825,5 @@ static int +@@ -5596,4 +5596,5 @@ static int @@ -42 +43 @@ -@@ -5952,4 +5953,10 @@ flow_sample_split_prep(struct rte_eth_dev *dev, +@@ -5723,4 +5724,10 @@ flow_sample_split_prep(struct rte_eth_dev *dev, @@ -53 +54 @@ -@@ -6564,5 +6571,5 @@ flow_create_split_sample(struct rte_eth_dev *dev, +@@ -6336,5 +6343,5 @@ flow_create_split_sample(struct rte_eth_dev *dev,