bonding: remove redundant NULL check in debugfs function
authorZhengchao Shao <shaozhengchao@huawei.com>
Thu, 10 Aug 2023 13:50:05 +0000 (21:50 +0800)
committerDavid S. Miller <davem@davemloft.net>
Fri, 11 Aug 2023 10:10:55 +0000 (11:10 +0100)
Because debugfs_create_dir returns ERR_PTR, so bonding_debug_root will
never be NULL. Remove redundant NULL check for bonding_debug_root in
debugfs function. The later debugfs_create_dir/debugfs_remove_recursive
/debugfs_remove_recursive functions will check the dentry with IS_ERR().

Signed-off-by: Zhengchao Shao <shaozhengchao@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/bonding/bond_debugfs.c

index 50e48136c6977fe895ca6fc87b2ef3d9c50b7735..b19492a7f6ad133c53ad68a58386704a19c653b8 100644 (file)
@@ -49,9 +49,6 @@ DEFINE_SHOW_ATTRIBUTE(bond_debug_rlb_hash);
 
 void bond_debug_register(struct bonding *bond)
 {
-       if (!bonding_debug_root)
-               return;
-
        bond->debug_dir =
                debugfs_create_dir(bond->dev->name, bonding_debug_root);
 
@@ -61,9 +58,6 @@ void bond_debug_register(struct bonding *bond)
 
 void bond_debug_unregister(struct bonding *bond)
 {
-       if (!bonding_debug_root)
-               return;
-
        debugfs_remove_recursive(bond->debug_dir);
 }
 
@@ -71,9 +65,6 @@ void bond_debug_reregister(struct bonding *bond)
 {
        struct dentry *d;
 
-       if (!bonding_debug_root)
-               return;
-
        d = debugfs_rename(bonding_debug_root, bond->debug_dir,
                           bonding_debug_root, bond->dev->name);
        if (!IS_ERR(d)) {