public inbox for stable@vger.kernel.org
 help / color / mirror / Atom feed
From: Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org>
To: gregkh@linuxfoundation.org, stable@vger.kernel.org
Subject: Re: Patch "net: ethernet: ti: cpsw: unsync mcast entries while switch promisc mode" has been added to the 4.19-stable tree
Date: Sun, 11 Nov 2018 20:31:25 +0200	[thread overview]
Message-ID: <20181111183124.GC2354@khorivan> (raw)
In-Reply-To: <1541877375192170@kroah.com>

On Sat, Nov 10, 2018 at 11:16:15AM -0800, gregkh@linuxfoundation.org wrote:

It shouldn't be added to 4.19
>
>This is a note to let you know that I've just added the patch titled
>
>    net: ethernet: ti: cpsw: unsync mcast entries while switch promisc mode
>
>to the 4.19-stable tree which can be found at:
>    http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary
>
>The filename of the patch is:
>     net-ethernet-ti-cpsw-unsync-mcast-entries-while-switch-promisc-mode.patch
>and it can be found in the queue-4.19 subdirectory.
>
>If you, or anyone else, feels it should not be added to the stable tree,
>please let <stable@vger.kernel.org> know about it.
>
>
>>From foo@baz Sat Nov 10 10:51:03 PST 2018
>From: Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org>
>Date: Mon, 22 Oct 2018 21:51:36 +0300
>Subject: net: ethernet: ti: cpsw: unsync mcast entries while switch promisc mode
>
>From: Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org>
>
>[ Upstream commit 9737cc99dd14b5b8b9d267618a6061feade8ea68 ]
>
>After flushing all mcast entries from the table, the ones contained in
>mc list of ndev are not restored when promisc mode is toggled off,
>because they are considered as synched with ALE, thus, in order to
>restore them after promisc mode - reset syncing info. This fix
>touches only switch mode devices, including single port boards
>like Beagle Bone.
>
>Fixes: commit 5da1948969bc
>("net: ethernet: ti: cpsw: fix lost of mcast packets while rx_mode update")
>
>Signed-off-by: Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org>
>Reviewed-by: Grygorii Strashko <grygorii.strashko@ti.com>
>Signed-off-by: David S. Miller <davem@davemloft.net>
>Signed-off-by: Sasha Levin <sashal@kernel.org>
>Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
>---
> drivers/net/ethernet/ti/cpsw.c |    1 +
> 1 file changed, 1 insertion(+)
>
>--- a/drivers/net/ethernet/ti/cpsw.c
>+++ b/drivers/net/ethernet/ti/cpsw.c
>@@ -642,6 +642,7 @@ static void cpsw_set_promiscious(struct
>
> 			/* Clear all mcast from ALE */
> 			cpsw_ale_flush_multicast(ale, ALE_ALL_PORTS, -1);
>+			__dev_mc_unsync(ndev, NULL);
>
> 			/* Flood All Unicast Packets to Host port */
> 			cpsw_ale_control_set(ale, 0, ALE_P0_UNI_FLOOD, 1);
>
>
>Patches currently in stable-queue which might be from ivan.khoronzhuk@linaro.org are
>
>queue-4.19/net-ethernet-ti-cpsw-unsync-mcast-entries-while-switch-promisc-mode.patch

-- 
Regards,
Ivan Khoronzhuk

       reply	other threads:[~2018-11-12  4:20 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1541877375192170@kroah.com>
2018-11-11 18:31 ` Ivan Khoronzhuk [this message]
2018-11-11 19:26   ` Patch "net: ethernet: ti: cpsw: unsync mcast entries while switch promisc mode" has been added to the 4.19-stable tree Greg KH
2018-11-11 22:22     ` Sasha Levin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20181111183124.GC2354@khorivan \
    --to=ivan.khoronzhuk@linaro.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=stable@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox