From patchwork Mon Sep 4 04:56:49 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Renyong Wan X-Patchwork-Id: 131121 X-Patchwork-Delegate: ferruh.yigit@amd.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 1FA83424E7; Mon, 4 Sep 2023 07:00:06 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 7891B40FDE; Mon, 4 Sep 2023 06:58:04 +0200 (CEST) Received: from VLXDG1SPAM1.ramaxel.com (email.unionmem.com [221.4.138.186]) by mails.dpdk.org (Postfix) with ESMTP id 9C66740E03 for ; Mon, 4 Sep 2023 06:57:58 +0200 (CEST) Received: from V12DG1MBS03.ramaxel.local ([172.26.18.33]) by VLXDG1SPAM1.ramaxel.com with ESMTP id 3844vHHr054277; Mon, 4 Sep 2023 12:57:17 +0800 (GMT-8) (envelope-from wanry@3snic.com) Received: from localhost.localdomain (10.64.136.151) by V12DG1MBS03.ramaxel.local (172.26.18.33) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2375.17; Mon, 4 Sep 2023 12:57:17 +0800 From: To: CC: , Renyong Wan , Steven Song Subject: [PATCH v5 23/32] net/sssnic: support Rx packet burst Date: Mon, 4 Sep 2023 12:56:49 +0800 Message-ID: <20230904045658.238185-24-wanry@3snic.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20230904045658.238185-1-wanry@3snic.com> References: <20230904045658.238185-1-wanry@3snic.com> MIME-Version: 1.0 X-Originating-IP: [10.64.136.151] X-ClientProxiedBy: V12DG1MBS03.ramaxel.local (172.26.18.33) To V12DG1MBS03.ramaxel.local (172.26.18.33) X-DNSRBL: X-SPAM-SOURCE-CHECK: pass X-MAIL: VLXDG1SPAM1.ramaxel.com 3844vHHr054277 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 From: Renyong Wan Signed-off-by: Steven Song Signed-off-by: Renyong Wan --- v2: * Fixed wrong format of printing uint64_t. --- doc/guides/nics/features/sssnic.ini | 2 + drivers/net/sssnic/sssnic_ethdev.c | 2 + drivers/net/sssnic/sssnic_ethdev_rx.c | 167 ++++++++++++++++++++++++++ drivers/net/sssnic/sssnic_ethdev_rx.h | 2 + 4 files changed, 173 insertions(+) diff --git a/doc/guides/nics/features/sssnic.ini b/doc/guides/nics/features/sssnic.ini index aba0b78c95..320ac4533d 100644 --- a/doc/guides/nics/features/sssnic.ini +++ b/doc/guides/nics/features/sssnic.ini @@ -8,6 +8,8 @@ Link status = Y Link status event = Y Queue start/stop = Y Rx interrupt = Y +Scattered Rx = Y +LRO = Y Promiscuous mode = Y Allmulticast mode = Y Unicast MAC filter = Y diff --git a/drivers/net/sssnic/sssnic_ethdev.c b/drivers/net/sssnic/sssnic_ethdev.c index 99e6d6152a..021fabcbe5 100644 --- a/drivers/net/sssnic/sssnic_ethdev.c +++ b/drivers/net/sssnic/sssnic_ethdev.c @@ -769,6 +769,8 @@ sssnic_ethdev_init(struct rte_eth_dev *ethdev) if (rte_eal_process_type() != RTE_PROC_PRIMARY) return 0; + ethdev->rx_pkt_burst = sssnic_ethdev_rx_pkt_burst; + netdev = SSSNIC_ETHDEV_PRIVATE(ethdev); pci_dev = RTE_ETH_DEV_TO_PCI(ethdev); hw = rte_zmalloc("sssnic_hw", sizeof(struct sssnic_hw), 0); diff --git a/drivers/net/sssnic/sssnic_ethdev_rx.c b/drivers/net/sssnic/sssnic_ethdev_rx.c index 66045f7a98..82e65f2482 100644 --- a/drivers/net/sssnic/sssnic_ethdev_rx.c +++ b/drivers/net/sssnic/sssnic_ethdev_rx.c @@ -1183,3 +1183,170 @@ sssnic_ethdev_rx_queue_stats_clear(struct rte_eth_dev *ethdev, uint16_t qid) memset(&rxq->stats, 0, sizeof(rxq->stats)); } }; + +static inline void +sssnic_ethdev_rx_csum_offload(struct sssnic_ethdev_rxq *rxq, + struct rte_mbuf *rxm, volatile struct sssnic_ethdev_rx_desc *rxd) +{ + /* no errors */ + if (likely(rxd->status_err == 0)) { + rxm->ol_flags |= RTE_MBUF_F_RX_IP_CKSUM_GOOD | + RTE_MBUF_F_RX_L4_CKSUM_GOOD; + return; + } + + /* bypass hw crc error*/ + if (unlikely(rxd->hw_crc_err)) { + rxm->ol_flags |= RTE_MBUF_F_RX_IP_CKSUM_UNKNOWN; + return; + } + + if (rxd->ip_csum_err) { + rxm->ol_flags |= RTE_MBUF_F_RX_IP_CKSUM_BAD; + rxq->stats.csum_errors++; + } + + if (rxd->tcp_csum_err || rxd->udp_csum_err || rxd->sctp_crc_err) { + rxm->ol_flags |= RTE_MBUF_F_RX_L4_CKSUM_BAD; + rxq->stats.csum_errors++; + } + + if (unlikely(rxd->other_err)) + rxq->stats.other_errors++; +} + +static inline void +sssnic_ethdev_rx_vlan_offload(struct rte_mbuf *rxm, + volatile struct sssnic_ethdev_rx_desc *rxd) +{ + if (rxd->vlan_en == 0 || rxd->vlan == 0) { + rxm->vlan_tci = 0; + return; + } + + rxm->vlan_tci = rxd->vlan; + rxm->ol_flags |= RTE_MBUF_F_RX_VLAN | RTE_MBUF_F_RX_VLAN_STRIPPED; +} + +static inline void +sssnic_ethdev_rx_segments(struct sssnic_ethdev_rxq *rxq, struct rte_mbuf *head, + uint32_t remain_size) +{ + struct sssnic_ethdev_rx_entry *rxe; + struct rte_mbuf *curr, *prev = head; + uint16_t rx_buf_size = rxq->rx_buf_size; + uint16_t ci; + uint32_t rx_size; + + while (remain_size > 0) { + ci = sssnic_ethdev_rxq_ci_get(rxq); + rxe = &rxq->rxe[ci]; + curr = rxe->pktmbuf; + + sssnic_ethdev_rxq_consume(rxq, 1); + + rx_size = RTE_MIN(remain_size, rx_buf_size); + remain_size -= rx_size; + + curr->data_len = rx_size; + curr->next = NULL; + prev->next = curr; + prev = curr; + head->nb_segs++; + } +} + +uint16_t +sssnic_ethdev_rx_pkt_burst(void *rx_queue, struct rte_mbuf **rx_pkts, + uint16_t nb_pkts) +{ + struct sssnic_ethdev_rxq *rxq = (struct sssnic_ethdev_rxq *)rx_queue; + struct sssnic_ethdev_rx_entry *rxe; + struct rte_mbuf *rxm; + struct sssnic_ethdev_rx_desc *rxd, rx_desc; + uint16_t ci, idle_entries; + uint16_t rx_buf_size; + uint32_t rx_size; + uint64_t nb_rx = 0; + uint64_t rx_bytes = 0; + + ci = sssnic_ethdev_rxq_ci_get(rxq); + rx_buf_size = rxq->rx_buf_size; + rxd = &rx_desc; + + while (nb_rx < nb_pkts) { + rxd->dword0 = __atomic_load_n(&rxq->desc[ci].dword0, + __ATOMIC_ACQUIRE); + /* check rx done */ + if (!rxd->done) + break; + + rxd->dword1 = rxq->desc[ci].dword1; + rxd->dword2 = rxq->desc[ci].dword2; + rxd->dword3 = rxq->desc[ci].dword3; + + /* reset rx desc status */ + rxq->desc[ci].dword0 = 0; + + /* get current pktmbuf */ + rxe = &rxq->rxe[ci]; + rxm = rxe->pktmbuf; + + /* prefetch next packet */ + sssnic_ethdev_rxq_consume(rxq, 1); + ci = sssnic_ethdev_rxq_ci_get(rxq); + rte_prefetch0(rxq->rxe[ci].pktmbuf); + + /* set pktmbuf len */ + rx_size = rxd->len; + rxm->pkt_len = rx_size; + if (likely(rx_size <= rx_buf_size)) { + rxm->data_len = rx_size; + } else { + rxm->data_len = rx_buf_size; + sssnic_ethdev_rx_segments(rxq, rxm, + rx_size - rx_buf_size); + } + rxm->data_off = RTE_PKTMBUF_HEADROOM; + rxm->port = rxq->port; + + /* process checksum offload*/ + sssnic_ethdev_rx_csum_offload(rxq, rxm, rxd); + + /* process vlan offload */ + sssnic_ethdev_rx_vlan_offload(rxm, rxd); + + /* process lro */ + if (unlikely(rxd->lro_num != 0)) { + rxm->ol_flags |= RTE_MBUF_F_RX_LRO; + rxm->tso_segsz = rx_size / rxd->lro_num; + } + + /* process RSS offload */ + if (likely(rxd->rss_type != 0)) { + rxm->hash.rss = rxd->rss_hash; + rxm->ol_flags |= RTE_MBUF_F_RX_RSS_HASH; + } + + rx_pkts[nb_rx++] = rxm; + rx_bytes += rx_size; + + SSSNIC_RX_LOG(DEBUG, + "Received one packet on port %u, len=%u, nb_seg=%u, tso_segsz=%u, ol_flags=%" + PRIx64, rxq->port, rxm->pkt_len, rxm->nb_segs, rxm->tso_segsz, + rxm->ol_flags); + } + + if (nb_rx > 0) { + rxq->stats.packets += nb_rx; + rxq->stats.bytes += rx_bytes; + rxq->stats.burst = nb_rx; + + /* refill packet mbuf */ + idle_entries = sssnic_ethdev_rxq_num_idle_entries(rxq) - 1; + if (idle_entries >= rxq->rx_free_thresh) + sssnic_ethdev_rxq_pktmbufs_fill(rxq); + } + + return nb_rx; +} diff --git a/drivers/net/sssnic/sssnic_ethdev_rx.h b/drivers/net/sssnic/sssnic_ethdev_rx.h index 5532aced4e..b0b35dee73 100644 --- a/drivers/net/sssnic/sssnic_ethdev_rx.h +++ b/drivers/net/sssnic/sssnic_ethdev_rx.h @@ -42,5 +42,7 @@ int sssnic_ethdev_rx_queue_stats_get(struct rte_eth_dev *ethdev, uint16_t qid, struct sssnic_ethdev_rxq_stats *stats); void sssnic_ethdev_rx_queue_stats_clear(struct rte_eth_dev *ethdev, uint16_t qid); +uint16_t sssnic_ethdev_rx_pkt_burst(void *rx_queue, struct rte_mbuf **rx_pkts, + uint16_t nb_pkts); #endif