Re: [PATCH net-next v2 00/10] net: dsa: add generic debugfs interface

2017-08-30 Thread Jiri Pirko
Tue, Aug 29, 2017 at 02:50:04PM CEST, and...@lunn.ch wrote: >On Tue, Aug 29, 2017 at 08:25:23AM +0200, Jiri Pirko wrote: >> Mon, Aug 28, 2017 at 10:08:34PM CEST, and...@lunn.ch wrote: >> >> I see this overlaps a lot with DPIPE. Why won't you use that to expose >> >> your hw state? >> > >> >We took

Re: [PATCH net-next v2 00/10] net: dsa: add generic debugfs interface

2017-08-30 Thread Jiri Pirko
Tue, Aug 29, 2017 at 05:57:54PM CEST, vivien.dide...@savoirfairelinux.com wrote: >Hi David, Jiri, > >Jiri Pirko writes: > >> Tue, Aug 29, 2017 at 06:38:37AM CEST, da...@davemloft.net wrote: >>>From: Vivien Didelot >>>Date: Mon, 28 Aug 2017 15:17:38 -0400 >>> This patch series adds a generic

Re: [PATCH net-next v2 00/10] net: dsa: add generic debugfs interface

2017-08-29 Thread Andrew Lunn
On Tue, Aug 29, 2017 at 12:19:08PM -0700, Florian Fainelli wrote: > On 08/29/2017 12:05 PM, Arkadi Sharshevsky wrote: > > > > > > On 08/29/2017 03:50 PM, Andrew Lunn wrote: > >> On Tue, Aug 29, 2017 at 08:25:23AM +0200, Jiri Pirko wrote: > >>> Mon, Aug 28, 2017 at 10:08:34PM CEST, and...@lunn.ch

Re: [PATCH net-next v2 00/10] net: dsa: add generic debugfs interface

2017-08-29 Thread Florian Fainelli
On 08/29/2017 12:05 PM, Arkadi Sharshevsky wrote: > > > On 08/29/2017 03:50 PM, Andrew Lunn wrote: >> On Tue, Aug 29, 2017 at 08:25:23AM +0200, Jiri Pirko wrote: >>> Mon, Aug 28, 2017 at 10:08:34PM CEST, and...@lunn.ch wrote: > I see this overlaps a lot with DPIPE. Why won't you use that to e

Re: [PATCH net-next v2 00/10] net: dsa: add generic debugfs interface

2017-08-29 Thread Arkadi Sharshevsky
On 08/29/2017 03:50 PM, Andrew Lunn wrote: > On Tue, Aug 29, 2017 at 08:25:23AM +0200, Jiri Pirko wrote: >> Mon, Aug 28, 2017 at 10:08:34PM CEST, and...@lunn.ch wrote: I see this overlaps a lot with DPIPE. Why won't you use that to expose your hw state? >>> >>> We took a look at dpipe a

Re: [PATCH net-next v2 00/10] net: dsa: add generic debugfs interface

2017-08-29 Thread Vivien Didelot
Hi David, Jiri, Jiri Pirko writes: > Tue, Aug 29, 2017 at 06:38:37AM CEST, da...@davemloft.net wrote: >>From: Vivien Didelot >>Date: Mon, 28 Aug 2017 15:17:38 -0400 >> >>> This patch series adds a generic debugfs interface for the DSA >>> framework, so that all switch devices benefit from it, e

Re: [PATCH net-next v2 00/10] net: dsa: add generic debugfs interface

2017-08-29 Thread Andrew Lunn
On Tue, Aug 29, 2017 at 08:25:23AM +0200, Jiri Pirko wrote: > Mon, Aug 28, 2017 at 10:08:34PM CEST, and...@lunn.ch wrote: > >> I see this overlaps a lot with DPIPE. Why won't you use that to expose > >> your hw state? > > > >We took a look at dpipe and i talked to you about using it for this > >sor

Re: [PATCH net-next v2 00/10] net: dsa: add generic debugfs interface

2017-08-28 Thread Jiri Pirko
Tue, Aug 29, 2017 at 06:38:37AM CEST, da...@davemloft.net wrote: >From: Vivien Didelot >Date: Mon, 28 Aug 2017 15:17:38 -0400 > >> This patch series adds a generic debugfs interface for the DSA >> framework, so that all switch devices benefit from it, e.g. Marvell, >> Broadcom, Microchip or any ot

Re: [PATCH net-next v2 00/10] net: dsa: add generic debugfs interface

2017-08-28 Thread Jiri Pirko
Mon, Aug 28, 2017 at 10:08:34PM CEST, and...@lunn.ch wrote: >> I see this overlaps a lot with DPIPE. Why won't you use that to expose >> your hw state? > >We took a look at dpipe and i talked to you about using it for this >sort of thing at netconf/netdev. But dpipe has issues displaying the >sort

Re: [PATCH net-next v2 00/10] net: dsa: add generic debugfs interface

2017-08-28 Thread David Miller
From: Vivien Didelot Date: Mon, 28 Aug 2017 15:17:38 -0400 > This patch series adds a generic debugfs interface for the DSA > framework, so that all switch devices benefit from it, e.g. Marvell, > Broadcom, Microchip or any other DSA driver. I've been thinking this over and I agree with the feed

Re: [PATCH net-next v2 00/10] net: dsa: add generic debugfs interface

2017-08-28 Thread Andrew Lunn
> I see this overlaps a lot with DPIPE. Why won't you use that to expose > your hw state? We took a look at dpipe and i talked to you about using it for this sort of thing at netconf/netdev. But dpipe has issues displaying the sort of information we have. I never figured out how to do two dimensio

Re: [PATCH net-next v2 00/10] net: dsa: add generic debugfs interface

2017-08-28 Thread Jiri Pirko
Mon, Aug 28, 2017 at 09:17:38PM CEST, vivien.dide...@savoirfairelinux.com wrote: >This patch series adds a generic debugfs interface for the DSA >framework, so that all switch devices benefit from it, e.g. Marvell, >Broadcom, Microchip or any other DSA driver. > >This is really convenient for debug