Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Vladimir Oltean <vladimir.oltean@nxp.com>
To:     netdev@vger.kernel.org
Cc:     Vinicius Costa Gomes <vinicius.gomes@intel.com>,
        Jamal Hadi Salim <jhs@mojatatu.com>,
        Cong Wang <xiyou.wangcong@gmail.com>,
        Jiri Pirko <jiri@resnulli.us>,
        "David S. Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>, linux-kernel@vger.kernel.org
Subject: [PATCH net-next] net/sched: taprio: simplify list iteration in taprio_dev_notifier()
Date:   Fri, 23 Sep 2022 17:59:21 +0300
Message-Id: <20220923145921.3038904-1-vladimir.oltean@nxp.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?Isrde4GfdU6zxr/F0b7z4NYrXXXHdlwe8zAohsNc0aBmFJnWMFk0kQz9e12H?=
 =?us-ascii?Q?9nGMr60K7HI48/nHKzHCq/9ZpPXmoaqpX4G3DWzm5gXjnjBIjRe/0voDrjLP?=
 =?us-ascii?Q?CuYxIT5Xt+qoMLXzFFfpZ7x92t+/328VqWLwfqGaY1uBrpQVer1R9Rhb3TCJ?=
 =?us-ascii?Q?bhmggdut1Cgk+Q0nQf4UfhjFHzdrwv0c32xiYsC8TPiiIOGIrCXURSD/+BBe?=
 =?us-ascii?Q?uXu2FLUaEqgNrdld3xjIZIRMnZytft+OpWOLJUCpAnpBdQ2ffrkVpbzLba1k?=
 =?us-ascii?Q?Dn7nwo8pA/AnWd0a6McoTyZkx33SfT1gRejhBsIRrdmBVDDOAmdFplZKZQY/?=
 =?us-ascii?Q?wIMqow4sr4SqB03QWUaGePTLzW41hlxai3QEeEbki8XzbbrEAC57IknsBz2A?=
 =?us-ascii?Q?dTX9EmlMmXE8WDdL/WwdFHrcPMazPYpMb/VMOp8gheLiIJwHMsgJD0L8K1b9?=
 =?us-ascii?Q?DsbvZyAzssRIX3qudrgCRB4gVAObtrYZrvgCSxLwyIJHHhQknHc8O6h+PIB8?=
 =?us-ascii?Q?gPsmxRz1nD+5kUzOZL8F7lJQFFpda46ow8B2iG31q6dAmR+50Ebu4yi73//i?=
 =?us-ascii?Q?1KV7Sm/LUWCLfVpyW6wLRjFzB2u423Clm+qMTUWLEPkWbcvcMvTHng5cpbKt?=
 =?us-ascii?Q?Ohc31MCNebyWfWU7HB6w8p4/JJDp3uXz5a+2ARe+zr9UOU0ETtnJz3N+rUPn?=
 =?us-ascii?Q?6/8C667ETK4ZQPEb4ffWKNMPtxaaA15i3DXgOeS1GWfF7QoZP80VO9LzOZCt?=
 =?us-ascii?Q?geu5+pBp0OfNy7N2DXpFwEIxaqOpLp+bHgqtyRFe2Y02b6ky96t8G3UTf8Bm?=
 =?us-ascii?Q?YGbctdG17MjBzxr2AKXoWMSXOLCg6OfRWmsvR5NWYnm3Pz87JQbN5jMhBQH5?=
 =?us-ascii?Q?qyXRaTryvLM42zVtT2noEw0ncIUCoz9k77fTnRh8k5V2sT9aCwoQaxhcfsjS?=
 =?us-ascii?Q?ZgclS56FxI4ajoY/MBRHHiWgHQTkmFBu989pHKyijmymzqqkY4iZCohYVoPM?=
 =?us-ascii?Q?R5TlhaUQhtxgsMrLCyutK0smqOpAhheIhwQ7d1tsOATrI4w72PgQotF4vO41?=
 =?us-ascii?Q?eZYwxnRq+7yoG9np/IN0ws8DA1Vb4DQCWPxJOD6fe2IapYG9QHMr2TPFryq6?=
 =?us-ascii?Q?xhLinTHwanNBLhaqw+mUnZ8gIM8y1nTzWh6n54W9iOkRrZZ65hD3jnCjrSrn?=
 =?us-ascii?Q?lgnHUo3OCixW1LRZmpLCLB+Ws/svwaFZPcqJqgVnM1WhsOoVsJhM2U6FNZR4?=
 =?us-ascii?Q?vjCXZVzQd5oKRVz6YcMqZ0fpEr5ZlKXlXMRQB43zNWozdsBRqKo1H5x4oykA?=
 =?us-ascii?Q?5uBprGPj0E3igK/zJoCbb1VYSOuRBnR39J/UdVGIbBDTQH8treDgYwNHXZN3?=
 =?us-ascii?Q?Gud9BuEGaJsbJ4HTvAgrNfIwE5mM6BZnp5jLcyLjOpFqAVrcG6TnuiDkYudQ?=
 =?us-ascii?Q?K4ehJUbeBQSvR6q3R3o1JY5haGrpX/g9FLgxjohZBVwoUE0Iklv3NASmtnPh?=
 =?us-ascii?Q?rvFVebBNYIPrlRpBp1sNYtiyneKpIjyOpijAecv2OZBEAKlW5BX40P0xRM26?=
 =?us-ascii?Q?WL3zVuRD3kr9rCFODgiRBLyTDCnjZUp6GcunK1rT8pAWj5S6Q3yh0pfdNsaI?=
 =?us-ascii?Q?PQ=3D=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: d7899da7-a05c-45cd-0d1a-08da9d7438c3
