Re: [PATCH 3.16 046/114] batman-adv: Fix broadcast/ogm queue limit on a removed interface

2016-06-13 Thread Sven Eckelmann
Hi, On Monday 13 June 2016 23:53:06 Ben Hutchings wrote: [...] > > Yes, I recognise this bug. I've updated this patch to the version below. looks good to me. Thanks for updating the patch :) Kind regards, Sven > --- > batman-adv: Fix broadcast/ogm queue limit on a removed interface >

Re: [PATCH 3.16 046/114] batman-adv: Fix broadcast/ogm queue limit on a removed interface

2016-06-13 Thread Ben Hutchings
On Mon, 2016-06-13 at 21:33 +0200, Sven Eckelmann wrote: > Hi, > > looks like the patch is broken (at least it does something different than the  > non-stable patch does). > > On Monday 13 June 2016 19:36:37 Ben Hutchings wrote: > > 3.16.36-rc1 review patch.  If anyone has any objections, please

Re: [PATCH 3.16 046/114] batman-adv: Fix broadcast/ogm queue limit on a removed interface

2016-06-13 Thread Linus Lüssing
On Mon, Jun 13, 2016 at 07:36:37PM +0100, Ben Hutchings wrote: > 3.16.36-rc1 review patch. If anyone has any objections, please let me know. > Hi Ben, This one looks weird. The version you added for 3.2.81-rc1 looked better. Cheers, Linus > -- > > From: Linus Lüssing > >

Re: [PATCH 3.16 046/114] batman-adv: Fix broadcast/ogm queue limit on a removed interface

2016-06-13 Thread Sven Eckelmann
Hi, looks like the patch is broken (at least it does something different than the non-stable patch does). On Monday 13 June 2016 19:36:37 Ben Hutchings wrote: > 3.16.36-rc1 review patch. If anyone has any objections, please let me know. [...] > --- a/net/batman-adv/send.c > +++ b/net/batman-adv

[PATCH 3.16 046/114] batman-adv: Fix broadcast/ogm queue limit on a removed interface

2016-06-13 Thread Ben Hutchings
3.16.36-rc1 review patch. If anyone has any objections, please let me know. -- From: Linus Lüssing commit c4fdb6cff2aa0ae740c5f19b6f745cbbe786d42f upstream. When removing a single interface while a broadcast or ogm packet is still pending then we will free the forward packet w