From patchwork Wed Mar 31 15:41:29 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ong Boon Leong X-Patchwork-Id: 12175631 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-11.8 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id EB5F4C433ED for ; Wed, 31 Mar 2021 15:38:29 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id BD9D461019 for ; Wed, 31 Mar 2021 15:38:29 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235706AbhCaPh4 (ORCPT ); Wed, 31 Mar 2021 11:37:56 -0400 Received: from mga03.intel.com ([134.134.136.65]:34667 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235520AbhCaPh0 (ORCPT ); Wed, 31 Mar 2021 11:37:26 -0400 IronPort-SDR: 0ZDlpzQwoAAcASMccIhJbjjPByNCu/5rNQK+h+iC2XJEoUR2KIZp1uDJ5osUSmfDsngJANJaus coQyH141kfUQ== X-IronPort-AV: E=McAfee;i="6000,8403,9940"; a="192054455" X-IronPort-AV: E=Sophos;i="5.81,293,1610438400"; d="scan'208";a="192054455" Received: from orsmga008.jf.intel.com ([10.7.209.65]) by orsmga103.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 31 Mar 2021 08:37:26 -0700 IronPort-SDR: bFLVobjcz36FiQAfEulhXkLO+PD+DqUOlkEpEmCQxJrLSR9BjRjPgUnK4afBfDx0Rdti0RG/UP kFByFLIJdbLg== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.81,293,1610438400"; d="scan'208";a="418729037" Received: from glass.png.intel.com ([10.158.65.59]) by orsmga008.jf.intel.com with ESMTP; 31 Mar 2021 08:37:20 -0700 From: Ong Boon Leong To: Giuseppe Cavallaro , Alexandre Torgue , Jose Abreu , "David S . Miller" , Jakub Kicinski , Alexei Starovoitov , Daniel Borkmann , Jesper Dangaard Brouer , John Fastabend Cc: Maxime Coquelin , Andrii Nakryiko , Martin KaFai Lau , Song Liu , Yonghong Song , KP Singh , netdev@vger.kernel.org, linux-stm32@st-md-mailman.stormreply.com, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, bpf@vger.kernel.org, Ong Boon Leong Subject: [PATCH net-next v3 0/6] stmmac: Add XDP support Date: Wed, 31 Mar 2021 23:41:29 +0800 Message-Id: <20210331154135.8507-1-boon.leong.ong@intel.com> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: bpf@vger.kernel.org X-Patchwork-Delegate: kuba@kernel.org Hi, This is the v3 patch series for adding XDP support to stmmac driver. Summary of the changes in v3 (per feedback from Jakub Kicinski):- 4/6: Factor in XDP buffer header and tail adjustment by XDP prog. 5/6: Added 'nq->trans_start = jiffies' to avoid TX time-out for XDP_TX. 6/6: Added 'nq->trans_start = jiffies' to avoid TX time-out for ndo_xdp_xmit. I retested this patch series on all the test steps listed in v1 and the results look good as expected. I also used xdp_adjust_tail test app in samples/bpf for checking out XDP head and tail adjustment introduced in 4/6 and the result below looks correct too. ######################################################################## DUT > root@intel-corei7-64:~ $ ./xdp_adjust_tail -i eth0 -P 400 -N ========================== icmp "packet too big" sent: 0 pkts icmp "packet too big" sent: 0 pkts icmp "packet too big" sent: 0 pkts icmp "packet too big" sent: 0 pkts icmp "packet too big" sent: 1 pkts icmp "packet too big" sent: 1 pkts icmp "packet too big" sent: 1 pkts icmp "packet too big" sent: 2 pkts icmp "packet too big" sent: 4 pkts icmp "packet too big" sent: 6 pkts icmp "packet too big" sent: 8 pkts icmp "packet too big" sent: 9 pkts icmp "packet too big" sent: 10 pkts icmp "packet too big" sent: 10 pkts LP > root@intel-corei7-64:~# ping 169.254.1.11 -s 300 PING 169.254.1.11 (169.254.1.11) 300(328) bytes of data. 308 bytes from 169.254.1.11: icmp_seq=1 ttl=64 time=1.17 ms 308 bytes from 169.254.1.11: icmp_seq=2 ttl=64 time=0.575 ms 308 bytes from 169.254.1.11: icmp_seq=3 ttl=64 time=0.582 ms 308 bytes from 169.254.1.11: icmp_seq=4 ttl=64 time=0.595 ms 308 bytes from 169.254.1.11: icmp_seq=5 ttl=64 time=0.585 ms 308 bytes from 169.254.1.11: icmp_seq=6 ttl=64 time=0.591 ms 308 bytes from 169.254.1.11: icmp_seq=7 ttl=64 time=0.599 ms --- 169.254.1.11 ping statistics --- 7 packets transmitted, 7 received, 0% packet loss, time 6103ms rtt min/avg/max/mdev = 0.575/0.670/1.166/0.202 ms LP > root@intel-corei7-64:~# ping 169.254.1.11 -s 500 PING 169.254.1.11 (169.254.1.11) 500(528) bytes of data. From 169.254.1.11 icmp_seq=1 Frag needed and DF set (mtu = 436) From 169.254.1.11 icmp_seq=2 Frag needed and DF set (mtu = 436) From 169.254.1.11 icmp_seq=3 Frag needed and DF set (mtu = 436) From 169.254.1.11 icmp_seq=4 Frag needed and DF set (mtu = 436) From 169.254.1.11 icmp_seq=5 Frag needed and DF set (mtu = 436) From 169.254.1.11 icmp_seq=6 Frag needed and DF set (mtu = 436) ######################################################################## History of the previous patch series: v2: https://patchwork.kernel.org/project/netdevbpf/list/?series=457757 v1: https://patchwork.kernel.org/project/netdevbpf/list/?series=457139 It will be great if community can help to test or review the v3 patch series on your platform and provide me any new feedback if any. Thank you very much. Boon Leong Ong Boon Leong (6): net: stmmac: set IRQ affinity hint for multi MSI vectors net: stmmac: make SPH enable/disable to be configurable net: stmmac: arrange Tx tail pointer update to stmmac_flush_tx_descriptors net: stmmac: Add initial XDP support net: stmmac: Add support for XDP_TX action net: stmmac: Add support for XDP_REDIRECT action drivers/net/ethernet/stmicro/stmmac/Makefile | 1 + drivers/net/ethernet/stmicro/stmmac/stmmac.h | 35 +- .../net/ethernet/stmicro/stmmac/stmmac_main.c | 529 +++++++++++++++--- .../net/ethernet/stmicro/stmmac/stmmac_xdp.c | 40 ++ .../net/ethernet/stmicro/stmmac/stmmac_xdp.h | 12 + 5 files changed, 537 insertions(+), 80 deletions(-) create mode 100644 drivers/net/ethernet/stmicro/stmmac/stmmac_xdp.c create mode 100644 drivers/net/ethernet/stmicro/stmmac/stmmac_xdp.h