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 B96A546468 for ; Mon, 24 Mar 2025 17:18:49 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id B324640DD3; Mon, 24 Mar 2025 17:18:49 +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 B69E040BA6 for ; Mon, 24 Mar 2025 17:18:48 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1742833128; 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=zViu2YXIENdOaVagO9b+vs1OKPKPWk0LGluf5IroJPw=; b=bMTD/18rCOFpUBh47wQwi5N60ngrL3U87al8tFRVqRkgvWyQUGgPt1FOYJvcNO/wZbm6Pf PP1hgOutTLUfbssz4Qa9i4Gz22R7GkdVhabwFjmYpSK3Gievr3O5Oyq/tuca/3Z++TTQjU fPb3JqGoTiCHS+L/+bN1UYipAP+zQOA= Received: from mx-prod-mc-08.mail-002.prod.us-west-2.aws.redhat.com (ec2-35-165-154-97.us-west-2.compute.amazonaws.com [35.165.154.97]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-483-b8kSTK6JPw6BAaXYTP0D2g-1; Mon, 24 Mar 2025 12:18:45 -0400 X-MC-Unique: b8kSTK6JPw6BAaXYTP0D2g-1 X-Mimecast-MFC-AGG-ID: b8kSTK6JPw6BAaXYTP0D2g_1742833125 Received: from mx-prod-int-01.mail-002.prod.us-west-2.aws.redhat.com (mx-prod-int-01.mail-002.prod.us-west-2.aws.redhat.com [10.30.177.4]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by mx-prod-mc-08.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTPS id 095681801A12; Mon, 24 Mar 2025 16:18:45 +0000 (UTC) Received: from rh.redhat.com (unknown [10.44.32.16]) by mx-prod-int-01.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTP id B90B130001A1; Mon, 24 Mar 2025 16:18:43 +0000 (UTC) From: Kevin Traynor To: Shani Peretz Cc: dpdk stable Subject: patch 'examples/flow_filtering: fix IPv4 matching snippet' has been queued to stable release 24.11.2 Date: Mon, 24 Mar 2025 16:16:35 +0000 Message-ID: <20250324161731.63950-28-ktraynor@redhat.com> In-Reply-To: <20250324161731.63950-1-ktraynor@redhat.com> References: <20250324161731.63950-1-ktraynor@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.4.1 on 10.30.177.4 X-Mimecast-Spam-Score: 0 X-Mimecast-MFC-PROC-ID: hntAuxISDeuXYP6BLsXoWFeGnwgK0lTALQDqlULAmk4_1742833125 X-Mimecast-Originator: redhat.com Content-Transfer-Encoding: 8bit content-type: text/plain; charset="US-ASCII"; x-default=true 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 24.11.2 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/28/25. 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/b540e51a864d25b1e254fa8c311b92223e0f4162 Thanks. Kevin --- >From b540e51a864d25b1e254fa8c311b92223e0f4162 Mon Sep 17 00:00:00 2001 From: Shani Peretz Date: Sun, 12 Jan 2025 09:52:00 +0200 Subject: [PATCH] examples/flow_filtering: fix IPv4 matching snippet [ upstream commit 5fa25916f1ab47c82310264449a6d67dbb15ca4d ] This patch corrects the destination IP address mask to restore the previous implementation's behavior. Also it fixes a misuse of rte_flow_item_tcp struct. Replace it with the appropriate rte_flow_item_ipv4 struct, as the code in this context filters ipv4 traffic. Fixes: 16158f349000 ("examples/flow_filtering: introduce use cases snippets") Signed-off-by: Shani Peretz --- examples/flow_filtering/snippets/snippet_match_ipv4.c | 11 ++++++----- examples/flow_filtering/snippets/snippet_match_ipv4.h | 1 - 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/examples/flow_filtering/snippets/snippet_match_ipv4.c b/examples/flow_filtering/snippets/snippet_match_ipv4.c index 65fb045e8b..d32f4ebfdc 100644 --- a/examples/flow_filtering/snippets/snippet_match_ipv4.c +++ b/examples/flow_filtering/snippets/snippet_match_ipv4.c @@ -56,5 +56,5 @@ snippet_ipv4_flow_create_patterns(struct rte_flow_item *patterns) ip_spec->hdr.dst_addr = htonl(DEST_IP); /* The dest ip value to match the input packet. */ - ip_mask->hdr.dst_addr = DEST_MASK; /* The mask to apply to the dest ip. */ + ip_mask->hdr.dst_addr = FULL_MASK; /* The mask to apply to the dest ip. */ ip_spec->hdr.src_addr = htonl(SRC_IP); /* The src ip value to match the input packet. */ ip_mask->hdr.src_addr = EMPTY_MASK; /* The mask to apply to the src ip. */ @@ -77,5 +77,6 @@ snippet_ipv4_flow_create_actions_template(uint16_t port_id, struct rte_flow_erro }; - tactions[0].type = RTE_FLOW_ACTION_TYPE_END; + tactions[0].type = RTE_FLOW_ACTION_TYPE_QUEUE; + tactions[1].type = RTE_FLOW_ACTION_TYPE_END; /* This sets the masks to match the actions, indicating that all fields of the actions @@ -93,5 +94,5 @@ snippet_ipv4_flow_create_pattern_template(uint16_t port_id, struct rte_flow_erro { struct rte_flow_item titems[MAX_PATTERN_NUM] = {0}; - struct rte_flow_item_tcp ip_mask = {0}; + struct rte_flow_item_ipv4 ip_mask = {0}; struct rte_flow_pattern_template_attr attr = { @@ -102,6 +103,6 @@ snippet_ipv4_flow_create_pattern_template(uint16_t port_id, struct rte_flow_erro titems[0].type = RTE_FLOW_ITEM_TYPE_ETH; titems[1].type = RTE_FLOW_ITEM_TYPE_IPV4; - ip_mask.hdr.src_port = EMPTY_MASK; - ip_mask.hdr.dst_port = DEST_MASK; + ip_mask.hdr.src_addr = EMPTY_MASK; + ip_mask.hdr.dst_addr = FULL_MASK; titems[1].mask = &ip_mask; titems[2].type = RTE_FLOW_ITEM_TYPE_END; diff --git a/examples/flow_filtering/snippets/snippet_match_ipv4.h b/examples/flow_filtering/snippets/snippet_match_ipv4.h index 847784beef..597a1c954e 100644 --- a/examples/flow_filtering/snippets/snippet_match_ipv4.h +++ b/examples/flow_filtering/snippets/snippet_match_ipv4.h @@ -14,5 +14,4 @@ #define FULL_MASK 0xffffffff /* full mask */ #define EMPTY_MASK 0x0 /* empty mask */ -#define DEST_MASK 0xffff /* full mask */ #define MAX_PATTERN_NUM 3 /* Maximal number of patterns for this example. */ -- 2.48.1 --- Diff of the applied patch vs upstream commit (please double-check if non-empty: --- --- - 2025-03-24 16:15:15.825469086 +0000 +++ 0028-examples-flow_filtering-fix-IPv4-matching-snippet.patch 2025-03-24 16:15:14.859131256 +0000 @@ -1 +1 @@ -From 5fa25916f1ab47c82310264449a6d67dbb15ca4d Mon Sep 17 00:00:00 2001 +From b540e51a864d25b1e254fa8c311b92223e0f4162 Mon Sep 17 00:00:00 2001 @@ -5,0 +6,2 @@ +[ upstream commit 5fa25916f1ab47c82310264449a6d67dbb15ca4d ] + @@ -14 +15,0 @@ -Cc: stable@dpdk.org