From nobody Mon Apr 6 09:41:23 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id C9E80ECAAD5 for ; Thu, 8 Sep 2022 08:29:01 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230306AbiIHI27 (ORCPT ); Thu, 8 Sep 2022 04:28:59 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:45668 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230115AbiIHI2x (ORCPT ); Thu, 8 Sep 2022 04:28:53 -0400 Received: from esa.microchip.iphmx.com (esa.microchip.iphmx.com [68.232.154.123]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 50A2065261; Thu, 8 Sep 2022 01:28:52 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=microchip.com; i=@microchip.com; q=dns/txt; s=mchp; t=1662625733; x=1694161733; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=vrAqCn0jfykHzJwc/hiNhKrO7efAl9Cz/8tuUkxVNys=; b=UhJ02HcfLeugZ/2w4FOg58+its7nPUhffPSyR2C8GDxJWoT/Ym2ROJzl HtKRYh940y4irquppLlBu1uF02zwozxWYwmBlE7PJO3EDiWstU44/ocOO aBLP3h6raJLEAw65QIaB5xjRv+hP8TtxzSMUdU1+aBoCfWPK+LK2Cq0lH vlFd2Tu/RrguquO5E/+X+vwfogqMdpgPq2vSby9umblIJrLBuvufVptJC S1vADQM+ZY1juygGIm4ZO8skMjH3zgHwVOwKpWtRz6ZDJLbMqN2cQz0ec joPbPb4KMmpPTjWxJ0G3/NaRtiMNTTSOKYGo1MKNweHQv1WqEjXtQ4L/u Q==; X-IronPort-AV: E=Sophos;i="5.93,299,1654585200"; d="scan'208";a="172912879" Received: from unknown (HELO email.microchip.com) ([170.129.1.10]) by esa4.microchip.iphmx.com with ESMTP/TLS/AES256-SHA256; 08 Sep 2022 01:28:52 -0700 Received: from chn-vm-ex04.mchp-main.com (10.10.85.152) by chn-vm-ex01.mchp-main.com (10.10.85.143) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.12; Thu, 8 Sep 2022 01:28:51 -0700 Received: from localhost.localdomain (10.10.115.15) by chn-vm-ex04.mchp-main.com (10.10.85.152) with Microsoft SMTP Server id 15.1.2507.12 via Frontend Transport; Thu, 8 Sep 2022 01:28:48 -0700 From: Raju Lakkaraju To: CC: , , , , , Subject: [PATCH net-next V2 1/2] net: lan743x: Fix to use multiqueue start/stop APIs Date: Thu, 8 Sep 2022 13:58:33 +0530 Message-ID: <20220908082834.5070-2-Raju.Lakkaraju@microchip.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20220908082834.5070-1-Raju.Lakkaraju@microchip.com> References: <20220908082834.5070-1-Raju.Lakkaraju@microchip.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" - Fix to use multiqueue start/stop APIs - Change to return NETDEV_TX_BUSY instead of holding the TX skb when busy - Increase Tx ring size to 128 to address performance issues in some platf= orms - Use NAPI_POLL_WEIGHT for Tx Napi handler instead of ring dependent value - Use multiqueue to register 4 Rx channels Signed-off-by: Raju Lakkaraju --- drivers/net/ethernet/microchip/lan743x_main.c | 50 +++++++++---------- drivers/net/ethernet/microchip/lan743x_main.h | 5 +- 2 files changed, 27 insertions(+), 28 deletions(-) diff --git a/drivers/net/ethernet/microchip/lan743x_main.c b/drivers/net/et= hernet/microchip/lan743x_main.c index a9a1dea6d731..3f4e1ab63f8a 100644 --- a/drivers/net/ethernet/microchip/lan743x_main.c +++ b/drivers/net/ethernet/microchip/lan743x_main.c @@ -2066,11 +2066,13 @@ static netdev_tx_t lan743x_tx_xmit_frame(struct lan= 743x_tx *tx, { int required_number_of_descriptors =3D 0; unsigned int start_frame_length =3D 0; + netdev_tx_t retval =3D NETDEV_TX_OK; unsigned int frame_length =3D 0; unsigned int head_length =3D 0; unsigned long irq_flags =3D 0; bool do_timestamp =3D false; bool ignore_sync =3D false; + struct netdev_queue *txq; int nr_frags =3D 0; bool gso =3D false; int j; @@ -2083,9 +2085,12 @@ static netdev_tx_t lan743x_tx_xmit_frame(struct lan7= 43x_tx *tx, if (required_number_of_descriptors > (tx->ring_size - 1)) { dev_kfree_skb_irq(skb); } else { - /* save to overflow buffer */ - tx->overflow_skb =3D skb; - netif_stop_queue(tx->adapter->netdev); + /* save how many descriptors we needed to restart the queue */ + tx->rqd_descriptors =3D required_number_of_descriptors; + retval =3D NETDEV_TX_BUSY; + txq =3D netdev_get_tx_queue(tx->adapter->netdev, + tx->channel_number); + netif_tx_stop_queue(txq); } goto unlock; } @@ -2144,15 +2149,15 @@ static netdev_tx_t lan743x_tx_xmit_frame(struct lan= 743x_tx *tx, =20 unlock: spin_unlock_irqrestore(&tx->ring_lock, irq_flags); - return NETDEV_TX_OK; + return retval; } =20 static int lan743x_tx_napi_poll(struct napi_struct *napi, int weight) { struct lan743x_tx *tx =3D container_of(napi, struct lan743x_tx, napi); struct lan743x_adapter *adapter =3D tx->adapter; - bool start_transmitter =3D false; unsigned long irq_flags =3D 0; + struct netdev_queue *txq; u32 ioc_bit =3D 0; =20 ioc_bit =3D DMAC_INT_BIT_TX_IOC_(tx->channel_number); @@ -2163,24 +2168,20 @@ static int lan743x_tx_napi_poll(struct napi_struct = *napi, int weight) =20 /* clean up tx ring */ lan743x_tx_release_completed_descriptors(tx); - if (netif_queue_stopped(adapter->netdev)) { - if (tx->overflow_skb) { - if (lan743x_tx_get_desc_cnt(tx, tx->overflow_skb) <=3D - lan743x_tx_get_avail_desc(tx)) - start_transmitter =3D true; + txq =3D netdev_get_tx_queue(adapter->netdev, tx->channel_number); + if (netif_tx_queue_stopped(txq)) { + if (tx->rqd_descriptors) { + if (tx->rqd_descriptors <=3D + lan743x_tx_get_avail_desc(tx)) { + tx->rqd_descriptors =3D 0; + netif_tx_wake_queue(txq); + } } else { - netif_wake_queue(adapter->netdev); + netif_tx_wake_queue(txq); } } spin_unlock_irqrestore(&tx->ring_lock, irq_flags); =20 - if (start_transmitter) { - /* space is now available, transmit overflow skb */ - lan743x_tx_xmit_frame(tx, tx->overflow_skb); - tx->overflow_skb =3D NULL; - netif_wake_queue(adapter->netdev); - } - if (!napi_complete(napi)) goto done; =20 @@ -2304,10 +2305,7 @@ static void lan743x_tx_close(struct lan743x_tx *tx) =20 lan743x_tx_release_all_descriptors(tx); =20 - if (tx->overflow_skb) { - dev_kfree_skb(tx->overflow_skb); - tx->overflow_skb =3D NULL; - } + tx->rqd_descriptors =3D 0; =20 lan743x_tx_ring_cleanup(tx); } @@ -2387,7 +2385,7 @@ static int lan743x_tx_open(struct lan743x_tx *tx) (tx->channel_number)); netif_napi_add_tx_weight(adapter->netdev, &tx->napi, lan743x_tx_napi_poll, - tx->ring_size - 1); + NAPI_POLL_WEIGHT); napi_enable(&tx->napi); =20 data =3D 0; @@ -3347,8 +3345,10 @@ static int lan743x_pcidev_probe(struct pci_dev *pdev, PCI11X1X_USED_TX_CHANNELS, LAN743X_USED_RX_CHANNELS); } else { - netdev =3D devm_alloc_etherdev(&pdev->dev, - sizeof(struct lan743x_adapter)); + netdev =3D devm_alloc_etherdev_mqs(&pdev->dev, + sizeof(struct lan743x_adapter), + LAN743X_USED_TX_CHANNELS, + LAN743X_USED_RX_CHANNELS); } =20 if (!netdev) diff --git a/drivers/net/ethernet/microchip/lan743x_main.h b/drivers/net/et= hernet/microchip/lan743x_main.h index 72adae4f2aa0..58eb7abf976b 100644 --- a/drivers/net/ethernet/microchip/lan743x_main.h +++ b/drivers/net/ethernet/microchip/lan743x_main.h @@ -954,8 +954,7 @@ struct lan743x_tx { =20 struct napi_struct napi; u32 frame_count; - - struct sk_buff *overflow_skb; + u32 rqd_descriptors; }; =20 void lan743x_tx_set_timestamping_mode(struct lan743x_tx *tx, @@ -1110,7 +1109,7 @@ struct lan743x_tx_buffer_info { unsigned int buffer_length; }; =20 -#define LAN743X_TX_RING_SIZE (50) +#define LAN743X_TX_RING_SIZE (128) =20 /* OWN bit is set. ie, Descs are owned by RX DMAC */ #define RX_DESC_DATA0_OWN_ (0x00008000) --=20 2.25.1 From nobody Mon Apr 6 09:41:23 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 230D5C38145 for ; Thu, 8 Sep 2022 08:29:17 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231163AbiIHI3N (ORCPT ); Thu, 8 Sep 2022 04:29:13 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:45788 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230162AbiIHI24 (ORCPT ); Thu, 8 Sep 2022 04:28:56 -0400 Received: from esa.microchip.iphmx.com (esa.microchip.iphmx.com [68.232.154.123]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 5C5C8D7414; Thu, 8 Sep 2022 01:28:55 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=microchip.com; i=@microchip.com; q=dns/txt; s=mchp; t=1662625736; x=1694161736; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=pvXGhsXqCvFjOKMAYkU55/LhhJsCGqqMvwdYH1QyFjE=; b=XsJB6sJC19t/kpbDCyrQB023JSHuP4vtGyTMZm7NIyRGb1vY6pQHwruA Ds5rCa9zxMJ6fDhAhandI3bVaXrcASxuWJCCBuEL7nLE/YuY/+g2zwVr7 RGFYVVbpaKk/TlxXoC9qv/pOKwm451iyw/MbVCqa1ntpfxefnYJAFTPwe dWLbXn94XEkckV6aeI+VyGHQHHRHizrY6aX2F92LJvL6AEhbAr0VF/HUS bD249orZj8KGS70bC62DpqLAi7FjEoj2Rkjcerv+P+8+xTmAy5TLPsyHh bqiVUU/ZQ+RZ223MJOlqJ97cd5MJE53vgxtHdngC9lO1v27GL7EiVJcM0 w==; X-IronPort-AV: E=Sophos;i="5.93,299,1654585200"; d="scan'208";a="172912891" Received: from unknown (HELO email.microchip.com) ([170.129.1.10]) by esa4.microchip.iphmx.com with ESMTP/TLS/AES256-SHA256; 08 Sep 2022 01:28:55 -0700 Received: from chn-vm-ex04.mchp-main.com (10.10.85.152) by chn-vm-ex04.mchp-main.com (10.10.85.152) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.12; Thu, 8 Sep 2022 01:28:54 -0700 Received: from localhost.localdomain (10.10.115.15) by chn-vm-ex04.mchp-main.com (10.10.85.152) with Microsoft SMTP Server id 15.1.2507.12 via Frontend Transport; Thu, 8 Sep 2022 01:28:52 -0700 From: Raju Lakkaraju To: CC: , , , , , , kernel test robot Subject: [PATCH net-next V2 2/2] net: lan743x: Add support for Rx IP & TCP checksum offload Date: Thu, 8 Sep 2022 13:58:34 +0530 Message-ID: <20220908082834.5070-3-Raju.Lakkaraju@microchip.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20220908082834.5070-1-Raju.Lakkaraju@microchip.com> References: <20220908082834.5070-1-Raju.Lakkaraju@microchip.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" Add Rx IP and TCP checksum offload Signed-off-by: Raju Lakkaraju Reported-by: kernel test robot --- Changes: =3D=3D=3D=3D=3D=3D=3D=3D V1 -> V2: - Fix the sparse warnings (>> drivers/net/ethernet/microchip/lan743x_main.c:2600:28: sparse: sparse: restricted __le32 degrades to integer) drivers/net/ethernet/microchip/lan743x_main.c | 14 +++++++++++++- drivers/net/ethernet/microchip/lan743x_main.h | 5 +++++ 2 files changed, 18 insertions(+), 1 deletion(-) diff --git a/drivers/net/ethernet/microchip/lan743x_main.c b/drivers/net/et= hernet/microchip/lan743x_main.c index 3f4e1ab63f8a..2599dfffd1da 100644 --- a/drivers/net/ethernet/microchip/lan743x_main.c +++ b/drivers/net/ethernet/microchip/lan743x_main.c @@ -1585,6 +1585,9 @@ static void lan743x_rfe_set_multicast(struct lan743x_= adapter *adapter) rfctl |=3D RFE_CTL_AM_; } =20 + if (netdev->features & NETIF_F_RXCSUM) + rfctl |=3D RFE_CTL_IP_COE_ | RFE_CTL_TCP_UDP_COE_; + memset(hash_table, 0, DP_SEL_VHF_HASH_LEN * sizeof(u32)); if (netdev_mc_count(netdev)) { struct netdev_hw_addr *ha; @@ -2547,6 +2550,7 @@ static int lan743x_rx_process_buffer(struct lan743x_r= x *rx) int result =3D RX_PROCESS_RESULT_NOTHING_TO_DO; struct lan743x_rx_buffer_info *buffer_info; int frame_length, buffer_length; + bool is_ice, is_tce, is_icsm; int extension_index =3D -1; bool is_last, is_first; struct sk_buff *skb; @@ -2593,6 +2597,9 @@ static int lan743x_rx_process_buffer(struct lan743x_r= x *rx) frame_length =3D RX_DESC_DATA0_FRAME_LENGTH_GET_(le32_to_cpu(descriptor->data0)); buffer_length =3D buffer_info->buffer_length; + is_ice =3D le32_to_cpu(descriptor->data1) & RX_DESC_DATA1_STATUS_ICE_; + is_tce =3D le32_to_cpu(descriptor->data1) & RX_DESC_DATA1_STATUS_TCE_; + is_icsm =3D le32_to_cpu(descriptor->data1) & RX_DESC_DATA1_STATUS_ICSM_; =20 netdev_dbg(netdev, "%s%schunk: %d/%d", is_first ? "first " : " ", @@ -2661,6 +2668,10 @@ static int lan743x_rx_process_buffer(struct lan743x_= rx *rx) if (is_last && rx->skb_head) { rx->skb_head->protocol =3D eth_type_trans(rx->skb_head, rx->adapter->netdev); + if (rx->adapter->netdev->features & NETIF_F_RXCSUM) { + if (!is_ice && !is_tce && !is_icsm) + skb->ip_summed =3D CHECKSUM_UNNECESSARY; + } netdev_dbg(netdev, "sending %d byte frame to OS", rx->skb_head->len); napi_gro_receive(&rx->napi, rx->skb_head); @@ -3383,7 +3394,8 @@ static int lan743x_pcidev_probe(struct pci_dev *pdev, =20 adapter->netdev->netdev_ops =3D &lan743x_netdev_ops; adapter->netdev->ethtool_ops =3D &lan743x_ethtool_ops; - adapter->netdev->features =3D NETIF_F_SG | NETIF_F_TSO | NETIF_F_HW_CSUM; + adapter->netdev->features =3D NETIF_F_SG | NETIF_F_TSO | + NETIF_F_HW_CSUM | NETIF_F_RXCSUM; adapter->netdev->hw_features =3D adapter->netdev->features; =20 /* carrier off reporting is important to ethtool even BEFORE open */ diff --git a/drivers/net/ethernet/microchip/lan743x_main.h b/drivers/net/et= hernet/microchip/lan743x_main.h index 58eb7abf976b..67877d3b6dd9 100644 --- a/drivers/net/ethernet/microchip/lan743x_main.h +++ b/drivers/net/ethernet/microchip/lan743x_main.h @@ -266,6 +266,8 @@ #define RFE_ADDR_FILT_LO(x) (0x404 + (8 * (x))) =20 #define RFE_CTL (0x508) +#define RFE_CTL_TCP_UDP_COE_ BIT(12) +#define RFE_CTL_IP_COE_ BIT(11) #define RFE_CTL_AB_ BIT(10) #define RFE_CTL_AM_ BIT(9) #define RFE_CTL_AU_ BIT(8) @@ -1121,6 +1123,9 @@ struct lan743x_tx_buffer_info { (((data0) & RX_DESC_DATA0_FRAME_LENGTH_MASK_) >> 16) #define RX_DESC_DATA0_EXT_ (0x00004000) #define RX_DESC_DATA0_BUF_LENGTH_MASK_ (0x00003FFF) +#define RX_DESC_DATA1_STATUS_ICE_ (0x00020000) +#define RX_DESC_DATA1_STATUS_TCE_ (0x00010000) +#define RX_DESC_DATA1_STATUS_ICSM_ (0x00000001) #define RX_DESC_DATA2_TS_NS_MASK_ (0x3FFFFFFF) =20 #if ((NET_IP_ALIGN !=3D 0) && (NET_IP_ALIGN !=3D 2)) --=20 2.25.1