* [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 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.