diff for duplicates of <87inthy441.fsf@kamboji.qca.qualcomm.com> diff --git a/a/1.txt b/N1/1.txt index a08ff0e..2efafcf 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,22 +1,22 @@ Arend Van Spriel <arend.vanspriel@broadcom.com> writes: -> On 27-9-2016 11:14, Rafa=C5=82 Mi=C5=82ecki wrote: ->> From: Rafa=C5=82 Mi=C5=82ecki <rafal@milecki.pl> ->>=20 +> On 27-9-2016 11:14, Rafał Miłecki wrote: +>> 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. ->>=20 +>> >> 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 <arend@broadcom.com> ->> Signed-off-by: Rafa=C5=82 Mi=C5=82ecki <rafal@milecki.pl> +>> 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. ->>=20 +>> >> 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. @@ -28,5 +28,5 @@ Ok, I'll queue this for 4.9. I can add that. Any ideas how old releases stable releases should this go to? ---=20 +-- Kalle Valo diff --git a/a/content_digest b/N1/content_digest index 81e2ac2..b4ed5ff 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -18,23 +18,23 @@ "b\0" "Arend Van Spriel <arend.vanspriel@broadcom.com> writes:\n" "\n" - "> On 27-9-2016 11:14, Rafa=C5=82 Mi=C5=82ecki wrote:\n" - ">> From: Rafa=C5=82 Mi=C5=82ecki <rafal@milecki.pl>\n" - ">>=20\n" + "> On 27-9-2016 11:14, Rafa\305\202 Mi\305\202ecki wrote:\n" + ">> From: Rafa\305\202 Mi\305\202ecki <rafal@milecki.pl>\n" + ">> \n" ">> Flowrings contain skbs waiting for transmission that were passed to us\n" ">> by netif. It means we checked every one of them looking for 802.1x\n" ">> Ethernet type. When deleting flowring we have to use freeing function\n" ">> that will check for 802.1x type as well.\n" - ">>=20\n" + ">> \n" ">> Freeing skbs without a proper check was leading to counter not being\n" ">> properly decreased. This was triggering a WARNING every time\n" ">> brcmf_netdev_wait_pend8021x was called.\n" ">\n" "> Acked-by: Arend van Spriel <arend@broadcom.com>\n" - ">> Signed-off-by: Rafa=C5=82 Mi=C5=82ecki <rafal@milecki.pl>\n" + ">> Signed-off-by: Rafa\305\202 Mi\305\202ecki <rafal@milecki.pl>\n" ">> ---\n" ">> Kalle: this isn't important enough for 4.8 as it's too late for that.\n" - ">>=20\n" + ">> \n" ">> I'd like to get it for 4.9 however, as this fixes bug that could lead\n" ">> to WARNING on every add_key/del_key call. We was struggling with these\n" ">> WARNINGs for some time and this fixes one of two problems causing them.\n" @@ -46,7 +46,7 @@ "I can add that. Any ideas how old releases stable releases should this\n" "go to?\n" "\n" - "--=20\n" + "-- \n" Kalle Valo -eb70755d8e2de7982ffee7efe616065e01046b8d34ac0519e7e7c899cb5187b1 +0835fa7ee6edc95940662eac1994af6306b7a41d143855c83544a1177da01b7d
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.