From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ilias Apalodimas Subject: Re: [RFC v2, net-next, PATCH 4/4] net/cpsw_switchdev: add switchdev mode of operation on cpsw driver Date: Wed, 20 Jun 2018 21:03:42 +0300 Message-ID: <20180620180342.GA28303@apalos> References: <1528974690-31600-1-git-send-email-ilias.apalodimas@linaro.org> <1528974690-31600-5-git-send-email-ilias.apalodimas@linaro.org> <20180618161627.GC5865@lunn.ch> <20180618201940.GA5890@apalos> <671c83ea-6227-cc09-e604-14cfa6804726@redhat.com> <20180620175128.GA27235@apalos> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Ivan Vecera , Andrew Lunn , netdev@vger.kernel.org, grygorii.strashko@ti.com, ivan.khoronzhuk@linaro.org, nsekhar@ti.com, jiri@resnulli.us, francois.ozog@linaro.org, yogeshs@ti.com, spatton@ti.com, Jose.Abreu@synopsys.com To: Florian Fainelli Return-path: Received: from mail-wm0-f65.google.com ([74.125.82.65]:35491 "EHLO mail-wm0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754144AbeFTSDr (ORCPT ); Wed, 20 Jun 2018 14:03:47 -0400 Received: by mail-wm0-f65.google.com with SMTP id j15-v6so1144736wme.0 for ; Wed, 20 Jun 2018 11:03:47 -0700 (PDT) Content-Disposition: inline In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: Hi Florian, On Wed, Jun 20, 2018 at 10:58:26AM -0700, Florian Fainelli wrote: > On 06/20/2018 10:51 AM, Ilias Apalodimas wrote: > > Hello Ivan, > > On Wed, Jun 20, 2018 at 02:56:48PM +0200, Ivan Vecera wrote: > >> On 18.6.2018 22:19, Ilias Apalodimas wrote: > >>> Jiri proposed using devlink, which makes sense, but i am not sure it's > >>> applicable on this patchset. This will change the driver completely and will > >>> totally break backwards compatibility. > >> > >> Another good reason for a new driver. > >> > >> I. > > This is actually conflicting at least to my understanding. Jiri proposed using > > devlink was used as an alternative method to enable a new mode instead of > > adding it on a .config option. A new driver wouldn't have a need for that right? > > Correct, with a new driver would likely behave correctly upon being > probed such that you could have your switch ports act as normal network > devices from which you could run IP-config and do NFS boot. The current driver also does NFS properly and the 2 ethernet ports act as normal network interfaces. The NFS section in the cover letter is to cover the cases were users running on NFS need to change the running switch configuration(starting from adding the 2 interfaces on a bridge). Since iproute2 is located on the NFS filesystem the moment network connectivity is lost, you loose the ability to perform further configuration and in certian configuration scenarios render the device unusable. > -- > Florian Thanks Ilias