From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 506AFC433EF for ; Sat, 2 Apr 2022 12:15:41 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S242324AbiDBMRa (ORCPT ); Sat, 2 Apr 2022 08:17:30 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:48862 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S242039AbiDBMRZ (ORCPT ); Sat, 2 Apr 2022 08:17:25 -0400 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id 845651905A8 for ; Sat, 2 Apr 2022 05:15:33 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1648901732; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding; bh=xZCY9WwfUJ1wQ7ihdlt+WF3BvYNQQqJJlaiT88Q2K8g=; b=YYOUxYYmlUWu8nNmEyD+NEEG6sVbKWt3HGyiwYKIxIvz1r6z7uH2kLXNiMEks8by1xGFv4 hz75H0b9s4cAbyaujCkRS23mMSJYRX2HMCKvIXnI9iFiFy1KlTfaI/ctglq4chieUI3uw2 hsXbRKWOZQM39Yva1jHnajqcOi/UTx4= Received: from mail-qk1-f200.google.com (mail-qk1-f200.google.com [209.85.222.200]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-556-BIGrwt_RN6aAGYtNbSU1AA-1; Sat, 02 Apr 2022 08:15:31 -0400 X-MC-Unique: BIGrwt_RN6aAGYtNbSU1AA-1 Received: by mail-qk1-f200.google.com with SMTP id y140-20020a376492000000b0067b14129a63so3397782qkb.9 for ; Sat, 02 Apr 2022 05:15:31 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=xZCY9WwfUJ1wQ7ihdlt+WF3BvYNQQqJJlaiT88Q2K8g=; b=8RhlcCpWwH2AkEefwgS/Jnhftn/zNGUT1mrvFBR3uJWkSOdo46NGy/dPLhkWsYARUe U140LkrUxPXSK2wtrogc8uPS6fXfu9HX+ak/UHxkT0oTg+GNiQsRulxwe74PMM7Aftdp A1UO+ds1tvTXYSgtSEkStzzeBBaN+XUQwhvj4i8gYgKXRTODtMMWacV92jOla6ksnd0u SVJ/CDGeyS2ZmedQNarX6AAihZkyhq0pNZSHm03Pq7ELH0RXibXUQxaCJX0iRwiXcr8Y LnxCpQXM5maL4/fyE3XCBUc8hkAMDh8YbUTfu/MIRS2Ml/cEIsOZpnUyNiEO64KLpYFf Lc8g== X-Gm-Message-State: AOAM532g9HP7MbaS7iCIupNEzs2+IWu9bd43BKT6lx21MVIAolmdfuk7 kKaixcUFYS7ZewevVIess6cUItdvAd8d/TKEQsbvQNmt8BzNHXvYpe5Ve9Pc399URwOizxOQ+BC LjWL9lkcRejgFh454i9aBgpXG X-Received: by 2002:a37:b984:0:b0:67f:64a2:313e with SMTP id j126-20020a37b984000000b0067f64a2313emr9005744qkf.3.1648901730804; Sat, 02 Apr 2022 05:15:30 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzLoYD71R5m6A8TNCdkEbRY7fG54F4nQY8FDR42H5wwtz/WpIZY5yf5K/GUNQ6h3p03DP/F1g== X-Received: by 2002:a37:b984:0:b0:67f:64a2:313e with SMTP id j126-20020a37b984000000b0067f64a2313emr9005725qkf.3.1648901730578; Sat, 02 Apr 2022 05:15:30 -0700 (PDT) Received: from dell-per740-01.7a2m.lab.eng.bos.redhat.com (nat-pool-bos-t.redhat.com. [66.187.233.206]) by smtp.gmail.com with ESMTPSA id g21-20020ac85815000000b002e06e2623a7sm3776791qtg.0.2022.04.02.05.15.29 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 02 Apr 2022 05:15:29 -0700 (PDT) From: Tom Rix To: idosch@nvidia.com, petrm@nvidia.com, davem@davemloft.net, kuba@kernel.org, pabeni@redhat.com Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, Tom Rix Subject: [PATCH] mlxsw: spectrum_router: simplify list unwinding Date: Sat, 2 Apr 2022 08:15:16 -0400 Message-Id: <20220402121516.2750284-1-trix@redhat.com> X-Mailer: git-send-email 2.27.0 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The setting of i here err_nexthop6_group_get: i = nrt6; Is redundant, i is already nrt6. So remove this statement. The for loop for the unwinding err_rt6_create: for (i--; i >= 0; i--) { Is equivelent to for (; i > 0; i--) { Two consecutive labels can be reduced to one. Signed-off-by: Tom Rix --- .../ethernet/mellanox/mlxsw/spectrum_router.c | 20 ++++++++----------- 1 file changed, 8 insertions(+), 12 deletions(-) diff --git a/drivers/net/ethernet/mellanox/mlxsw/spectrum_router.c b/drivers/net/ethernet/mellanox/mlxsw/spectrum_router.c index 79deb19e3a19..79fd486e29e3 100644 --- a/drivers/net/ethernet/mellanox/mlxsw/spectrum_router.c +++ b/drivers/net/ethernet/mellanox/mlxsw/spectrum_router.c @@ -7010,7 +7010,7 @@ mlxsw_sp_fib6_entry_nexthop_add(struct mlxsw_sp *mlxsw_sp, mlxsw_sp_rt6 = mlxsw_sp_rt6_create(rt_arr[i]); if (IS_ERR(mlxsw_sp_rt6)) { err = PTR_ERR(mlxsw_sp_rt6); - goto err_rt6_create; + goto err_rt6_unwind; } list_add_tail(&mlxsw_sp_rt6->list, &fib6_entry->rt6_list); @@ -7019,14 +7019,12 @@ mlxsw_sp_fib6_entry_nexthop_add(struct mlxsw_sp *mlxsw_sp, err = mlxsw_sp_nexthop6_group_update(mlxsw_sp, op_ctx, fib6_entry); if (err) - goto err_nexthop6_group_update; + goto err_rt6_unwind; return 0; -err_nexthop6_group_update: - i = nrt6; -err_rt6_create: - for (i--; i >= 0; i--) { +err_rt6_unwind: + for (; i > 0; i--) { fib6_entry->nrt6--; mlxsw_sp_rt6 = list_last_entry(&fib6_entry->rt6_list, struct mlxsw_sp_rt6, list); @@ -7154,7 +7152,7 @@ mlxsw_sp_fib6_entry_create(struct mlxsw_sp *mlxsw_sp, mlxsw_sp_rt6 = mlxsw_sp_rt6_create(rt_arr[i]); if (IS_ERR(mlxsw_sp_rt6)) { err = PTR_ERR(mlxsw_sp_rt6); - goto err_rt6_create; + goto err_rt6_unwind; } list_add_tail(&mlxsw_sp_rt6->list, &fib6_entry->rt6_list); fib6_entry->nrt6++; @@ -7162,7 +7160,7 @@ mlxsw_sp_fib6_entry_create(struct mlxsw_sp *mlxsw_sp, err = mlxsw_sp_nexthop6_group_get(mlxsw_sp, fib6_entry); if (err) - goto err_nexthop6_group_get; + goto err_rt6_unwind; err = mlxsw_sp_nexthop_group_vr_link(fib_entry->nh_group, fib_node->fib); @@ -7181,10 +7179,8 @@ mlxsw_sp_fib6_entry_create(struct mlxsw_sp *mlxsw_sp, mlxsw_sp_nexthop_group_vr_unlink(fib_entry->nh_group, fib_node->fib); err_nexthop_group_vr_link: mlxsw_sp_nexthop6_group_put(mlxsw_sp, fib_entry); -err_nexthop6_group_get: - i = nrt6; -err_rt6_create: - for (i--; i >= 0; i--) { +err_rt6_unwind: + for (; i > 0; i--) { fib6_entry->nrt6--; mlxsw_sp_rt6 = list_last_entry(&fib6_entry->rt6_list, struct mlxsw_sp_rt6, list); -- 2.27.0