Received: by 2002:a5b:505:0:0:0:0:0 with SMTP id o5csp3203494ybp; Sun, 6 Oct 2019 06:58:18 -0700 (PDT) X-Google-Smtp-Source: APXvYqxYx/Cqh2GL2l6kod5ZGu1iYmDfYnt6OsnyFq0OrXbL5mqtXNE+0nMhfVF5onoUZtH8596D X-Received: by 2002:a17:906:eb86:: with SMTP id mh6mr19639557ejb.171.1570370298105; Sun, 06 Oct 2019 06:58:18 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1570370298; cv=none; d=google.com; s=arc-20160816; b=WvrJDF9TgBgj/skHwnXNg+A1lVNL8Cld6Vbab9wq94z/+Jms/FLg3W6LJpatYgJd1p cbgvb8Wh8oe9bpWNLKGEAMuXc9JATztBsWM758HSqF4sSW1/sd7IBkRL1t5M+OMJcu6m nTHCDDB5m2lgpCcBWA2WvozMQQZzmMTxu/9MjC7kcVObb99SvhH5Bz2FLw1lWe+YOOq6 L7yxWy/toOdh1kmuWB7nPxUplWcBE8TgscoG2vwon/nghS7/9YtA5YSZGL2VA4rQayea yy5kk+f9GF9Bg2XMRtebyRnxoUjMKrplVN4G58AlsxKy2Uer5+Y9UhCadnu+6Inq/zPB LpSw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :references:in-reply-to:from:subject:cc:to:message-id:date; bh=x/1WFiQUxbQI1c2doDz1bnGuZoTXpqSnSnX2wR2kIac=; b=yx7Een6Zg9MhrNihsyU3osttRmY2NpdjDGpYmT7PN23eFfke1P4SCh4STi1U4XX2eG +vrG0TLsNrvEQMj40gDxgToH517Ez1ElYG35tZBREa+BvGNv8+VYkTr7Rx4O4pYf26v+ EHVwzhuLLv1dK16/BLc+jsGt7c7j8Hw3FEvMSqtiwV0D30Z4Z0KZqU2wiAGsstVPN9kH 8LoN3JRH6kJBei7ywQXoqCm5G7QHyA3qssEtIUD5nr15SMYxlZ9/gYF6pa4FVigG8YKw qi4+j8vNKBlxHH0YWxx9l/q4lmSDdnpQAViRWtVE9I3N4RK4UXVuPtrmjnmApyszUCLx 7Fyw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id h48si6777995ede.31.2019.10.06.06.57.17; Sun, 06 Oct 2019 06:58:18 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726490AbfJFN4n (ORCPT + 99 others); Sun, 6 Oct 2019 09:56:43 -0400 Received: from shards.monkeyblade.net ([23.128.96.9]:44386 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726424AbfJFN4n (ORCPT ); Sun, 6 Oct 2019 09:56:43 -0400 Received: from localhost (unknown [63.64.162.234]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (Client did not present a certificate) (Authenticated sender: davem-davemloft) by shards.monkeyblade.net (Postfix) with ESMTPSA id A3D921429F7D9; Sun, 6 Oct 2019 06:56:42 -0700 (PDT) Date: Sun, 06 Oct 2019 15:56:42 +0200 (CEST) Message-Id: <20191006.155642.1768701400675135903.davem@davemloft.net> To: f.fainelli@gmail.com Cc: netdev@vger.kernel.org, h.feurstein@gmail.com, andrew@lunn.ch, vivien.didelot@gmail.com, linux-kernel@vger.kernel.org Subject: Re: [PATCH net] net: dsa: b53: Do not clear existing mirrored port mask From: David Miller In-Reply-To: <20191005220518.14008-1-f.fainelli@gmail.com> References: <20191005220518.14008-1-f.fainelli@gmail.com> X-Mailer: Mew version 6.8 on Emacs 26.2 Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.5.12 (shards.monkeyblade.net [149.20.54.216]); Sun, 06 Oct 2019 06:56:42 -0700 (PDT) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Florian Fainelli 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 > Fixes: ed3af5fd08eb ("net: dsa: b53: Add support for port mirroring") > Signed-off-by: Florian Fainelli Applied and queued up for -stable, thanks.