* [PATCH] radix-tree: move declarations to header
@ 2023-05-16 19:41 Arnd Bergmann
2023-05-21 9:54 ` [PATCH] radix tree test suite: Fix building radix tree test suite Peng Zhang
0 siblings, 1 reply; 3+ messages in thread
From: Arnd Bergmann @ 2023-05-16 19:41 UTC (permalink / raw)
To: Matthew Wilcox; +Cc: Arnd Bergmann, Andrew Morton, linux-kernel, linux-fsdevel
From: Arnd Bergmann <arnd@arndb.de>
The xarray.c file contains the only call to radix_tree_node_rcu_free(),
and it comes with its own extern declaration for it. This means the
function definition causes a missing-prototype warning:
lib/radix-tree.c:288:6: error: no previous prototype for 'radix_tree_node_rcu_free' [-Werror=missing-prototypes]
Instead, move the declaration for this function to a new header that
can be included by both, and do the same for the radix_tree_node_cachep
variable that has the same underlying problem but does not cause
a warning with gcc.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
lib/radix-tree.c | 2 ++
lib/radix-tree.h | 8 ++++++++
lib/xarray.c | 6 ++----
3 files changed, 12 insertions(+), 4 deletions(-)
create mode 100644 lib/radix-tree.h
diff --git a/lib/radix-tree.c b/lib/radix-tree.c
index 049ba132f7ef..1a31065b2036 100644
--- a/lib/radix-tree.c
+++ b/lib/radix-tree.c
@@ -27,6 +27,8 @@
#include <linux/string.h>
#include <linux/xarray.h>
+#include "radix-tree.h"
+
/*
* Radix tree node cache.
*/
diff --git a/lib/radix-tree.h b/lib/radix-tree.h
new file mode 100644
index 000000000000..40d5c03e2b09
--- /dev/null
+++ b/lib/radix-tree.h
@@ -0,0 +1,8 @@
+// SPDX-License-Identifier: GPL-2.0+
+/* radix-tree helpers that are only shared with xarray */
+
+struct kmem_cache;
+struct rcu_head;
+
+extern struct kmem_cache *radix_tree_node_cachep;
+extern void radix_tree_node_rcu_free(struct rcu_head *head);
diff --git a/lib/xarray.c b/lib/xarray.c
index ea9ce1f0b386..2071a3718f4e 100644
--- a/lib/xarray.c
+++ b/lib/xarray.c
@@ -12,6 +12,8 @@
#include <linux/slab.h>
#include <linux/xarray.h>
+#include "radix-tree.h"
+
/*
* Coding conventions in this file:
*
@@ -247,10 +249,6 @@ void *xas_load(struct xa_state *xas)
}
EXPORT_SYMBOL_GPL(xas_load);
-/* Move the radix tree node cache here */
-extern struct kmem_cache *radix_tree_node_cachep;
-extern void radix_tree_node_rcu_free(struct rcu_head *head);
-
#define XA_RCU_FREE ((struct xarray *)1)
static void xa_node_free(struct xa_node *node)
--
2.39.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH] radix tree test suite: Fix building radix tree test suite.
2023-05-16 19:41 [PATCH] radix-tree: move declarations to header Arnd Bergmann
@ 2023-05-21 9:54 ` Peng Zhang
2023-05-21 18:45 ` Andrew Morton
0 siblings, 1 reply; 3+ messages in thread
From: Peng Zhang @ 2023-05-21 9:54 UTC (permalink / raw)
To: arnd, willy; +Cc: akpm, linux-kernel, linux-fsdevel, Liam.Howlett, Peng Zhang
The build of radix tree test suite failed due to a new internal header
file added to radix-tree.c. Adding the header directory in the Makefile
fixes it.
Signed-off-by: Peng Zhang <zhangpeng.00@bytedance.com>
---
tools/testing/radix-tree/Makefile | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/tools/testing/radix-tree/Makefile b/tools/testing/radix-tree/Makefile
index caf32a9b9608..7527f738b4a1 100644
--- a/tools/testing/radix-tree/Makefile
+++ b/tools/testing/radix-tree/Makefile
@@ -1,7 +1,7 @@
# SPDX-License-Identifier: GPL-2.0
-CFLAGS += -I. -I../../include -g -Og -Wall -D_LGPL_SOURCE -fsanitize=address \
- -fsanitize=undefined
+CFLAGS += -I. -I../../include -I../../../lib -g -Og -Wall \
+ -D_LGPL_SOURCE -fsanitize=address -fsanitize=undefined
LDFLAGS += -fsanitize=address -fsanitize=undefined
LDLIBS+= -lpthread -lurcu
TARGETS = main idr-test multiorder xarray maple
@@ -49,6 +49,7 @@ $(OFILES): Makefile *.h */*.h generated/map-shift.h generated/bit-length.h \
../../../include/linux/xarray.h \
../../../include/linux/maple_tree.h \
../../../include/linux/radix-tree.h \
+ ../../../lib/radix-tree.h \
../../../include/linux/idr.h
radix-tree.c: ../../../lib/radix-tree.c
--
2.20.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-05-21 18:45 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-05-16 19:41 [PATCH] radix-tree: move declarations to header Arnd Bergmann
2023-05-21 9:54 ` [PATCH] radix tree test suite: Fix building radix tree test suite Peng Zhang
2023-05-21 18:45 ` Andrew Morton
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).