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:   Jay Vosburgh <jay.vosburgh@canonical.com>
To:     Wang Ming <machel@vivo.com>
cc:     Andy Gospodarek <andy@greyhouse.net>,
        "David S. Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>, netdev@vger.kernel.org,
        linux-kernel@vger.kernel.org, opensource.kernel@vivo.com
Subject: Re: [PATCH net v2] net: bonding: Remove error checking for debugfs_create_dir()
In-reply-to: <20230714090856.11571-1-machel@vivo.com>
References: <20230714090856.11571-1-machel@vivo.com>
Comments: In-reply-to Wang Ming <machel@vivo.com>
   message dated "Fri, 14 Jul 2023 17:08:42 +0800."
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-ID: <3369.1689351142.1@famine>
Content-Transfer-Encoding: quoted-printable
Date:   Fri, 14 Jul 2023 09:12:22 -0700
Message-ID: <3370.1689351142@famine>
Precedence: bulk

Wang Ming <machel@vivo.com> wrote:

>It is expected that most callers should _ignore_ the errors
>return by debugfs_create_dir() in bond_debug_reregister().

	Why should the error be ignored?  It's not a fatal error, in the
sense that the bond itself should be unregistered, but I'm not sure why
an error message that the debugfs registration failed is undesirable.

	Also, the code in question is in bond_create_debugfs(), not
bond_debug_reregister().  The diff below looks a bit odd in that the
context line lists _reregister, but that's not the function being
changed.

	I thought the v1 patch was fine.

	-J

>Signed-off-by: Wang Ming <machel@vivo.com>
>---
> drivers/net/bonding/bond_debugfs.c | 3 ---
> 1 file changed, 3 deletions(-)
>
>diff --git a/drivers/net/bonding/bond_debugfs.c b/drivers/net/bonding/bon=
d_debugfs.c
>index 594094526648..a41f765420dc 100644
>--- a/drivers/net/bonding/bond_debugfs.c
>+++ b/drivers/net/bonding/bond_debugfs.c
>@@ -87,9 +87,6 @@ void bond_debug_reregister(struct bonding *bond)
> void bond_create_debugfs(void)
> {
> 	bonding_debug_root =3D debugfs_create_dir("bonding", NULL);
>-
>-	if (!bonding_debug_root)
>-		pr_warn("Warning: Cannot create bonding directory in debugfs\n");
> }
> =

> void bond_destroy_debugfs(void)
>-- =

>2.25.1
>

---
	-Jay Vosburgh, jay.vosburgh@canonical.com