Re: [PATCH 4.9] brcmfmac: use correct skb freeing helper when deleting flowring

On 27-9-2016 13:27, Kalle Valo wrote:

Arend Van Spriel [off-list ref] writes:

quoted

On 27-9-2016 11:14, Rafał Miłecki wrote:

quoted

From: Rafał Miłecki <rafal@milecki.pl>

Flowrings contain skbs waiting for transmission that were passed to us
by netif. It means we checked every one of them looking for 802.1x
Ethernet type. When deleting flowring we have to use freeing function
that will check for 802.1x type as well.

Freeing skbs without a proper check was leading to counter not being
properly decreased. This was triggering a WARNING every time
brcmf_netdev_wait_pend8021x was called.

Acked-by: Arend van Spriel <redacted>

quoted

Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
---
Kalle: this isn't important enough for 4.8 as it's too late for that.

I'd like to get it for 4.9 however, as this fixes bug that could lead
to WARNING on every add_key/del_key call. We was struggling with these
WARNINGs for some time and this fixes one of two problems causing them.

Ok, I'll queue this for 4.9.

quoted

Please mark it for stable as well.

I can add that. Any ideas how old releases stable releases should this
go to?

Not sure if the vendor directory move causes issues as stable can not
fallback to three-way merge. I assumed it would so my last stable tag
was only for 4.7 and I took care of older kernels at later time with
backported patch. I can do that for this one as well.

Regards,
Arend