From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1033241AbdEXU2b (ORCPT ); Wed, 24 May 2017 16:28:31 -0400 Received: from shards.monkeyblade.net ([184.105.139.130]:50036 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1031570AbdEXU2a (ORCPT ); Wed, 24 May 2017 16:28:30 -0400 Date: Wed, 24 May 2017 16:28:28 -0400 (EDT) Message-Id: <20170524.162828.2304525828467660236.davem@davemloft.net> To: vivien.didelot@savoirfairelinux.com Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, kernel@savoirfairelinux.com, f.fainelli@gmail.com, andrew@lunn.ch Subject: Re: [PATCH net-next] net: dsa: support cross-chip ageing time From: David Miller In-Reply-To: <20170523192059.1720-1-vivien.didelot@savoirfairelinux.com> References: <20170523192059.1720-1-vivien.didelot@savoirfairelinux.com> X-Mailer: Mew version 6.7 on Emacs 25.2 / Mule 6.0 (HANACHIRUSATO) 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]); Wed, 24 May 2017 12:46:55 -0700 (PDT) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Vivien Didelot Date: Tue, 23 May 2017 15:20:59 -0400 > Now that the switchdev bridge ageing time attribute is propagated to all > switch chips of the fabric, each switch can check if the requested value > is valid and program itself, so that the whole fabric shares a common > ageing time setting. > > This is especially needed for switch chips in between others, containing > no bridge port members but evidently used in the data path. > > To achieve that, remove the condition which skips the other switches. We > also don't need to identify the target switch anymore, thus remove the > sw_index member of the dsa_notifier_ageing_time_info notifier structure. > > On ZII Dev Rev B (with two 88E6352 and one 88E6185) and ZII Dev Rev C > (with two 88E6390X), we have the following hardware configuration: ... > Before this patch: ... > After this patch: ... > Signed-off-by: Vivien Didelot Applied, thanks.