OSDN Git Service

XArray: Turn xa_erase into an exported function
authorMatthew Wilcox <willy@infradead.org>
Mon, 5 Nov 2018 20:48:49 +0000 (15:48 -0500)
committerMatthew Wilcox <willy@infradead.org>
Mon, 5 Nov 2018 21:38:09 +0000 (16:38 -0500)
Make xa_erase() take the spinlock and then call __xa_erase(), but make
it out of line since it's such a common function.

Signed-off-by: Matthew Wilcox <willy@infradead.org>
include/linux/xarray.h
lib/xarray.c

index 8e59d4f..4c839c1 100644 (file)
@@ -289,6 +289,7 @@ struct xarray {
 void xa_init_flags(struct xarray *, gfp_t flags);
 void *xa_load(struct xarray *, unsigned long index);
 void *xa_store(struct xarray *, unsigned long index, void *entry, gfp_t);
+void *xa_erase(struct xarray *, unsigned long index);
 void *xa_store_range(struct xarray *, unsigned long first, unsigned long last,
                        void *entry, gfp_t);
 bool xa_get_mark(struct xarray *, unsigned long index, xa_mark_t);
@@ -341,23 +342,6 @@ static inline bool xa_marked(const struct xarray *xa, xa_mark_t mark)
 }
 
 /**
- * xa_erase() - Erase this entry from the XArray.
- * @xa: XArray.
- * @index: Index of entry.
- *
- * This function is the equivalent of calling xa_store() with %NULL as
- * the third argument.  The XArray does not need to allocate memory, so
- * the user does not need to provide GFP flags.
- *
- * Context: Process context.  Takes and releases the xa_lock.
- * Return: The entry which used to be at this index.
- */
-static inline void *xa_erase(struct xarray *xa, unsigned long index)
-{
-       return xa_store(xa, index, NULL, 0);
-}
-
-/**
  * xa_for_each() - Iterate over a portion of an XArray.
  * @xa: XArray.
  * @entry: Entry retrieved from array.
index 77671d4..b55aa8c 100644 (file)
@@ -1337,6 +1337,30 @@ void *__xa_erase(struct xarray *xa, unsigned long index)
 EXPORT_SYMBOL(__xa_erase);
 
 /**
+ * xa_erase() - Erase this entry from the XArray.
+ * @xa: XArray.
+ * @index: Index of entry.
+ *
+ * This function is the equivalent of calling xa_store() with %NULL as
+ * the third argument.  The XArray does not need to allocate memory, so
+ * the user does not need to provide GFP flags.
+ *
+ * Context: Any context.  Takes and releases the xa_lock.
+ * Return: The entry which used to be at this index.
+ */
+void *xa_erase(struct xarray *xa, unsigned long index)
+{
+       void *entry;
+
+       xa_lock(xa);
+       entry = __xa_erase(xa, index);
+       xa_unlock(xa);
+
+       return entry;
+}
+EXPORT_SYMBOL(xa_erase);
+
+/**
  * xa_store() - Store this entry in the XArray.
  * @xa: XArray.
  * @index: Index into array.