Re: [Bridge] [PATCH net-next] net: bridge: update multicast stats from maybe_deliver()

2019-04-04 Thread David Miller
From: Pablo Neira Ayuso Date: Thu, 4 Apr 2019 13:56:38 +0200 > Simplify this code by updating bridge multicast stats from > maybe_deliver(). > > Note that commit 6db6f0eae605 ("bridge: multicast to unicast"), in case > the port flag BR_MULTICAST_TO_UNICAST is set, never updates the previous >

Re: [Bridge] [PATCH net-next] net: bridge: update multicast stats from maybe_deliver()

2019-04-04 Thread Nikolay Aleksandrov
On 04/04/2019 15:21, Pablo Neira Ayuso wrote: > On Thu, Apr 04, 2019 at 03:02:19PM +0300, Nikolay Aleksandrov wrote: >> On 04/04/2019 14:56, Pablo Neira Ayuso wrote: >>> Simplify this code by updating bridge multicast stats from >>> maybe_deliver(). >>> >>> Note that commit 6db6f0eae605 ("bridge:

Re: [Bridge] [PATCH net-next] net: bridge: update multicast stats from maybe_deliver()

2019-04-04 Thread Pablo Neira Ayuso
On Thu, Apr 04, 2019 at 03:02:19PM +0300, Nikolay Aleksandrov wrote: > On 04/04/2019 14:56, Pablo Neira Ayuso wrote: > > Simplify this code by updating bridge multicast stats from > > maybe_deliver(). > > > > Note that commit 6db6f0eae605 ("bridge: multicast to unicast"), in case > > the port

Re: [Bridge] [PATCH net-next] net: bridge: update multicast stats from maybe_deliver()

2019-04-04 Thread Nikolay Aleksandrov
On 04/04/2019 15:02, Nikolay Aleksandrov wrote: > On 04/04/2019 14:56, Pablo Neira Ayuso wrote: >> Simplify this code by updating bridge multicast stats from >> maybe_deliver(). >> >> Note that commit 6db6f0eae605 ("bridge: multicast to unicast"), in case >> the port flag BR_MULTICAST_TO_UNICAST

Re: [Bridge] [PATCH net-next] net: bridge: update multicast stats from maybe_deliver()

2019-04-04 Thread Nikolay Aleksandrov
On 04/04/2019 14:56, Pablo Neira Ayuso wrote: > Simplify this code by updating bridge multicast stats from > maybe_deliver(). > > Note that commit 6db6f0eae605 ("bridge: multicast to unicast"), in case > the port flag BR_MULTICAST_TO_UNICAST is set, never updates the previous > port pointer,

[Bridge] [PATCH net-next] net: bridge: update multicast stats from maybe_deliver()

2019-04-04 Thread Pablo Neira Ayuso
Simplify this code by updating bridge multicast stats from maybe_deliver(). Note that commit 6db6f0eae605 ("bridge: multicast to unicast"), in case the port flag BR_MULTICAST_TO_UNICAST is set, never updates the previous port pointer, therefore it is always going to be different from the existing