From patchwork Wed Dec 27 04:21:02 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ajit Khaparde X-Patchwork-Id: 135589 X-Patchwork-Delegate: ajit.khaparde@broadcom.com Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 3F21F437A1; Wed, 27 Dec 2023 05:21:33 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 67806402E2; Wed, 27 Dec 2023 05:21:29 +0100 (CET) Received: from mail-qt1-f178.google.com (mail-qt1-f178.google.com [209.85.160.178]) by mails.dpdk.org (Postfix) with ESMTP id 0572E402D4 for ; Wed, 27 Dec 2023 05:21:27 +0100 (CET) Received: by mail-qt1-f178.google.com with SMTP id d75a77b69052e-427b1bf1896so35393711cf.3 for ; Tue, 26 Dec 2023 20:21:27 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=broadcom.com; s=google; t=1703650886; x=1704255686; darn=dpdk.org; h=mime-version:references:in-reply-to:message-id:date:subject:cc:to :from:from:to:cc:subject:date:message-id:reply-to; bh=6dKa0pLpkzftUTo0mqNwA03p0/8U/DeVQDJjuuT6np8=; b=Nn8jMZIsRLyxk9SNd3GtDtBb/oXyEw7DvUT/U6plZPhikm1YhrvAHQf1/Y9whAgWNG 1nOx4ybKf5ZHiXZMyjpCSkQ1o772FSaEFkqbeI39Sfb6/ui0WjEIqg8C3B90OGwLZTFv tDLmHLcd/uzHrYvBgof62LodEiCBQHMebInfs= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1703650886; x=1704255686; h=mime-version:references:in-reply-to:message-id:date:subject:cc:to :from:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=6dKa0pLpkzftUTo0mqNwA03p0/8U/DeVQDJjuuT6np8=; b=xFO2JJnMtb1PlMze/jUk5SDrYCpA2fEn52kDgzJp2uaHea2tyQjvOCN9tQI31b3Wyy ZpEJawXp//W/JN8zzwTsEqOPHLT929LqMSeHPIHDXjS24SBMfOcHaH1b8ON4TLoXlv+C sFsfFh4ZxZYspH0d050TmcnzJFLhVkVhyTCkMw8PHK/EAthSz1ZpiORxM1K8zinvJ8R1 EdRNpypL3IFVvwQQeDPVeDGr0qhjnqji4OuIPfJqU+Te4J2hi3WiM53wOc0sccj892Xb 2b9c3EpGVTEMnlUnLhSba7UCbKuIKk9APaSksWhSU/fByMGHjKWybqtUKlZ/NeMRdlSq Nmhg== X-Gm-Message-State: AOJu0YxxWNQ+NOLdsJabUmfeLUWGQXFK8MZ9Tz3IEN2ETmsBV3dl8QDy r4JHJgdS0c/Nfa7idzlVFaSu6Pak3lvrgdBciit8Ru0xEIENL9zF0f0PDmsbppNFz3TtcZuHr1i 6C/T9UuSsDnLems5IIVAp0SiridR4jPDOt+h1PcKjVU2k23E1pohY08y7QxFSE57qyxR8NAeGWn 4= X-Google-Smtp-Source: AGHT+IHLVYTFVZuhe0xtaAtgZ07dIESdgkwzyznXnxBiT4Pq2mzqj1redBnWHLZ3FYHTfdT+M49SKQ== X-Received: by 2002:a05:622a:3cc:b0:427:7a3e:57b3 with SMTP id k12-20020a05622a03cc00b004277a3e57b3mr11255390qtx.93.1703650886005; Tue, 26 Dec 2023 20:21:26 -0800 (PST) Received: from localhost.localdomain ([2605:a601:a780:1400:c066:75e3:74c8:50e6]) by smtp.gmail.com with ESMTPSA id bt7-20020ac86907000000b00427e120889bsm1415488qtb.91.2023.12.26.20.21.24 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 26 Dec 2023 20:21:25 -0800 (PST) From: Ajit Khaparde To: dev@dpdk.org Cc: Damodharam Ammepalli Subject: [PATCH v3 01/18] net/bnxt: add support for UDP GSO Date: Tue, 26 Dec 2023 20:21:02 -0800 Message-Id: <20231227042119.72469-2-ajit.khaparde@broadcom.com> X-Mailer: git-send-email 2.39.2 (Apple Git-143) In-Reply-To: <20231227042119.72469-1-ajit.khaparde@broadcom.com> References: <20231227042119.72469-1-ajit.khaparde@broadcom.com> MIME-Version: 1.0 X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org UDP GSO aka UDP Fragmentation Offload allows an application or stack to provide a data payload larger than the MTU. The application then updates the mbuf ol_flags and sets the PKT_TX_UDP_SEG flag. Then based on the tso_segs and tso_mss fields in the mbuf the PMD can indicate the UDP GSO transmit request to the hardware. This feature is supported on Thor2 and will be enabled when the firmware sets the UDP GSO support via the HWRM_FUNC_QCAPS. Signed-off-by: Ajit Khaparde Reviewed-by: Damodharam Ammepalli --- drivers/net/bnxt/bnxt.h | 1 + drivers/net/bnxt/bnxt_hwrm.c | 2 ++ drivers/net/bnxt/bnxt_txq.c | 2 ++ drivers/net/bnxt/bnxt_txr.c | 7 ++++++- 4 files changed, 11 insertions(+), 1 deletion(-) diff --git a/drivers/net/bnxt/bnxt.h b/drivers/net/bnxt/bnxt.h index 7aed4c3da3..4b5c2c4b8f 100644 --- a/drivers/net/bnxt/bnxt.h +++ b/drivers/net/bnxt/bnxt.h @@ -801,6 +801,7 @@ struct bnxt { (BNXT_CHIP_P5_P7((bp)) && \ (bp)->hwrm_spec_code >= HWRM_VERSION_1_9_2 && \ !BNXT_VF((bp))) +#define BNXT_FW_CAP_UDP_GSO BIT(13) #define BNXT_TRUFLOW_EN(bp) ((bp)->fw_cap & BNXT_FW_CAP_TRUFLOW_EN &&\ (bp)->app_id != 0xFF) diff --git a/drivers/net/bnxt/bnxt_hwrm.c b/drivers/net/bnxt/bnxt_hwrm.c index e56f7693af..37cf179938 100644 --- a/drivers/net/bnxt/bnxt_hwrm.c +++ b/drivers/net/bnxt/bnxt_hwrm.c @@ -950,6 +950,8 @@ static int __bnxt_hwrm_func_qcaps(struct bnxt *bp) flags_ext2 = rte_le_to_cpu_32(resp->flags_ext2); if (flags_ext2 & HWRM_FUNC_QCAPS_OUTPUT_FLAGS_EXT2_RX_ALL_PKTS_TIMESTAMPS_SUPPORTED) bp->fw_cap |= BNXT_FW_CAP_RX_ALL_PKT_TS; + if (flags_ext2 & HWRM_FUNC_QCAPS_OUTPUT_FLAGS_EXT2_UDP_GSO_SUPPORTED) + bp->fw_cap |= BNXT_FW_CAP_UDP_GSO; unlock: HWRM_UNLOCK(); diff --git a/drivers/net/bnxt/bnxt_txq.c b/drivers/net/bnxt/bnxt_txq.c index 4df4604975..f99ad211db 100644 --- a/drivers/net/bnxt/bnxt_txq.c +++ b/drivers/net/bnxt/bnxt_txq.c @@ -42,6 +42,8 @@ uint64_t bnxt_get_tx_port_offloads(struct bnxt *bp) tx_offload_capa |= RTE_ETH_TX_OFFLOAD_GENEVE_TNL_TSO; if (BNXT_TUNNELED_OFFLOADS_CAP_IPINIP_EN(bp)) tx_offload_capa |= RTE_ETH_TX_OFFLOAD_IPIP_TNL_TSO; + if (bp->fw_cap & BNXT_FW_CAP_UDP_GSO) + tx_offload_capa |= RTE_ETH_TX_OFFLOAD_UDP_TSO; return tx_offload_capa; } diff --git a/drivers/net/bnxt/bnxt_txr.c b/drivers/net/bnxt/bnxt_txr.c index 899986764f..38da2d2829 100644 --- a/drivers/net/bnxt/bnxt_txr.c +++ b/drivers/net/bnxt/bnxt_txr.c @@ -123,6 +123,10 @@ bnxt_xmit_need_long_bd(struct rte_mbuf *tx_pkt, struct bnxt_tx_queue *txq) return false; } +/* Used for verifying TSO segments during TCP Segmentation Offload or + * UDP Fragmentation Offload. tx_pkt->tso_segsz stores the number of + * segments or fragments in those cases. + */ static bool bnxt_zero_data_len_tso_segsz(struct rte_mbuf *tx_pkt, uint8_t data_len_chk) { @@ -308,7 +312,8 @@ static uint16_t bnxt_start_xmit(struct rte_mbuf *tx_pkt, else txbd1->cfa_action = txq->bp->tx_cfa_action; - if (tx_pkt->ol_flags & RTE_MBUF_F_TX_TCP_SEG) { + if (tx_pkt->ol_flags & RTE_MBUF_F_TX_TCP_SEG || + tx_pkt->ol_flags & RTE_MBUF_F_TX_UDP_SEG) { uint16_t hdr_size; /* TSO */