From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ilias Apalodimas Subject: Re: [PATCH 0/4] RFC CPSW switchdev mode Date: Wed, 6 Jun 2018 09:42:25 +0300 Message-ID: <20180606064225.GA14283@apalos> References: <1527144984-31236-1-git-send-email-ilias.apalodimas@linaro.org> <4277c55c-bea2-1061-35e9-62d1d0f60d28@ti.com> <20180602140833.GA8574@lunn.ch> <85eb9f54-a4de-969e-4658-38878d481292@ti.com> <20180605235356.GC6237@lunn.ch> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Grygorii Strashko , netdev@vger.kernel.org, ivan.khoronzhuk@linaro.org, nsekhar@ti.com, jiri@resnulli.us, ivecera@redhat.com, francois.ozog@linaro.org, yogeshs@ti.com, spatton@ti.com To: Andrew Lunn Return-path: Received: from mail-wr0-f171.google.com ([209.85.128.171]:45637 "EHLO mail-wr0-f171.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932107AbeFFGmb (ORCPT ); Wed, 6 Jun 2018 02:42:31 -0400 Received: by mail-wr0-f171.google.com with SMTP id o12-v6so4914219wrm.12 for ; Tue, 05 Jun 2018 23:42:30 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20180605235356.GC6237@lunn.ch> Sender: netdev-owner@vger.kernel.org List-ID: Hi Andrew, >On Wed, Jun 06, 2018 at 01:53:56AM +0200, Andrew Lunn wrote: > > And I just have to look a little bit in the future as selected approach > > expected to be extended on future SoC (and other parts of existing SoCs - ICSS-G SW switch) > > where we going to have more features, like TSN, EST and packet Policing and Classification. > > You should probably took a closer look at the Mellonex driver. It has > a lot of TC offload, which is what policing and classification is. > I did take a close look to both Mellanox and rocker drivers before issuing this RFC and we seem to be close on the approach. What Grygorii is reffering to, is that for CBS to work properly on CPSW there *must* be a way to configure the CPU port individually. > TSN is being worked on in general, and i think the i210 is taking the > lead. So you probably want to keep an eye on that, and join the > discussion. > TSN is not just "one thing". It's a few IEEE standards bundled up to provide the needed functionality. i210 is only implementing CBS at the moment and there's an RFC out there to support what they call "Time based scheduling". I am already having discussions with Jesus on their current work. The idea behind using switchdev is that due to it's design, it's a very convenient way of utilizing netlink and iproute2/tc to configure any kind of future shapers. Since net_device_ops now has a callback to configure hardware schedulers being able to expose netdevs as hardware ports and configure them individually is great. As you can understand you'll end up with TSN capable switches and NICs being configured with the same commands from a userspace point of view. I am not sure this is the proper way to go, but at least for me, sounds like a viable solution. Thanks Ilias