X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5136.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Sep 2022 14:59:32.1826
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: YBk6amYyU7NEgNJxK6mDl38MQ67jSB0SGA8rmBQ3piqyEc4KYscpAe+XtNKqOjzoOr7Uua+pb7vM9rI7yrKGrQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PR3PR04MB7436
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

taprio_dev_notifier() subscribes to netdev state changes in order to
determine whether interfaces which have a taprio root qdisc have changed
their link speed, so the internal calculations can be adapted properly.

The 'qdev' temporary variable serves no purpose, because we just use it
only once, and can just as well use qdisc_dev(q->root) directly (or the
"dev" that comes from the netdev notifier; this is because qdev is only
interesting if it was the subject of the state change, _and_ its root
qdisc belongs in the taprio list).

The 'found' variable also doesn't really serve too much of a purpose
either; we can just call taprio_set_picos_per_byte() within the loop,
and exit immediately afterwards.

Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
---
 net/sched/sch_taprio.c | 13 ++++---------
 1 file changed, 4 insertions(+), 9 deletions(-)

diff --git a/net/sched/sch_taprio.c b/net/sched/sch_taprio.c
index 136ae21ebce9..0bc6d90e1e51 100644
--- a/net/sched/sch_taprio.c
+++ b/net/sched/sch_taprio.c
@@ -1072,9 +1072,7 @@ static int taprio_dev_notifier(struct notifier_block *nb, unsigned long event,
 			       void *ptr)
 {
 	struct net_device *dev = netdev_notifier_info_to_dev(ptr);
-	struct net_device *qdev;
 	struct taprio_sched *q;
-	bool found = false;
 
 	ASSERT_RTNL();
 
@@ -1082,15 +1080,12 @@ static int taprio_dev_notifier(struct notifier_block *nb, unsigned long event,
 		return NOTIFY_DONE;
 
 	list_for_each_entry(q, &taprio_list, taprio_list) {
-		qdev = qdisc_dev(q->root);
-		if (qdev == dev) {
-			found = true;
-			break;
-		}
-	}
+		if (dev != qdisc_dev(q->root))
+			continue;
 
-	if (found)
 		taprio_set_picos_per_byte(dev, q);
+		break;
+	}
 
 	return NOTIFY_DONE;
 }
-- 
2.34.1