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 51E3B46BAE for ; Fri, 18 Jul 2025 21:38:49 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 4CA9940611; Fri, 18 Jul 2025 21:38:49 +0200 (CEST) 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 0033640B8F for ; Fri, 18 Jul 2025 21:38:47 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1752867527; 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=OJtg0sJok0B7Rk8gkFqsF6mjvlu0n1pQjLescaq5x70=; b=FdJfGOPUuS4n+WN8SJxspu5vHwpa1a3WLBH7YaXrWarRvbywUv6YVPPQ7BCugAKybiHOvA nQw2wViIFED5v0qb3pI/Z2rwChz3AIu0RzROmmaLi/9eTjMZYb0iV6Wvta4psIv+a6WCH8 CXG1OyYO/cKln8V8NP2N6HUw/nxuMtk= Received: from mx-prod-mc-04.mail-002.prod.us-west-2.aws.redhat.com (ec2-54-186-198-63.us-west-2.compute.amazonaws.com [54.186.198.63]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-33-MvBGWhNsMQGa8N06Mo4Gvw-1; Fri, 18 Jul 2025 15:38:44 -0400 X-MC-Unique: MvBGWhNsMQGa8N06Mo4Gvw-1 X-Mimecast-MFC-AGG-ID: MvBGWhNsMQGa8N06Mo4Gvw_1752867523 Received: from mx-prod-int-06.mail-002.prod.us-west-2.aws.redhat.com (mx-prod-int-06.mail-002.prod.us-west-2.aws.redhat.com [10.30.177.93]) (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-04.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTPS id 37B3C19560A2; Fri, 18 Jul 2025 19:38:43 +0000 (UTC) Received: from rh.redhat.com (unknown [10.44.32.40]) by mx-prod-int-06.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTP id 5838018003FC; Fri, 18 Jul 2025 19:38:41 +0000 (UTC) From: Kevin Traynor To: Chaoyong He Cc: Long Wu , Peng Zhang , dpdk stable Subject: patch 'net/nfp: standardize NFD3 Tx descriptor endianness' has been queued to stable release 24.11.3 Date: Fri, 18 Jul 2025 20:31:42 +0100 Message-ID: <20250718193247.1008129-168-ktraynor@redhat.com> In-Reply-To: <20250718193247.1008129-1-ktraynor@redhat.com> References: <20250718193247.1008129-1-ktraynor@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.4.1 on 10.30.177.93 X-Mimecast-Spam-Score: 0 X-Mimecast-MFC-PROC-ID: DiTfz3MSGJV_4-x4zw7l_-8qZPC-NUUUd0X9QDhwA08_1752867523 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.3 Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet. It will be pushed if I get no objections before 07/23/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/1a18833d453a5f78890ddc2170ceee5a7bd9a4c3 Thanks. Kevin --- >From 1a18833d453a5f78890ddc2170ceee5a7bd9a4c3 Mon Sep 17 00:00:00 2001 From: Chaoyong He Date: Tue, 15 Apr 2025 10:54:52 +0800 Subject: [PATCH] net/nfp: standardize NFD3 Tx descriptor endianness [ upstream commit 1095bb0f86fafee3c7dca7b2dd23075ae869f689 ] The data endian of NFD3 Tx descriptor should be little, and the related logic also should modify. Fixes: 3745dd9dd86f ("net/nfp: adjust coding style for NFD3") Signed-off-by: Chaoyong He Reviewed-by: Long Wu Reviewed-by: Peng Zhang --- drivers/net/nfp/flower/nfp_flower_ctrl.c | 4 ++-- drivers/net/nfp/nfd3/nfp_nfd3.h | 14 +++++++------- drivers/net/nfp/nfd3/nfp_nfd3_dp.c | 8 ++++---- 3 files changed, 13 insertions(+), 13 deletions(-) diff --git a/drivers/net/nfp/flower/nfp_flower_ctrl.c b/drivers/net/nfp/flower/nfp_flower_ctrl.c index 23d1b770d8..6ac2caf367 100644 --- a/drivers/net/nfp/flower/nfp_flower_ctrl.c +++ b/drivers/net/nfp/flower/nfp_flower_ctrl.c @@ -195,8 +195,8 @@ nfp_flower_ctrl_vnic_nfd3_xmit(struct nfp_app_fw_flower *app_fw_flower, dma_addr = rte_mbuf_data_iova(mbuf); - txds->data_len = mbuf->pkt_len; + txds->data_len = rte_cpu_to_le_16(mbuf->pkt_len); txds->dma_len = txds->data_len; txds->dma_addr_hi = (dma_addr >> 32) & 0xff; - txds->dma_addr_lo = (dma_addr & 0xffffffff); + txds->dma_addr_lo = rte_cpu_to_le_32(dma_addr & 0xffffffff); txds->offset_eop = FLOWER_PKT_DATA_OFFSET | NFD3_DESC_TX_EOP; diff --git a/drivers/net/nfp/nfd3/nfp_nfd3.h b/drivers/net/nfp/nfd3/nfp_nfd3.h index 3ba562cc3f..6e636a86b8 100644 --- a/drivers/net/nfp/nfd3/nfp_nfd3.h +++ b/drivers/net/nfp/nfd3/nfp_nfd3.h @@ -18,11 +18,11 @@ struct nfp_net_nfd3_tx_desc { union { struct { - uint8_t dma_addr_hi; /**< High bits of host buf address */ - uint16_t dma_len; /**< Length to DMA for this desc */ + uint8_t dma_addr_hi; /**< High bits of host buf address */ + rte_le16_t dma_len; /**< Length to DMA for this desc */ /** Offset in buf where pkt starts + highest bit is eop flag */ uint8_t offset_eop; - uint32_t dma_addr_lo; /**< Low 32bit of host buf addr */ + rte_le32_t dma_addr_lo; /**< Low 32bit of host buf addr */ - uint16_t mss; /**< MSS to be used for LSO */ + rte_le16_t mss; /**< MSS to be used for LSO */ uint8_t lso_hdrlen; /**< LSO, where the data starts */ uint8_t flags; /**< TX Flags, see @NFD3_DESC_TX_* */ @@ -33,9 +33,9 @@ struct nfp_net_nfd3_tx_desc { uint8_t l4_offset; /**< L4 header offset */ }; - uint16_t vlan; /**< VLAN tag to add if indicated */ + rte_le16_t vlan; /**< VLAN tag to add if indicated */ }; - uint16_t data_len; /**< Length of frame + meta data */ + rte_le16_t data_len; /**< Length of frame + meta data */ } __rte_packed; - uint32_t vals[4]; + rte_le32_t vals[4]; }; }; diff --git a/drivers/net/nfp/nfd3/nfp_nfd3_dp.c b/drivers/net/nfp/nfd3/nfp_nfd3_dp.c index 3ffcbb2576..6466a5a4fb 100644 --- a/drivers/net/nfp/nfd3/nfp_nfd3_dp.c +++ b/drivers/net/nfp/nfd3/nfp_nfd3_dp.c @@ -140,5 +140,5 @@ nfp_net_nfd3_tx_vlan(struct nfp_net_txq *txq, if ((mb->ol_flags & RTE_MBUF_F_TX_VLAN) != 0) { txd->flags |= NFD3_DESC_TX_VLAN; - txd->vlan = mb->vlan_tci; + txd->vlan = rte_cpu_to_le_16(mb->vlan_tci); } } @@ -301,5 +301,5 @@ nfp_net_nfd3_xmit_pkts_common(void *tx_queue, * multisegment packet, but TSO info needs to be in all of them. */ - txd.data_len = pkt->pkt_len; + txd.data_len = rte_cpu_to_le_16((uint16_t)pkt->pkt_len); nfp_net_nfd3_tx_tso(txq, &txd, pkt); nfp_net_nfd3_tx_cksum(txq, &txd, pkt); @@ -331,8 +331,8 @@ nfp_net_nfd3_xmit_pkts_common(void *tx_queue, /* Filling descriptors fields */ - txds->dma_len = dma_size; + txds->dma_len = rte_cpu_to_le_16(dma_size); txds->data_len = txd.data_len; txds->dma_addr_hi = (dma_addr >> 32) & 0xff; - txds->dma_addr_lo = (dma_addr & 0xffffffff); + txds->dma_addr_lo = rte_cpu_to_le_32(dma_addr & 0xffffffff); free_descs--; -- 2.50.0 --- Diff of the applied patch vs upstream commit (please double-check if non-empty: --- --- - 2025-07-18 20:29:16.853261081 +0100 +++ 0168-net-nfp-standardize-NFD3-Tx-descriptor-endianness.patch 2025-07-18 20:29:11.178908091 +0100 @@ -1 +1 @@ -From 1095bb0f86fafee3c7dca7b2dd23075ae869f689 Mon Sep 17 00:00:00 2001 +From 1a18833d453a5f78890ddc2170ceee5a7bd9a4c3 Mon Sep 17 00:00:00 2001 @@ -5,0 +6,2 @@ +[ upstream commit 1095bb0f86fafee3c7dca7b2dd23075ae869f689 ] + @@ -10 +11,0 @@ -Cc: stable@dpdk.org @@ -37 +38 @@ -index 96ea23b3df..dc24f4fa1d 100644 +index 3ba562cc3f..6e636a86b8 100644 @@ -42 +43 @@ - struct __rte_packed_begin { + struct { @@ -64 +65 @@ - } __rte_packed_end; + } __rte_packed;