* [PATCH net] net: dsa: b53: Do not clear existing mirrored port mask
@ 2019-10-05 22:05 Florian Fainelli
2019-10-06 4:44 ` Vivien Didelot
2019-10-06 13:56 ` David Miller
0 siblings, 2 replies; 3+ messages in thread
From: Florian Fainelli @ 2019-10-05 22:05 UTC (permalink / raw)
To: netdev
Cc: Hubert Feurstein, Florian Fainelli, Andrew Lunn, Vivien Didelot,
David S. Miller, open list
Clearing the existing bitmask of mirrored ports essentially prevents us
from capturing more than one port at any given time. This is clearly
wrong, do not clear the bitmask prior to setting up the new port.
Reported-by: Hubert Feurstein <h.feurstein@gmail.com>
Fixes: ed3af5fd08eb ("net: dsa: b53: Add support for port mirroring")
Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
---
drivers/net/dsa/b53/b53_common.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/net/dsa/b53/b53_common.c b/drivers/net/dsa/b53/b53_common.c
index 526ba2ab66f1..cc3536315eff 100644
--- a/drivers/net/dsa/b53/b53_common.c
+++ b/drivers/net/dsa/b53/b53_common.c
@@ -1845,7 +1845,6 @@ int b53_mirror_add(struct dsa_switch *ds, int port,
loc = B53_EG_MIR_CTL;
b53_read16(dev, B53_MGMT_PAGE, loc, ®);
- reg &= ~MIRROR_MASK;
reg |= BIT(port);
b53_write16(dev, B53_MGMT_PAGE, loc, reg);
--
2.17.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH net] net: dsa: b53: Do not clear existing mirrored port mask
2019-10-05 22:05 [PATCH net] net: dsa: b53: Do not clear existing mirrored port mask Florian Fainelli
@ 2019-10-06 4:44 ` Vivien Didelot
2019-10-06 13:56 ` David Miller
1 sibling, 0 replies; 3+ messages in thread
From: Vivien Didelot @ 2019-10-06 4:44 UTC (permalink / raw)
To: Florian Fainelli
Cc: netdev, Hubert Feurstein, Florian Fainelli, Andrew Lunn,
David S. Miller, open list
On Sat, 5 Oct 2019 15:05:18 -0700, Florian Fainelli <f.fainelli@gmail.com> wrote:
> Clearing the existing bitmask of mirrored ports essentially prevents us
> from capturing more than one port at any given time. This is clearly
> wrong, do not clear the bitmask prior to setting up the new port.
>
> Reported-by: Hubert Feurstein <h.feurstein@gmail.com>
> Fixes: ed3af5fd08eb ("net: dsa: b53: Add support for port mirroring")
> Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Reviewed-by: Vivien Didelot <vivien.didelot@gmail.com>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH net] net: dsa: b53: Do not clear existing mirrored port mask
2019-10-05 22:05 [PATCH net] net: dsa: b53: Do not clear existing mirrored port mask Florian Fainelli
2019-10-06 4:44 ` Vivien Didelot
@ 2019-10-06 13:56 ` David Miller
1 sibling, 0 replies; 3+ messages in thread
From: David Miller @ 2019-10-06 13:56 UTC (permalink / raw)
To: f.fainelli; +Cc: netdev, h.feurstein, andrew, vivien.didelot, linux-kernel
From: Florian Fainelli <f.fainelli@gmail.com>
Date: Sat, 5 Oct 2019 15:05:18 -0700
> Clearing the existing bitmask of mirrored ports essentially prevents us
> from capturing more than one port at any given time. This is clearly
> wrong, do not clear the bitmask prior to setting up the new port.
>
> Reported-by: Hubert Feurstein <h.feurstein@gmail.com>
> Fixes: ed3af5fd08eb ("net: dsa: b53: Add support for port mirroring")
> Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Applied and queued up for -stable, thanks.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-10-06 13:56 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-05 22:05 [PATCH net] net: dsa: b53: Do not clear existing mirrored port mask Florian Fainelli
2019-10-06 4:44 ` Vivien Didelot
2019-10-06 13:56 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).