From mboxrd@z Thu Jan 1 00:00:00 1970 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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 17ED1C3DA45 for ; Fri, 12 Jul 2024 15:00:46 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:MIME-Version:References:In-Reply-To:Message-ID:Subject:Cc:To: From:Date:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=7yczoF2T7fZER67nVN0JjdvJgbh38GhSzyKI+ok5zbY=; b=rWKS2OO1jnVYCj1zZmbiIhXVP2 munzhgoF+T7xVnBPjPrXU3xk7QkgN6X1+3AqyJwoG5YuTC1pqg7p14K4D4/WWKlRVpZ7C2TN6Iosb nun7RXJIxG656mfprhwF3WdICGATHLVEjs7URtzsMJgEuCjYOoBD9yWtY0QaCupMr0FV9ntJfizYN lvx9o1BNnFk91zKLRj/K9Hy6ZtFNqxhA/a60zLb1IuabjBkjeFr8x+oW4KGQBfKR6yne14ePnO/2V x1b4qsP6hYGnG0fwrtueMGNRNy9l0YKFJYNgCx4Vw/0mFzMgJnbDwryEIsloNEe2hFRqzU5qjca9b DumOAeNg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sSHlC-00000000R7t-0RUo; Fri, 12 Jul 2024 15:00:34 +0000 Received: from sin.source.kernel.org ([145.40.73.55]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sSHkm-00000000Qwr-1l5T for linux-arm-kernel@lists.infradead.org; Fri, 12 Jul 2024 15:00:12 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by sin.source.kernel.org (Postfix) with ESMTP id 08F8FCE1B01; Fri, 12 Jul 2024 15:00:06 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 7831CC32782; Fri, 12 Jul 2024 15:00:03 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1720796404; bh=pLkYVWWa4KVYmUB7SuThwAZhsHjrcNB+hX34mYYBkrU=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=G94/R3lLY3+LV0geoU6JHl7pjmqr4IPDSqahqtDnYG5cqNo7tvR/0Njx7CZjJNe+b 9oDsJlmmJdcxM63FbL81CdKuHdLHftDAT396o+nG8xHeQGRgbsAe2ywqRgWzIeNVux jEnkiCRB0xibLSL5t5tkRora7AFfW4mUHKLIkFoO34hZabrf6cJwqjxLZqc64NCVtP IQA8h7yKQV7odGEDD/0NF/K4f4DPjlGm5/SRe4B4oARWtTnXahSgVdUKJHHy/qBte8 RhjXbIiye4W5LFb0x9gefTJEGUVrDL5p9gvK+j+CeT2+aX1ltjzJazQBI+D9hGa5nL kr4T/eV5BJlFg== Date: Fri, 12 Jul 2024 08:00:02 -0700 From: Jakub Kicinski To: Lorenzo Bianconi Cc: netdev@vger.kernel.org, nbd@nbd.name, lorenzo.bianconi83@gmail.com, davem@davemloft.net, edumazet@google.com, pabeni@redhat.com, conor@kernel.org, linux-arm-kernel@lists.infradead.org, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org, devicetree@vger.kernel.org, catalin.marinas@arm.com, will@kernel.org, upstream@airoha.com, angelogioacchino.delregno@collabora.com, benjamin.larsson@genexis.eu, rkannoth@marvell.com, sgoutham@marvell.com, andrew@lunn.ch, arnd@arndb.de, horms@kernel.org Subject: Re: [PATCH v7 net-next 2/2] net: airoha: Introduce ethernet support for EN7581 SoC Message-ID: <20240712080002.37c11d02@kernel.org> In-Reply-To: References: <8ca603f8cea1ad64b703191b4c780bab87cb7dff.1720600905.git.lorenzo@kernel.org> <20240711181003.4089a633@kernel.org> <20240712072819.4f43062c@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240712_080008_932366_0AD22859 X-CRM114-Status: GOOD ( 20.71 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Fri, 12 Jul 2024 16:43:58 +0200 Lorenzo Bianconi wrote: > > On Fri, 12 Jul 2024 15:47:38 +0200 Lorenzo Bianconi wrote: > > > The Airoha eth SoC architecture is similar to mtk_eth_soc one (e.g MT7988a). > > > The FrameEngine (FE) module has multiple GDM ports that are connected to > > > different blocks. Current airoha_eth driver supports just GDM1 that is connected > > > to a MT7530 DSA switch (I have not posted a tiny patch for mt7530 driver yet). > > > In the future we will support even GDM{2,3,4} that will connect to differ > > > phy modues (e.g. 2.5Gbps phy). > > > > What I'm confused by is the mentioned of DSA. You put the port in the > > descriptor, and there can only be one switch on the other side, right? > > do you mean fport in msg1 (airoha_dev_xmit())? > > fport = port->id == 4 ? FE_PSE_PORT_GDM4 : port->id; > msg1 = FIELD_PREP(QDMA_ETH_TXMSG_FPORT_MASK, fport) | > ... > > fport refers to the GDM port and not to the dsa user port. Am I missing > something? Ooh, I see, reading what you explained previously now makes sense. So only 1 of the ports goes to the DSA switch, and the other ones are connected to SoC pins? A diagram would be worth a 1000 words ;) > > be in a setup like this :( It will have no way to figure out the real > > egress rate given that each netdev only sees a (non-)random sample > > of traffic sharing the queue :( > > do you prefer to remove BQL support? No strong preference, I worry it will do more harm than good in this case. It's not what it's designed for basically. But without testing it's all speculation, so up to you, users can always disable using sysfs.