Received: by 2002:a05:6a10:17d3:0:0:0:0 with SMTP id hz19csp1418304pxb; Sun, 11 Apr 2021 19:21:41 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzJnDNyWdCkruFB6kxooQDjS2Xoq+sLkVcABcSiBp5Qw13FdrLIMPLwSuPvvDSfMzDzmUrb X-Received: by 2002:a50:8d8a:: with SMTP id r10mr14260311edh.132.1618194101269; Sun, 11 Apr 2021 19:21:41 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1618194101; cv=none; d=google.com; s=arc-20160816; b=gt4HIOzVPbm2GpWzMyWFTaa9xepX2Bk8CRbn6E8dx1dPZ3HxpzrJzXTPMvYOPhZbG9 4y/sL7R8DMGxlvksO2ZYC0NoklrhRlE4wWnlxUT0epHgYiTiu+0aahl6hS5A5EM8Q4Q8 mVj1NhCMu+zEq71f3TIxjA79QWj/b5FFNvmGte2cWVnYf8J2IVMIOg9fhsD3EHVgWRZV /mF1cs8w03mi6ARhSInVF7tisPgep26ejr+5vSCMFNlOWICIIEIow9L8EiENbsHMdQrf 8ct7hY+Ds7ZM79iZDOCK9RWVFUjOIdeLeXmJH2MEcVujuIdJ3EkGsQ2Ji0gUJX+Kd73g wIFA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:content-language :in-reply-to:mime-version:user-agent:date:message-id:from:references :cc:to:subject:dkim-signature; bh=yaVUjVzZXG//isVveNusfn2LxuJbXyw20jQCaiyjYIc=; b=VJoC7qTepv4mLiWDmsJsqpDjZupMNThk1CtTfUuvDS2rFB8iXMbq7yFM4ayg6gzm5S 2Zr8sqrv/YiO9Hp8EOkKtomWznQS2F6h31Da6nwGyjDlZpvFwpx1kgxERih2fASPkebx zxaUkd9fi/b88kz82EvdL9cf+8aKUzv9BOOwRT1xkOVJvR3GW/UIf1Prhnl/bfQ+k5J3 laLwQVHI8pvWtcze1Cd8EdMSuLtVl23n9B9ylAyBtZ2cbvPXDRWSYb4Yz345k+hfuhpy g/J20cNZ24VrgDL2H1ufB/Zy2aRmLp9hVraCHXeeH8PHaECuaAkpgkqhYJP4ZH5Zw9Co xf1A== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=g+A+zCWw; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id e26si6479924edv.198.2021.04.11.19.21.18; Sun, 11 Apr 2021 19:21:41 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=g+A+zCWw; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236463AbhDLCLI (ORCPT + 99 others); Sun, 11 Apr 2021 22:11:08 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:34354 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235543AbhDLCLH (ORCPT ); Sun, 11 Apr 2021 22:11:07 -0400 Received: from mail-pl1-x62e.google.com (mail-pl1-x62e.google.com [IPv6:2607:f8b0:4864:20::62e]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D582DC061574; Sun, 11 Apr 2021 19:10:50 -0700 (PDT) Received: by mail-pl1-x62e.google.com with SMTP id p16so1586749plf.12; Sun, 11 Apr 2021 19:10:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=yaVUjVzZXG//isVveNusfn2LxuJbXyw20jQCaiyjYIc=; b=g+A+zCWwzK8ZAyXoHM/Ve1WTT/2MDH7nWwtxrikbn+RKWzXglEh05nwTduS4mBiQ2Q LNajmxu/Y0PuNDWOAd8TIMWtrEbC6hQLt1W5LuLEyU36NwXSP9tDRNnzxidIqHdsHLSg P6WxljovJLA16cW+C/8RkYO2fq6wRurjRAw6NBjACmNFqlzHL+xtzOofT9VOnqJbt2J+ ACE9aFC7hGBDO/EpTn6YDdfyLan04ZncrLwTKdmaA1NYmaVbHR07KYvZ0F+MrHnJJJ1n fsC9NSDf4f56zkegwM8L2Wv9V/WqxbRC9UQjFw6OhMoRoRH9JpKyWiqAYKolqAuLWHoq 7eBg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=yaVUjVzZXG//isVveNusfn2LxuJbXyw20jQCaiyjYIc=; b=Fy8vau9OScnKW444Xju85uyKSj3uz1BuvwFs97/R+Qd9y8BYP2aFDyDjxJva3eaZiH C/IEwz6NhCcu1zT0xiWdvqe/daHqZyI1V2FXd7gDzkC3Mr1S8OXMilmreTJZEG9FaWX1 pKnmSNyvwUfyW9XWFSlJ6GxCUIAVKZYYDRCCekznRASV5brQdCq1uWxmIu07zcOMsB4S uGLn6C6legpSPtWnVF45/BLPIbKf4eKaVgNesMdGi7N8cRex9G4UFdw79u+lLiGcyWyl UJz0BRyBNV5jqxBpb89mSyPK9T8aZJtGBEU+8IRKD/uUnOtogMAsIU7tc6O+t/MWLc6u rzCQ== X-Gm-Message-State: AOAM530zBNjZ4WtIJvV0XAsx2Dp+GhGJxPQUdNi1WBEgo6Seseuh7mGr F1R+D7YMDvs6JctUK8ofNr8KEVYmW5o= X-Received: by 2002:a17:90b:2317:: with SMTP id mt23mr10290838pjb.24.1618193449717; Sun, 11 Apr 2021 19:10:49 -0700 (PDT) Received: from [10.230.2.159] ([192.19.228.250]) by smtp.gmail.com with ESMTPSA id j10sm8912785pjs.11.2021.04.11.19.10.39 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sun, 11 Apr 2021 19:10:49 -0700 (PDT) Subject: Re: [PATCH RFC net-next 0/3] Multi-CPU DSA support To: Vladimir Oltean , Marek Behun Cc: Ansuel Smith , netdev@vger.kernel.org, "David S. Miller" , Jakub Kicinski , Andrew Lunn , Vivien Didelot , Alexei Starovoitov , Daniel Borkmann , Andrii Nakryiko , Eric Dumazet , Wei Wang , Cong Wang , Taehee Yoo , =?UTF-8?B?QmrDtnJuIFTDtnBlbA==?= , zhang kai , Weilong Chen , Roopa Prabhu , Di Zhu , Francis Laniel , linux-kernel@vger.kernel.org References: <20210410133454.4768-1-ansuelsmth@gmail.com> <20210411200135.35fb5985@thinkpad> <20210411185017.3xf7kxzzq2vefpwu@skbuf> <20210411235358.vpql2mppobjhknfg@skbuf> From: Florian Fainelli Message-ID: <84a27330-73cb-3ebe-a0a9-eb6afec55902@gmail.com> Date: Sun, 11 Apr 2021 19:10:36 -0700 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101 Firefox/78.0 Thunderbird/78.9.0 MIME-Version: 1.0 In-Reply-To: <20210411235358.vpql2mppobjhknfg@skbuf> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 4/11/2021 4:53 PM, Vladimir Oltean wrote: > On Sun, Apr 11, 2021 at 09:50:17PM +0300, Vladimir Oltean wrote: >> On Sun, Apr 11, 2021 at 08:01:35PM +0200, Marek Behun wrote: >>> On Sat, 10 Apr 2021 15:34:46 +0200 >>> Ansuel Smith wrote: >>> >>>> Hi, >>>> this is a respin of the Marek series in hope that this time we can >>>> finally make some progress with dsa supporting multi-cpu port. >>>> >>>> This implementation is similar to the Marek series but with some tweaks. >>>> This adds support for multiple-cpu port but leave the driver the >>>> decision of the type of logic to use about assigning a CPU port to the >>>> various port. The driver can also provide no preference and the CPU port >>>> is decided using a round-robin way. >>> >>> In the last couple of months I have been giving some thought to this >>> problem, and came up with one important thing: if there are multiple >>> upstream ports, it would make a lot of sense to dynamically reallocate >>> them to each user port, based on which user port is actually used, and >>> at what speed. >>> >>> For example on Turris Omnia we have 2 CPU ports and 5 user ports. All >>> ports support at most 1 Gbps. Round-robin would assign: >>> CPU port 0 - Port 0 >>> CPU port 1 - Port 1 >>> CPU port 0 - Port 2 >>> CPU port 1 - Port 3 >>> CPU port 0 - Port 4 >>> >>> Now suppose that the user plugs ethernet cables only into ports 0 and 2, >>> with 1, 3 and 4 free: >>> CPU port 0 - Port 0 (plugged) >>> CPU port 1 - Port 1 (free) >>> CPU port 0 - Port 2 (plugged) >>> CPU port 1 - Port 3 (free) >>> CPU port 0 - Port 4 (free) >>> >>> We end up in a situation where ports 0 and 2 share 1 Gbps bandwidth to >>> CPU, and the second CPU port is not used at all. >>> >>> A mechanism for automatic reassignment of CPU ports would be ideal here. >>> >>> What do you guys think? >> >> The reason why I don't think this is such a great idea is because the >> CPU port assignment is a major reconfiguration step which should at the >> very least be done while the network is down, to avoid races with the >> data path (something which this series does not appear to handle). >> And if you allow the static user-port-to-CPU-port assignment to change >> every time a link goes up/down, I don't think you really want to force >> the network down through the entire switch basically. >> >> So I'd be tempted to say 'tough luck' if all your ports are not up, and >> the ones that are are assigned statically to the same CPU port. It's a >> compromise between flexibility and simplicity, and I would go for >> simplicity here. That's the most you can achieve with static assignment, >> just put the CPU ports in a LAG if you want better dynamic load balancing >> (for details read on below). > > Just one more small comment, because I got so carried away with > describing what I already had in mind, that I forgot to completely > address your idea. > > I think that DSA should provide the means to do what you want but not > the policy. Could not agree more, this point is what has historically prevented any multi-CPU port patch series from landing because what everyone seems to have wanted so far is along these lines: - assign LAN ports 0-3 to CPU port #0 - assign WAN port 4 to CPU port #1 and do that from Device Tree, problem solved? Not entirely unfortunately. Being able to change the mapping via iproute2 is definitively an improvement, and to echo to your comment on the iproute2 change proper we can try to agree on a more specialized syntax. > Meaning that you can always write a user space program that > monitors the NETLINK_ROUTE rtnetlink through a socket and listens for > link state change events on it with poll(), then does whatever (like > moves the static user-to-CPU port mapping in the way that is adequate to > your network's requirements). The link up/down events are already > emitted, and the patch set here gives user space the rope to hang itself. That seems like an entirely reasonable approach to me, and solving how to map a given user-port to a particular CPU port definitively belongs in user-space, within the constraints expressed by what the switch driver can do of course. > > If you need inspiration, one user of the rtnetlink socket that I know of > is ptp4l: > https://github.com/richardcochran/linuxptp/blob/master/rtnl.c > -- Florian