Message ID | 20221112043141.13291-1-hkelam@marvell.com |
---|---|
Headers |
Return-Path: <linux-kernel-owner@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a5d:6687:0:0:0:0:0 with SMTP id l7csp1107467wru; Fri, 11 Nov 2022 20:33:14 -0800 (PST) X-Google-Smtp-Source: AA0mqf5nFXsF1A/PMYKMj4xkcxHs2Tj8K53D8ETnt/8ulPloakkVvF8e2pUujmbMKvDrDEvbq0OK X-Received: by 2002:a17:906:1746:b0:79f:9ff6:6576 with SMTP id d6-20020a170906174600b0079f9ff66576mr3948487eje.414.1668227594360; Fri, 11 Nov 2022 20:33:14 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1668227594; cv=none; d=google.com; s=arc-20160816; b=mp3lOf8rMO0Ips2F8cQpTl9cq//UtsSMUUl6sl5sIoiIvFrubAt+5hxPAXFgRpF9oo Vl5NUR7I2A2DWlIQVUR9DwllvBXtFJswnURFwAXBManQENKVDvPXG8Z4c4wCxbxx2jv4 /DQW2MaiqP86ko9/no4KvXgY0aCf/COzq8F299GJzDLyBZorW9FF/v0uKjDc0WbdUwLm KyAewpaVOlV2+jaZcX631qhxIntaNp8ngfpTyC2PFNzCtl7IsuZA7sdaESeusUpBq6Bl B78ImMmK5bx8KBNlree1+3S3PaKNlfnKUSRKKaX3u3AXiJYBB1FoOunOfS3FWOqLWeey 0crw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:mime-version:message-id:date:subject:cc:to:from :dkim-signature; bh=onc0iUOJCiEjMHdcph/EcvpHR1oCwbsiyzH3+kekBbg=; b=GlbdTUvqsqbOj84KF99Q3JuZhHc5jnB785hQ2/aMDZ5WWAROwKxfQ64rgxo6X/GkP8 Bl84kAlWOChYa/V/rNjih0MtcaKVaaqNO7sq0klhEl5CD906I7oIYz15Yla1VHT71h4I Px1AVl3K6gWIulx2qHt/iAAszTbBGqkbSsziavfcKy7LcxussoLtWsmE/m7nQTBX6dn5 gWZFEQmm9gRmWgZCktJhCRXc3gY33xxLNeeKu4lepKk7wkfvo8NuVjh4gK4umoD7zSoO K+Ccg0lnTy9NjCZtz8FW8UUgSV7LqtFJQOGHTkvnP6sJESiaWalq7yZlRfbv2GTm8sO3 Q0Mw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@marvell.com header.s=pfpt0220 header.b=DD9BTXme; 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; dmarc=pass (p=NONE sp=REJECT dis=NONE) header.from=marvell.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id nd7-20020a170907628700b0078dcc87b1c4si3885378ejc.923.2022.11.11.20.32.46; Fri, 11 Nov 2022 20:33:14 -0800 (PST) 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=@marvell.com header.s=pfpt0220 header.b=DD9BTXme; 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; dmarc=pass (p=NONE sp=REJECT dis=NONE) header.from=marvell.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234503AbiKLEcS (ORCPT <rfc822;winker.wchi@gmail.com> + 99 others); Fri, 11 Nov 2022 23:32:18 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:44766 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234262AbiKLEcD (ORCPT <rfc822;linux-kernel@vger.kernel.org>); Fri, 11 Nov 2022 23:32:03 -0500 Received: from mx0b-0016f401.pphosted.com (mx0b-0016f401.pphosted.com [67.231.156.173]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 35F075D697; Fri, 11 Nov 2022 20:32:02 -0800 (PST) Received: from pps.filterd (m0045851.ppops.net [127.0.0.1]) by mx0b-0016f401.pphosted.com (8.17.1.19/8.17.1.19) with ESMTP id 2AC4P1av017181; Fri, 11 Nov 2022 20:31:48 -0800 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=marvell.com; h=from : to : cc : subject : date : message-id : mime-version : content-type; s=pfpt0220; bh=onc0iUOJCiEjMHdcph/EcvpHR1oCwbsiyzH3+kekBbg=; b=DD9BTXmeK1dfmIiGfL2cPQcdjW0HBmpXVaKWTcU212QZbgOPcNrPJWdbdFSLcyWoV0ms rE5jXFNqLwL5dKJnVmzlSNNyNt/VITOjoUvpst07fG+A8/rpOonjYckOJmTkazcE5920 h92UYnw5Vu6tjQVTozYASSqMjxli8w/SpfpWcTtTthebHJL8t5Vppc1YoGkPmrvGuxqB o80+EZpyYt5Cw3Qm2iV7zehkAJauJyV2hQG1MG2TkpL3EFK3SUlYOWwFAvcNQjA98ji7 ieOW1miqHLMscV5/RFyJfd4EGpr1vCJX8321gcqvpJnuliL6UXyKc26Lc29TifpVrRcz Gw== Received: from dc5-exch02.marvell.com ([199.233.59.182]) by mx0b-0016f401.pphosted.com (PPS) with ESMTPS id 3kt1jn8by2-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-SHA384 bits=256 verify=NOT); Fri, 11 Nov 2022 20:31:48 -0800 Received: from DC5-EXCH02.marvell.com (10.69.176.39) by DC5-EXCH02.marvell.com (10.69.176.39) with Microsoft SMTP Server (TLS) id 15.0.1497.18; Fri, 11 Nov 2022 20:31:46 -0800 Received: from maili.marvell.com (10.69.176.80) by DC5-EXCH02.marvell.com (10.69.176.39) with Microsoft SMTP Server id 15.0.1497.18 via Frontend Transport; Fri, 11 Nov 2022 20:31:46 -0800 Received: from hyd1soter3.marvell.com (unknown [10.29.37.12]) by maili.marvell.com (Postfix) with ESMTP id D55A53F7043; Fri, 11 Nov 2022 20:31:42 -0800 (PST) From: Hariprasad Kelam <hkelam@marvell.com> To: <netdev@vger.kernel.org>, <linux-kernel@vger.kernel.org> CC: <kuba@kernel.org>, <davem@davemloft.net>, <pabeni@redhat.com>, <edumazet@google.com>, <sgoutham@marvell.com>, <lcherian@marvell.com>, <gakula@marvell.com>, <jerinj@marvell.com>, <sbhatta@marvell.com> Subject: [net-next PATCH 0/9] CN10KB MAC block support Date: Sat, 12 Nov 2022 10:01:32 +0530 Message-ID: <20221112043141.13291-1-hkelam@marvell.com> X-Mailer: git-send-email 2.17.1 MIME-Version: 1.0 Content-Type: text/plain X-Proofpoint-GUID: Y2EQ2b1TdexC4RsMVdYVi_E2o2chyRfU X-Proofpoint-ORIG-GUID: Y2EQ2b1TdexC4RsMVdYVi_E2o2chyRfU X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.219,Aquarius:18.0.895,Hydra:6.0.545,FMLib:17.11.122.1 definitions=2022-11-12_02,2022-11-11_01,2022-06-22_01 X-Spam-Status: No, score=-2.8 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_LOW,SPF_HELO_NONE, SPF_NONE 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: <linux-kernel.vger.kernel.org> X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1749263418393807896?= X-GMAIL-MSGID: =?utf-8?q?1749263418393807896?= |
Series |
CN10KB MAC block support
|
|
Message
Hariprasad Kelam
Nov. 12, 2022, 4:31 a.m. UTC
The nextgen silicon CN10KB supports new MAC block RPM2 and has a variable number of LMACS. This series of patches defines new mac_ops and configures csrs specific to new MAC. Defines new mailbox support to Reset LMAC stats, read FEC stats and set Physical state such that PF netdev can use mailbox support to use the features. Extends debugfs support for MAC block to show dropped packets by DMAC filters and show FEC stats Hariprasad Kelam (5): octeontx2-af: Support More number of MAC blocks octeontx2-af: CN10KB MAC RPM_100/USX support octeontx2-af: Reset MAC specific csrs on FLR octeontx2-af: Show count of dropped packets by DMAC filters octeontx2-af: Add support for RPM FEC stats Naveen Mamindlapalli (1): octeontx2-af: Add proper return codes for AF mbox handlers Revital Regev (1): octeontx2-af: Reset MAC block RX/TX statistics Vamsi Attunuru (1): octeontx2-af: physical link state change Vidhya Vidhyaraman (1): octeontx2-af: Add programmed macaddr to RVU pfvf .../net/ethernet/marvell/octeontx2/af/cgx.c | 169 ++++++++-- .../net/ethernet/marvell/octeontx2/af/cgx.h | 15 +- .../marvell/octeontx2/af/lmac_common.h | 19 +- .../net/ethernet/marvell/octeontx2/af/mbox.h | 8 + .../net/ethernet/marvell/octeontx2/af/rpm.c | 298 ++++++++++++++++-- .../net/ethernet/marvell/octeontx2/af/rpm.h | 42 ++- .../net/ethernet/marvell/octeontx2/af/rvu.c | 1 + .../net/ethernet/marvell/octeontx2/af/rvu.h | 14 +- .../ethernet/marvell/octeontx2/af/rvu_cgx.c | 193 +++++++++--- .../marvell/octeontx2/af/rvu_debugfs.c | 16 +- .../ethernet/marvell/octeontx2/af/rvu_nix.c | 10 +- .../marvell/octeontx2/af/rvu_npc_hash.c | 2 +- 12 files changed, 667 insertions(+), 120 deletions(-) -- 2.17.1
Comments
On Sat, 12 Nov 2022 10:01:32 +0530 Hariprasad Kelam wrote: > The nextgen silicon CN10KB supports new MAC block RPM2 > and has a variable number of LMACS. This series of patches > defines new mac_ops and configures csrs specific to new > MAC. > > Defines new mailbox support to Reset LMAC stats, read > FEC stats and set Physical state such that PF netdev > can use mailbox support to use the features. > > Extends debugfs support for MAC block to show dropped > packets by DMAC filters and show FEC stats I personally see no reason for us to keep merging your AF patches. Upstream is for working together and there is no synergy between your code, other drivers and the user APIs we build. Why not just keep it out of tree?
On Fri, Nov 11, 2022 at 09:12:35PM -0800, Jakub Kicinski wrote: > On Sat, 12 Nov 2022 10:01:32 +0530 Hariprasad Kelam wrote: > > The nextgen silicon CN10KB supports new MAC block RPM2 > > and has a variable number of LMACS. This series of patches > > defines new mac_ops and configures csrs specific to new > > MAC. > > > > Defines new mailbox support to Reset LMAC stats, read > > FEC stats and set Physical state such that PF netdev > > can use mailbox support to use the features. > > > > Extends debugfs support for MAC block to show dropped > > packets by DMAC filters and show FEC stats > > I personally see no reason for us to keep merging your AF patches. > Upstream is for working together and there is no synergy between > your code, other drivers and the user APIs we build. Why not just > keep it out of tree? Yes, see my comment about the ethtool .get_fec_stats. Maybe it is there, hidden amongst all the code, but it is not obvious. If you do want to stay in tree, may i suggest you move all your statistics in your debugfs to official kernel APIs, and then remove them from debugfs. This might require you work with the community to extend the current APIs, which is the synergy thing Jakub is taking about. Andrew
On Fri, Nov 11, 2022 at 09:12:35PM -0800, Jakub Kicinski wrote: > On Sat, 12 Nov 2022 10:01:32 +0530 Hariprasad Kelam wrote: > > The nextgen silicon CN10KB supports new MAC block RPM2 and has a > > variable number of LMACS. This series of patches defines new mac_ops > > and configures csrs specific to new MAC. > > > > Defines new mailbox support to Reset LMAC stats, read FEC stats and > > set Physical state such that PF netdev can use mailbox support to > > use the features. > > > > Extends debugfs support for MAC block to show dropped packets by > > DMAC filters and show FEC stats > > I personally see no reason for us to keep merging your AF patches. > Upstream is for working together and there is no synergy between your > code, other drivers and the user APIs we build. Why not just keep it > out of tree? Yes, see my comment about the ethtool .get_fec_stats. Maybe it is there, hidden amongst all the code, but it is not obvious. If you do want to stay in tree, may i suggest you move all your statistics in your debugfs to official kernel APIs, and then remove them from debugfs. This might require you work with the community to extend the current APIs, which is the synergy thing Jakub is taking about. Andrew This patch series adds two debugfs entries, one for FEC stats and other to show "Dropped packets by DMAC filters". For FEC stats will remove debugfs entry and integrate to ethtool API in the next version. For "Dropped packets by DMAC filters" did not find any equivalent APIs, so will keep it as is.
> For "Dropped packets by DMAC filters" did not find any equivalent APIs, so will keep it as is.
Nope. Propose a generic netlink API, probably as an extension to
ethtool, which other vendors can use as well.
This is exactly Jakub and my point. Is your hardware so special it
does things which no other vendor does? Is the concept of DMAC filters
dropping packets unique to your hardware? Is this feature so obscure
that no other vendor will ever implement it? And if it is that
obscure, is any user actually going to use it?
Andrew
> For "Dropped packets by DMAC filters" did not find any equivalent APIs, so will keep it as is. >> Nope. Propose a generic netlink API, probably as an extension to ethtool, which other vendors can use as well. ACK, will remove DMAC filters debugfs from this patch set and will submit this change with netlink API in separate set. This is exactly Jakub and my point. Is your hardware so special it does things which no other vendor does? Is the concept of DMAC filters dropping packets unique to your hardware? Is this feature so obscure that no other vendor will ever implement it? And if it is that obscure, is any user actually going to use it? Andrew