Re: [PATCH net-next 0/6] net: dsa: push switchdev prepare phase in FDB ops

2015-10-08 Thread Vivien Didelot
Hi Jiri, David, On Oct. Thursday 08 (41) 03:47 PM, Jiri Pirko wrote: > Thu, Oct 08, 2015 at 03:32:51PM CEST, vivien.dide...@savoirfairelinux.com > wrote: > >Hi David, > > > >On Oct. Thursday 08 (41) 05:28 AM, David Miller wrote: > >> From: Vivien Didelot > >> Date: Wed, 7 Oct 2015 19:48:25 -040

Re: [PATCH net-next 0/6] net: dsa: push switchdev prepare phase in FDB ops

2015-10-08 Thread Jiri Pirko
Thu, Oct 08, 2015 at 03:32:51PM CEST, vivien.dide...@savoirfairelinux.com wrote: >Hi David, > >On Oct. Thursday 08 (41) 05:28 AM, David Miller wrote: >> From: Vivien Didelot >> Date: Wed, 7 Oct 2015 19:48:25 -0400 >> >> > The first 3 patches removes the dsa.h include from linux/netdevice.h, whic

Re: [PATCH net-next 0/6] net: dsa: push switchdev prepare phase in FDB ops

2015-10-08 Thread Vivien Didelot
Hi David, On Oct. Thursday 08 (41) 05:28 AM, David Miller wrote: > From: Vivien Didelot > Date: Wed, 7 Oct 2015 19:48:25 -0400 > > > The first 3 patches removes the dsa.h include from linux/netdevice.h, which > > broke the inclusion of switchdev.h in dsa.h. > > I still don't agree with bloatin

Re: [PATCH net-next 0/6] net: dsa: push switchdev prepare phase in FDB ops

2015-10-08 Thread David Miller
From: Vivien Didelot Date: Wed, 7 Oct 2015 19:48:25 -0400 > The first 3 patches removes the dsa.h include from linux/netdevice.h, which > broke the inclusion of switchdev.h in dsa.h. I still don't agree with bloating up struct netdevice just to deal with an include file ordering issue, sorry. -

[PATCH net-next 0/6] net: dsa: push switchdev prepare phase in FDB ops

2015-10-07 Thread Vivien Didelot
This patchset pushes the switchdev prepare phase for the FDB add and del operations down to the DSA drivers. Currently only mv88e6xxx is affected. Since the dump requires a bit of refactoring in the driver, it'll come in a future patchset. The first 3 patches removes the dsa.h include from linux/