Received: by 2002:a05:6a10:c7c6:0:0:0:0 with SMTP id h6csp2555507pxy; Tue, 3 Aug 2021 09:06:36 -0700 (PDT) X-Google-Smtp-Source: ABdhPJz44xknx7G2Txj1T16eqVWNP9Y+iFT/5D7zvYgypV8l30ubuub++FYBIOKVn88qXLzekdk7 X-Received: by 2002:a6b:8d0d:: with SMTP id p13mr535501iod.8.1628006796162; Tue, 03 Aug 2021 09:06:36 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1628006796; cv=none; d=google.com; s=arc-20160816; b=HnDQOWOvVcBpZjqT90opcDq/PuWytOR0GSCn6f1BIpuHgfpu1UdbBXIUNI54ts8xlY axwHFvp7M/sN2xPnaeJHTl/RyJPEjdFz3iJ7SA1MzxDb3LOrXXc9/07/uI8VYpwM8XtR o8RvjA5IIPHbQOYgbIJ4BX88ao4x/jntQR58RJRFfm2TfHwRETgiUsq9ZN9TiCcASaq1 p7y0DybWqfngdP4PLZEkHbrMtyL9NDH3I1pe/LG1ha7WNEB1Yp9aj6oXZ2EGhgcsNmMi NFHMLXLImhAzE6zrq5rtE1yFirHf2t95+05idmlwL8RAYVwb6EKkq3IIZb2gdlpDs9W5 mufA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=8uiSeD2OkBp9zNpUDtNBLvudtMpehXughyaF6cE7kWE=; b=VeDCOmpNfztLb2kb6FhGeSDpZWyE/Lsoq7U2wHP6r4UEfJxvLUbLUPE6AHtHK94w4t KBVF+MwB0A5ZoCoMdDYYbnd70Nj3Ry49qyl0ODSgjRjSEgzzWEUB9uvIPr0Os2DB0Dq5 bmD3k2xFSMmUq3OaaTKMs/kprf1BHWRrfvHyecAKYcRmy4W9y8a1bTFFLR9ZIFJZPwKv hyKGpDmqFfELZ0fymSwFefFkn+2sHALRxsTXX7LHA4yY8tYGxMnMUIqvIRsdHNlluLOk OMRND63qOlL6s0Rr6g/dXTXJjxjN+H+9HOX/UDOHpR2cW0wDTGFQEaF9WGCWja06/PZC 3XMw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b="HI/RNgsT"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id s22si16434266iow.33.2021.08.03.09.06.13; Tue, 03 Aug 2021 09:06:36 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b="HI/RNgsT"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231871AbhHCQFC (ORCPT + 99 others); Tue, 3 Aug 2021 12:05:02 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:54796 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231315AbhHCQE4 (ORCPT ); Tue, 3 Aug 2021 12:04:56 -0400 Received: from mail-pl1-x633.google.com (mail-pl1-x633.google.com [IPv6:2607:f8b0:4864:20::633]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 4AFE7C06175F; Tue, 3 Aug 2021 09:04:44 -0700 (PDT) Received: by mail-pl1-x633.google.com with SMTP id e21so24292406pla.5; Tue, 03 Aug 2021 09:04:44 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=8uiSeD2OkBp9zNpUDtNBLvudtMpehXughyaF6cE7kWE=; b=HI/RNgsTbqZEYCiV99s2NdnzjSSL+jfEFNFwt9FPZJ+Ru6ksZeQK6DBubeeiqi8Nul RmjCDvqRoFbmQfnMP5QlNwwFCceSlVwmwDF45xDsL5gR77eoei+zQMAFwg8umjkmZ7Hp zb4gX8VrzX0VQ1V8m1tnb0+3e10Nyt7N5WxyY8RzdQZFBtg0HMFSYKA6JRaP30Xx0VKU ZhYFitTn78WccAQOmO9Bp/vm4B1nspHMVV2m19O2sKs6Vn5sOuYoI3o0D3YuPrxGA0Tw 6MR2MQUmgaBq63GbpLiDQFYCEFxWj05lL12rhWQVKFz1cbvbieAg0cn+0rHw1RZE73AG 3hZw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=8uiSeD2OkBp9zNpUDtNBLvudtMpehXughyaF6cE7kWE=; b=PrjXKhHy6bITMFSBnBo8sQ3KlFj6zXKlzL3rH2sOQE12dKw6eXEphMjX1h+kAYybcu nA2bTY+bLUeD4i0xrLCtFMYqws02E1Bz4b/xOxMXT6XUvgW01yty4VcmMBpIM9wVDouL lUYOF0pwmbBw0OMRnnwewbCGo3ndkL61IWOdQoKxVrrXWynPRGRplAY8+VZxGxgUH3w6 vJxRRqz7g0pJi1GsUCDvd7VZ/gO9n/eMkQ+n39Pf6dVD3td1MCSw+wlJq2tA3HIfzJwE 559TzqMs/dMcH+/X+WM+y7UVJ9ePZoa9VU7x+En+7fRipOxfHcPaJqlj2K1r3lLOqX4J J14Q== X-Gm-Message-State: AOAM530snQ1ftrsSYCJKo1Zv7VCx9x4lkh9LBb+AoRo/0gs5lFu1dyvB C/Q0vd9J6sPMSS45FDu7X8U= X-Received: by 2002:a17:90b:344d:: with SMTP id lj13mr4940643pjb.24.1628006683927; Tue, 03 Aug 2021 09:04:43 -0700 (PDT) Received: from haswell-ubuntu20.lan ([138.197.212.246]) by smtp.gmail.com with ESMTPSA id y6sm14390653pjr.48.2021.08.03.09.04.38 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 03 Aug 2021 09:04:43 -0700 (PDT) From: DENG Qingfang To: Sean Wang , Landen Chao , Andrew Lunn , Vivien Didelot , Florian Fainelli , Vladimir Oltean , "David S. Miller" , Jakub Kicinski , Matthias Brugger , netdev@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, linux-kernel@vger.kernel.org Cc: Eric Woudstra , =?UTF-8?q?Ren=C3=A9=20van=20Dorst?= , Frank Wunderlich , Ilya Lipnitskiy Subject: [PATCH net-next v2 4/4] net: dsa: mt7530: always install FDB entries with IVL and FID 1 Date: Wed, 4 Aug 2021 00:04:04 +0800 Message-Id: <20210803160405.3025624-5-dqfext@gmail.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20210803160405.3025624-1-dqfext@gmail.com> References: <20210803160405.3025624-1-dqfext@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org This reverts commit 7e777021780e ("mt7530 mt7530_fdb_write only set ivl bit vid larger than 1"). Before this series, the default value of all ports' PVID is 1, which is copied into the FDB entry, even if the ports are VLAN unaware. So `bridge fdb show` will show entries like `dev swp0 vlan 1 self` even on a VLAN-unaware bridge. The blamed commit does not solve that issue completely, instead it may cause a new issue that FDB is inaccessible in a VLAN-aware bridge with PVID 1. This series sets PVID to 0 on VLAN-unaware ports, so `bridge fdb show` will no longer print `vlan 1` on VLAN-unaware bridges, and that special case in fdb_write is not required anymore. Set FDB entries' filter ID to 1 to match the VLAN table. Signed-off-by: DENG Qingfang --- v1 -> v2: use FID enum instead of hardcoding. drivers/net/dsa/mt7530.c | 4 ++-- drivers/net/dsa/mt7530.h | 1 + 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/net/dsa/mt7530.c b/drivers/net/dsa/mt7530.c index 9b39ccd9dd4c..385e169080d9 100644 --- a/drivers/net/dsa/mt7530.c +++ b/drivers/net/dsa/mt7530.c @@ -366,8 +366,8 @@ mt7530_fdb_write(struct mt7530_priv *priv, u16 vid, int i; reg[1] |= vid & CVID_MASK; - if (vid > 1) - reg[1] |= ATA2_IVL; + reg[1] |= ATA2_IVL; + reg[1] |= ATA2_FID(FID_BRIDGED); reg[2] |= (aging & AGE_TIMER_MASK) << AGE_TIMER; reg[2] |= (port_mask & PORT_MAP_MASK) << PORT_MAP; /* STATIC_ENT indicate that entry is static wouldn't diff --git a/drivers/net/dsa/mt7530.h b/drivers/net/dsa/mt7530.h index 5b70ccef9459..4a91d80f51bb 100644 --- a/drivers/net/dsa/mt7530.h +++ b/drivers/net/dsa/mt7530.h @@ -80,6 +80,7 @@ enum mt753x_bpdu_port_fw { #define STATIC_ENT 3 #define MT7530_ATA2 0x78 #define ATA2_IVL BIT(15) +#define ATA2_FID(x) (((x) & 0x7) << 12) /* Register for address table write data */ #define MT7530_ATWD 0x7c -- 2.25.1