From patchwork Tue Jul 18 07:56:56 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Markus Schneider-Pargmann X-Patchwork-Id: 12193 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:c923:0:b0:3e4:2afc:c1 with SMTP id j3csp1589343vqt; Tue, 18 Jul 2023 01:28:59 -0700 (PDT) X-Google-Smtp-Source: APBJJlGO7E60vzFyNXazft8qPBUjqbKnyBTatVNmYAyixmdH7fLRWmzBXW4JGtL1z4l5tgp+aNah X-Received: by 2002:a17:907:6da2:b0:991:b292:695 with SMTP id sb34-20020a1709076da200b00991b2920695mr14140541ejc.55.1689668939650; Tue, 18 Jul 2023 01:28:59 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1689668939; cv=none; d=google.com; s=arc-20160816; b=iWjrfVdZwltEiDMIdJFKQneYG/Gbsu9NChVrrMQk4kYJvvAy4h18p7zRwgKDgN/H8F hDRwQlo4zMUgm6FWP05G8ULJrhqAjGnyDnSdXWDBKR4i5+LLkScz+CXnsF5TV6lbjQ+Z A9DpqtCCQGXB2cAuxauD+IpCHAuZ69ml7klKUFGx9YjRaP45mGoNVNrBgFKAK4bVzofQ Kd7V+84wx3s1TRXc5ZLGyHZ0aR3zNWEPqptC1ZzqHdI/YxntQyXXcD+irGBHAVJrXqsF 2wNWew0fGMmsIXgpl0cNTuyjmZbs6jE6jdcnExsKt6O9xSjSMlL3p3QyapUx+mrEc2vj Ys8g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:dkim-signature; bh=Yw8S8ohFAjJvqm8HRm1Ib4h2RNi+dhnA4IaPY526ysM=; fh=HEvpsrq2tYe3N9GuSJCCr6vVuOqDDDbQ2hrVJYmjlns=; b=b9bTe4AAw/dJt2NL1fWu306jATaRrVxeVc6GmHSXwgDMcBWb6OiBkIi63KzFycxhW6 I2eZ14ai7UerME/yrtq/7rUw1uoJkC465qDflvjhYloifDMMG/ijYysW1VnAFdRJh7HS axxY1zHRlMAUYL/FU1PCnpOSVj5pITgtXf0BLA3tOuI3ePRiAoUdYES0+r4G9AKsDY3x /h6rvKrUkfqhs9ZMRJ/i6cN9WXWFlAc931GyhTjnqVD4oWaGldiMpAWqt7np/GVRTx0U wWn9YejWxYqDEb2t7d0+FWAEkk7x+k/RMCpoH+T9TjiDUuvAZVS62RVQTK63lB4eD1yL 4SSQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@baylibre-com.20221208.gappssmtp.com header.s=20221208 header.b=xuotYMKh; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id b23-20020a170906195700b00993a37b5e5csi818509eje.394.2023.07.18.01.28.35; Tue, 18 Jul 2023 01:28:59 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@baylibre-com.20221208.gappssmtp.com header.s=20221208 header.b=xuotYMKh; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231511AbjGRH5n (ORCPT + 99 others); Tue, 18 Jul 2023 03:57:43 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:60012 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229847AbjGRH5i (ORCPT ); Tue, 18 Jul 2023 03:57:38 -0400 Received: from mail-lf1-x12c.google.com (mail-lf1-x12c.google.com [IPv6:2a00:1450:4864:20::12c]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 5A0771735 for ; Tue, 18 Jul 2023 00:57:14 -0700 (PDT) Received: by mail-lf1-x12c.google.com with SMTP id 2adb3069b0e04-4fdb3f66fd6so4520057e87.3 for ; Tue, 18 Jul 2023 00:57:14 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20221208.gappssmtp.com; s=20221208; t=1689667032; x=1690271832; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=Yw8S8ohFAjJvqm8HRm1Ib4h2RNi+dhnA4IaPY526ysM=; b=xuotYMKhWom8XlUQzbv1lWFQHoApOVmGWqonV/mG0O2JornO3adaNKalFzJSwG3xe9 HKBn/TVVQB2sZaQ9D9v48G7eGqizLOZcxgPyMPr7+fjzudxg+UXKbRK3Y87vD+k0tTfq 0FWaaou+5SFObSYd9wmn+4YCqHYt1m8bCMXzNxX60Xmm/R+T6ZL/NgEHNxGgqiSchNKI RzYLGXllSWwxyPhLzS7SMA5wp9EpH7kJA1RNoCNdR3hxgpa5Lzi4I81ak1A9DqWuAitA RhTVTBrzPBBYNaq7xK2ETPUWGIpaLQCHK4kD8DB911JpPi+iQxnsGIl3EwLv1gZp3rvc 52+Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1689667032; x=1690271832; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=Yw8S8ohFAjJvqm8HRm1Ib4h2RNi+dhnA4IaPY526ysM=; b=SSClqK//kqsQOYEjP2mOjlDfOIPObbjvYCJt3qDZcAqMrzcha5qmm277APKllFaoox /NECWobpbbpV/GlkInZeR+FMPVuVNwDpM4WZLIR76KtZk4ZQcL5deqvXKCN1KiCxXrUC df6Z/hBcKF4eaSOwSpE7+ZY3fk7C/5h9oadwlKVaUyOLFsOwJ/wcLHw9HQJljk2QMZb0 SGjhjrOLPqaUiRG20KpLdGyRU0JmTHA7TzQNkiHQJnjsJCxj+w8Pacm16LRoFLCfHHTq acLqYQ3ek/2JcC7N86535ueKokIFHbOoymMPKA9m7EAMLkSYBdKDH+5pgac3k6nNOW6A qkWw== X-Gm-Message-State: ABy/qLZJ23zeaJ+wv4yh+aa59y23pY/PyJegu2kJeMaqfb5PSx3sI7sn 7igreK9wutCFNoDMAU8dwe0y1A== X-Received: by 2002:ac2:4a87:0:b0:4f9:547c:a3cc with SMTP id l7-20020ac24a87000000b004f9547ca3ccmr8551001lfp.14.1689667032251; Tue, 18 Jul 2023 00:57:12 -0700 (PDT) Received: from blmsp.fritz.box ([2001:4091:a247:82fa:b762:4f68:e1ed:5041]) by smtp.gmail.com with ESMTPSA id x4-20020a5d54c4000000b003142439c7bcsm1585959wrv.80.2023.07.18.00.57.11 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 18 Jul 2023 00:57:11 -0700 (PDT) From: Markus Schneider-Pargmann To: Marc Kleine-Budde , Chandrasekar Ramakrishnan , Wolfgang Grandegger Cc: Vincent MAILHOL , Simon Horman , "David S . Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , linux-can@vger.kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, Julien Panis , Markus Schneider-Pargmann Subject: [PATCH v5 00/12] can: m_can: Optimizations for m_can/tcan part 2 Date: Tue, 18 Jul 2023 09:56:56 +0200 Message-Id: <20230718075708.958094-1-msp@baylibre.com> X-Mailer: git-send-email 2.40.1 MIME-Version: 1.0 X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS, T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1771746298147130569 X-GMAIL-MSGID: 1771746298147130569 Hi Marc, Simon and everyone, v5 got a rebase on v6.5 with some small style fixes as pointed out in v4. It is tested on tcan455x but I don't have hardware with mcan on the SoC myself so any testing is appreciated. The series implements many small and bigger throughput improvements and adds rx/tx coalescing at the end. Based on v6.5-rc1. Also available at https://gitlab.baylibre.com/msp8/linux/-/tree/topic/mcan-optimization/v6.5?ref_type=heads Best, Markus Changes in v5: - Add back parenthesis in m_can_set_coalesce(). This will make checkpatch unhappy but gcc happy. - Remove unused fifo_header variable in m_can_tx_handler(). - Rebased to v6.5-rc1 Changes in v4: - Create and use struct m_can_fifo_element in m_can_tx_handler - Fix memcpy_and_pad to copy the full buffer - Fixed a few checkpatch warnings - Change putidx to be unsigned - Print hard_xmit error only once when TX FIFO is full Changes in v3: - Remove parenthesis in error messages - Use memcpy_and_pad for buffer copy in 'can: m_can: Write transmit header and data in one transaction'. - Replace spin_lock with spin_lock_irqsave. I got a report of a interrupt that was calling start_xmit just after the netqueue was woken up before the locked region was exited. spin_lock_irqsave should fix this. I attached the full stack at the end of the mail if someone wants to know. - Rebased to v6.3-rc1. - Removed tcan4x5x patches from this series. Changes in v2: - Rebased on v6.2-rc5 - Fixed missing/broken accounting for non peripheral m_can devices. previous versions: v1 - https://lore.kernel.org/lkml/20221221152537.751564-1-msp@baylibre.com v2 - https://lore.kernel.org/lkml/20230125195059.630377-1-msp@baylibre.com v3 - https://lore.kernel.org/lkml/20230315110546.2518305-1-msp@baylibre.com/ v4 - https://lore.kernel.org/lkml/20230621092350.3130866-1-msp@baylibre.com/ Markus Schneider-Pargmann (12): can: m_can: Write transmit header and data in one transaction can: m_can: Implement receive coalescing can: m_can: Implement transmit coalescing can: m_can: Add rx coalescing ethtool support can: m_can: Add tx coalescing ethtool support can: m_can: Use u32 for putidx can: m_can: Cache tx putidx can: m_can: Use the workqueue as queue can: m_can: Introduce a tx_fifo_in_flight counter can: m_can: Use tx_fifo_in_flight for netif_queue control can: m_can: Implement BQL can: m_can: Implement transmit submission coalescing drivers/net/can/m_can/m_can.c | 517 +++++++++++++++++++++++++--------- drivers/net/can/m_can/m_can.h | 35 ++- 2 files changed, 418 insertions(+), 134 deletions(-) base-commit: 06c2afb862f9da8dc5efa4b6076a0e48c3fbaaa5