lkml.org 
[lkml]   [2020]   [Jun]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH AUTOSEL 5.7 201/274] net: dsa: mt7530: set CPU port to fallback mode
    Date
    From: DENG Qingfang <dqfext@gmail.com>

    [ Upstream commit 38152ea37d8bdaffa22603e0a5b5b86cfa8714c9 ]

    Currently, setting a bridge's self PVID to other value and deleting
    the default VID 1 renders untagged ports of that VLAN unable to talk to
    the CPU port:

    bridge vlan add dev br0 vid 2 pvid untagged self
    bridge vlan del dev br0 vid 1 self
    bridge vlan add dev sw0p0 vid 2 pvid untagged
    bridge vlan del dev sw0p0 vid 1
    # br0 cannot send untagged frames out of sw0p0 anymore

    That is because the CPU port is set to security mode and its PVID is
    still 1, and untagged frames are dropped due to VLAN member violation.

    Set the CPU port to fallback mode so untagged frames can pass through.

    Fixes: 83163f7dca56 ("net: dsa: mediatek: add VLAN support for MT7530")
    Signed-off-by: DENG Qingfang <dqfext@gmail.com>
    Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
    Signed-off-by: David S. Miller <davem@davemloft.net>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/net/dsa/mt7530.c | 11 ++++++++---
    drivers/net/dsa/mt7530.h | 6 ++++++
    2 files changed, 14 insertions(+), 3 deletions(-)

    diff --git a/drivers/net/dsa/mt7530.c b/drivers/net/dsa/mt7530.c
    index 34e4aadfa705..b75d09783a05 100644
    --- a/drivers/net/dsa/mt7530.c
    +++ b/drivers/net/dsa/mt7530.c
    @@ -807,10 +807,15 @@ mt7530_port_set_vlan_aware(struct dsa_switch *ds, int port)
    PCR_MATRIX_MASK, PCR_MATRIX(MT7530_ALL_MEMBERS));

    /* Trapped into security mode allows packet forwarding through VLAN
    - * table lookup.
    + * table lookup. CPU port is set to fallback mode to let untagged
    + * frames pass through.
    */
    - mt7530_rmw(priv, MT7530_PCR_P(port), PCR_PORT_VLAN_MASK,
    - MT7530_PORT_SECURITY_MODE);
    + if (dsa_is_cpu_port(ds, port))
    + mt7530_rmw(priv, MT7530_PCR_P(port), PCR_PORT_VLAN_MASK,
    + MT7530_PORT_FALLBACK_MODE);
    + else
    + mt7530_rmw(priv, MT7530_PCR_P(port), PCR_PORT_VLAN_MASK,
    + MT7530_PORT_SECURITY_MODE);

    /* Set the port as a user port which is to be able to recognize VID
    * from incoming packets before fetching entry within the VLAN table.
    diff --git a/drivers/net/dsa/mt7530.h b/drivers/net/dsa/mt7530.h
    index 82af4d2d406e..14de60d0b9ca 100644
    --- a/drivers/net/dsa/mt7530.h
    +++ b/drivers/net/dsa/mt7530.h
    @@ -153,6 +153,12 @@ enum mt7530_port_mode {
    /* Port Matrix Mode: Frames are forwarded by the PCR_MATRIX members. */
    MT7530_PORT_MATRIX_MODE = PORT_VLAN(0),

    + /* Fallback Mode: Forward received frames with ingress ports that do
    + * not belong to the VLAN member. Frames whose VID is not listed on
    + * the VLAN table are forwarded by the PCR_MATRIX members.
    + */
    + MT7530_PORT_FALLBACK_MODE = PORT_VLAN(1),
    +
    /* Security Mode: Discard any frame due to ingress membership
    * violation or VID missed on the VLAN table.
    */
    --
    2.25.1
    \
     
     \ /
      Last update: 2020-06-09 02:51    [W:4.055 / U:0.176 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site