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. -