]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commitdiff
lib/radix-tree: remove unused argument of insert_entries
authorwuchi <wuchi.zero@gmail.com>
Sat, 25 Jun 2022 13:53:24 +0000 (21:53 +0800)
committerakpm <akpm@linux-foundation.org>
Mon, 18 Jul 2022 00:31:38 +0000 (17:31 -0700)
insert_entries() doesn't use the 'bool replace' argument, and the function
is only used locally, remove the argument.

The historical context of the unused argument is as follow:

2: commit <3a08cd52c37c79> (radix tree: Remove multiorder support)
  Remove the code related to macro CONFIG_RADIX_TREE_MULTIORDER
to convert to the xArray.
  Without the macro, there is no need to retain the argument.

1: commit <175542f575723e> (radix-tree: add radix_tree_join)
  Add insert_entries(..., bool replace) function, depending on the
macro CONFIG_RADIX_TREE_MULTIORDER definition, the implementation
is different. Notice that the implementation without the macro doesn't
use the argument.

[Matthew Wilcox: add historical context for argument]

Link: https://lkml.kernel.org/r/20220625135324.72574-1-wuchi.zero@gmail.com
Signed-off-by: wuchi <wuchi.zero@gmail.com>
Cc: Matthew Wilcox <willy@infradead.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
lib/radix-tree.c

index b3afafe46fffbca7d7afff3b645b47e1c6faae23..3c78e1e8b2ad60ead8b598eb03fc26b1da431155 100644 (file)
@@ -677,7 +677,7 @@ static void radix_tree_free_nodes(struct radix_tree_node *node)
 }
 
 static inline int insert_entries(struct radix_tree_node *node,
-               void __rcu **slot, void *item, bool replace)
+               void __rcu **slot, void *item)
 {
        if (*slot)
                return -EEXIST;
@@ -711,7 +711,7 @@ int radix_tree_insert(struct radix_tree_root *root, unsigned long index,
        if (error)
                return error;
 
-       error = insert_entries(node, slot, item, false);
+       error = insert_entries(node, slot, item);
        if (error < 0)
                return error;