From patchwork Wed Dec 6 00:41:06 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Matthew Wilcox (Oracle)" X-Patchwork-Id: 10094805 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 975C560329 for ; Wed, 6 Dec 2017 01:11:59 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 8467222B1F for ; Wed, 6 Dec 2017 01:11:59 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 7858F27968; Wed, 6 Dec 2017 01:11:59 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.8 required=2.0 tests=BAYES_00,DKIM_SIGNED, RCVD_IN_DNSWL_HI,T_DKIM_INVALID autolearn=unavailable version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id A92A622B1F for ; Wed, 6 Dec 2017 01:11:57 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754201AbdLFBHP (ORCPT ); Tue, 5 Dec 2017 20:07:15 -0500 Received: from bombadil.infradead.org ([65.50.211.133]:59063 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753292AbdLFAmK (ORCPT ); Tue, 5 Dec 2017 19:42:10 -0500 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=bombadil.20170209; h=References:In-Reply-To:Message-Id: Date:Subject:Cc:To:From:Sender:Reply-To:MIME-Version:Content-Type: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=8s8kqLqwdYJcdD3hejoEcfrC2w2xHqes2beW/uKIobk=; b=qAGymfKsXaacIY9Aq5EAcTL4w yZc38LirMiqX0NecgSguZT1hoDbDsbtGkYSw1jWZMeQYJz9pyOKcBEkLcM6DzIwyaaPAkrg9U4+yK 7BSpstv5FseTvxkp6oiDhZxIuZn/pEbj8TR5nI8qZzLxjEC881h/7CrQQURtOIkI1Abi8R8gf2GAn pDR3aV5o45l1dp9rqkx8g5CZyoVqA2dICWfA9Hd1Jev2Rko1CWVkUyN100H2p9eY3PwPu/Kn9LnlQ /cS7ygtOKDC1hwp7zcDTDH57sk4cfYZkXzQQGqEEI3pYN9L1U0V2yxMiKJZOBDNst7mxTHwHAt9Qy Z6AR+1bbg==; Received: from willy by bombadil.infradead.org with local (Exim 4.87 #1 (Red Hat Linux)) id 1eMNmp-00012O-BX; Wed, 06 Dec 2017 00:42:07 +0000 From: Matthew Wilcox Cc: Matthew Wilcox , Ross Zwisler , Jens Axboe , Rehas Sachdeva , linux-mm@kvack.org, linux-fsdevel@vger.kernel.org, linux-f2fs-devel@lists.sourceforge.net, linux-nilfs@vger.kernel.org, linux-btrfs@vger.kernel.org, linux-xfs@vger.kernel.org, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH v4 20/73] idr: Convert to XArray Date: Tue, 5 Dec 2017 16:41:06 -0800 Message-Id: <20171206004159.3755-21-willy@infradead.org> X-Mailer: git-send-email 2.9.5 In-Reply-To: <20171206004159.3755-1-willy@infradead.org> References: <20171206004159.3755-1-willy@infradead.org> To: unlisted-recipients:; (no To-header on input) Sender: linux-btrfs-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-btrfs@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP From: Matthew Wilcox The IDR distinguishes between unallocated entries (read as NULL) and entries where the user has chosen to store NULL. The radix tree was modified to consider NULL entries which had tag 0 _clear_ as being allocated, but it added a lot of complexity. Instead, the XArray has a 'zero entry', which the normal API will treat as NULL, but is distinct from NULL when using the advanced API. The IDR code converts between NULL and zero entries. The idr_for_each_entry_ul() iterator becomes an alias for xa_for_each(), so we drop the idr_get_next_ul() function as it has no users. The exported IDR API was a weird mix of GPL-only and general symbols; I converted them all to GPL as there was no way to use the IDR API without being GPL. Signed-off-by: Matthew Wilcox --- Documentation/core-api/xarray.rst | 6 + include/linux/idr.h | 161 +++++++++++++------- include/linux/xarray.h | 27 +++- lib/idr.c | 282 +++++++++++++++++++++--------------- lib/radix-tree.c | 77 +++++----- lib/xarray.c | 6 + tools/testing/radix-tree/idr-test.c | 23 +++ 7 files changed, 367 insertions(+), 215 deletions(-) diff --git a/Documentation/core-api/xarray.rst b/Documentation/core-api/xarray.rst index 871161539242..b252bf3dc23f 100644 --- a/Documentation/core-api/xarray.rst +++ b/Documentation/core-api/xarray.rst @@ -200,6 +200,12 @@ to :c:func:`xas_retry`, and retry the operation if it returns ``true``. this RCU period. You should restart the lookup from the head of the array. + * - Zero + - :c:func:`xa_is_zero` + - Zero entries appear as ``NULL`` through the Normal API, but occupy an + entry in the XArray which can be tagged or otherwise used to reserve + the index. + Other internal entries may be added in the future. As far as possible, they will be handled by :c:func:`xas_retry`. diff --git a/include/linux/idr.h b/include/linux/idr.h index 4ffdb7058121..06412fbaa65f 100644 --- a/include/linux/idr.h +++ b/include/linux/idr.h @@ -9,33 +9,34 @@ * tables. */ -#ifndef __IDR_H__ -#define __IDR_H__ +#ifndef _LINUX_IDR_H +#define _LINUX_IDR_H #include #include #include +#include struct idr { - struct radix_tree_root idr_rt; - unsigned int idr_next; + struct xarray idr_xa; + unsigned int idr_next; }; -/* - * The IDR API does not expose the tagging functionality of the radix tree - * to users. Use tag 0 to track whether a node has free space below it. - */ -#define IDR_FREE 0 - -/* Set the IDR flag and the IDR_FREE tag */ -#define IDR_RT_MARKER ((__force gfp_t)(3 << __GFP_BITS_SHIFT)) +#define IDR_INIT_FLAGS (XA_FLAGS_TRACK_FREE | XA_FLAGS_TAG(0)) #define IDR_INIT(name) \ { \ - .idr_rt = RADIX_TREE_INIT(name, IDR_RT_MARKER) \ + .idr_xa = __XARRAY_INIT(name.idr_xa, IDR_INIT_FLAGS), \ + .idr_next = 0, \ } #define DEFINE_IDR(name) struct idr name = IDR_INIT(name) +static inline void idr_init(struct idr *idr) +{ + __xa_init(&idr->idr_xa, IDR_INIT_FLAGS); + idr->idr_next = 0; +} + /** * idr_get_cursor - Return the current position of the cyclic allocator * @idr: idr handle @@ -64,62 +65,97 @@ static inline void idr_set_cursor(struct idr *idr, unsigned int val) /** * DOC: idr sync - * idr synchronization (stolen from radix-tree.h) - * - * idr_find() is able to be called locklessly, using RCU. The caller must - * ensure calls to this function are made within rcu_read_lock() regions. - * Other readers (lock-free or otherwise) and modifications may be running - * concurrently. - * - * It is still required that the caller manage the synchronization and - * lifetimes of the items. So if RCU lock-free lookups are used, typically - * this would mean that the items have their own locks, or are amenable to - * lock-free access; and that the items are freed by RCU (or only freed after - * having been deleted from the idr tree *and* a synchronize_rcu() grace - * period). + * idr synchronization + * + * The IDR manages its own locking, using irqsafe spinlocks for operations + * which modify the IDR and RCU for operations which do not. The user of + * the IDR may choose to wrap accesses to it in a lock if it needs to + * guarantee the IDR does not change during a read access. The easiest way + * to do this is to grab the same lock the IDR uses for write accesses + * using one of the idr_lock() wrappers. + * + * The caller must still manage the synchronization and lifetimes of the + * items. So if RCU lock-free lookups are used, typically this would mean + * that the items have their own locks, or are amenable to lock-free access; + * and that the items are freed by RCU (or only freed after having been + * deleted from the IDR *and* a synchronize_rcu() grace period has elapsed). */ -void idr_preload(gfp_t gfp_mask); +#define idr_lock(idr) xa_lock(&(idr)->idr_xa) +#define idr_unlock(idr) xa_unlock(&(idr)->idr_xa) +#define idr_lock_bh(idr) xa_lock_bh(&(idr)->idr_xa) +#define idr_unlock_bh(idr) xa_unlock_bh(&(idr)->idr_xa) +#define idr_lock_irq(idr) xa_lock_irq(&(idr)->idr_xa) +#define idr_unlock_irq(idr) xa_unlock_irq(&(idr)->idr_xa) +#define idr_lock_irqsave(idr, flags) \ + xa_lock_irqsave(&(idr)->idr_xa, flags) +#define idr_unlock_irqrestore(idr, flags) \ + xa_unlock_irqrestore(&(idr)->idr_xa, flags) + +void idr_preload(gfp_t); int idr_alloc(struct idr *, void *, int start, int end, gfp_t); int __must_check idr_alloc_ul(struct idr *, void *, unsigned long *nextid, unsigned long max, gfp_t); int idr_alloc_cyclic(struct idr *, void *entry, int start, int end, gfp_t); -int idr_for_each(const struct idr *, +int idr_for_each(struct idr *, int (*fn)(int id, void *p, void *data), void *data); void *idr_get_next(struct idr *, int *nextid); -void *idr_get_next_ul(struct idr *, unsigned long *nextid); void *idr_replace(struct idr *, void *, unsigned long id); -void idr_destroy(struct idr *); +#ifdef CONFIG_64BIT +int __must_check idr_alloc_u32(struct idr *, void *, unsigned int *nextid, + unsigned int max, gfp_t); +#else /* !CONFIG_64BIT */ static inline int __must_check idr_alloc_u32(struct idr *idr, void *ptr, - u32 *nextid, unsigned long max, gfp_t gfp) + unsigned int *nextid, unsigned int max, gfp_t gfp) { - unsigned long tmp = *nextid; - int ret = idr_alloc_ul(idr, ptr, &tmp, max, gfp); - *nextid = tmp; - return ret; + return idr_alloc_ul(idr, ptr, (unsigned long *)nextid, max, gfp); } +#endif +/** + * idr_remove() - Remove an item from the IDR. + * @idr: IDR handle. + * @id: Object ID. + * + * Once this function returns, the ID is available for allocation again. + * This function protects itself with the IDR lock. + * + * Return: The pointer associated with this ID. + */ static inline void *idr_remove(struct idr *idr, unsigned long id) { - return radix_tree_delete_item(&idr->idr_rt, id, NULL); + return xa_erase(&idr->idr_xa, id); } -static inline void idr_init(struct idr *idr) +/** + * idr_is_empty() - Determine if there are no entries in the IDR + * @idr: IDR handle. + * + * Return: %true if there are no entries in the IDR. + */ +static inline bool idr_is_empty(const struct idr *idr) { - INIT_RADIX_TREE(&idr->idr_rt, IDR_RT_MARKER); - idr->idr_next = 0; + return xa_empty(&idr->idr_xa); } -static inline bool idr_is_empty(const struct idr *idr) +/** + * idr_destroy() - Free all internal memory used by an IDR. + * @idr: IDR handle. + * + * When you have finished using an IDR, you can free all the memory used + * for the IDR data structure by calling this function. If you also + * wish to free the objects referenced by the IDR, you can use idr_for_each() + * or idr_for_each_entry() to do that first. + */ +static inline void idr_destroy(struct idr *idr) { - return radix_tree_empty(&idr->idr_rt) && - radix_tree_tagged(&idr->idr_rt, IDR_FREE); + xa_destroy(&idr->idr_xa); } /** - * idr_preload_end - end preload section started with idr_preload() + * idr_preload_end() - end preload section started with idr_preload() * * Each idr_preload() should be matched with an invocation of this * function. See idr_preload() for details. @@ -130,7 +166,7 @@ static inline void idr_preload_end(void) } /** - * idr_find - return pointer for given id + * idr_find() - return pointer for given id * @idr: idr handle * @id: lookup key * @@ -138,14 +174,35 @@ static inline void idr_preload_end(void) * return indicates that @id is not valid or you passed %NULL in * idr_get_new(). * - * This function can be called under rcu_read_lock(), given that the leaf - * pointers lifetimes are correctly managed. + * This function is protected by the RCU read lock. If you want to ensure + * that it does not race with a call to idr_remove(), perhaps because you + * need to establish a refcount on the object, you can use idr_lock() and + * idr_unlock() to prevent simultaneous modification. */ -static inline void *idr_find(const struct idr *idr, unsigned long id) +static inline void *idr_find(struct idr *idr, unsigned long id) { - return radix_tree_lookup(&idr->idr_rt, id); + return xa_load(&idr->idr_xa, id); } +/** + * idr_for_each_entry_ul() - Iterate over the entries in an IDR. + * @idr: IDR handle. + * @entry: Pointer to each entry in turn. + * @id: ID of each entry. + * + * Initialise @id to the lowest ID before using this iterator. + * In the body of the loop, @entry will point to the object stored in the + * IDR. After the loop has finished normally, @entry will be %NULL, which + * is a convenient way to distinguish between a 'break' exit from the loop + * and normal termination. + * + * The control elements of this loop protect themselves with the RCU read + * lock, which is dropped before invoking the body. You may sleep unless + * your own locking prevents that. + */ +#define idr_for_each_entry_ul(idr, entry, id) \ + xa_for_each(&(idr)->idr_xa, entry, id, ULONG_MAX) + /** * idr_for_each_entry - iterate over an idr's elements of a given type * @idr: idr handle @@ -158,8 +215,6 @@ static inline void *idr_find(const struct idr *idr, unsigned long id) */ #define idr_for_each_entry(idr, entry, id) \ for (id = 0; ((entry) = idr_get_next(idr, &(id))) != NULL; ++id) -#define idr_for_each_entry_ul(idr, entry, id) \ - for (id = 0; ((entry) = idr_get_next_ul(idr, &(id))) != NULL; ++id) /** * idr_for_each_entry_continue - continue iteration over an idr's elements of a given type @@ -194,7 +249,7 @@ struct ida { }; #define IDA_INIT(name) { \ - .ida_rt = RADIX_TREE_INIT(name, IDR_RT_MARKER | GFP_NOWAIT), \ + .ida_rt = RADIX_TREE_INIT(name, IDR_INIT_FLAGS | GFP_NOWAIT), \ } #define DEFINE_IDA(name) struct ida name = IDA_INIT(name) @@ -209,7 +264,7 @@ void ida_simple_remove(struct ida *ida, unsigned int id); static inline void ida_init(struct ida *ida) { - INIT_RADIX_TREE(&ida->ida_rt, IDR_RT_MARKER | GFP_NOWAIT); + INIT_RADIX_TREE(&ida->ida_rt, IDR_INIT_FLAGS | GFP_NOWAIT); } /** @@ -228,4 +283,4 @@ static inline bool ida_is_empty(const struct ida *ida) { return radix_tree_empty(&ida->ida_rt); } -#endif /* __IDR_H__ */ +#endif /* _LINUX_IDR_H */ diff --git a/include/linux/xarray.h b/include/linux/xarray.h index afa3374f20bd..7017153d89e8 100644 --- a/include/linux/xarray.h +++ b/include/linux/xarray.h @@ -331,7 +331,8 @@ static inline void *xa_entry_locked(struct xarray *xa, * internal entries are pointers to the next node in the tree. Since the * kernel unmaps page 0 to trap NULL pointer dereferences, we can use values * 0-1023 for special purposes. Values 0-62 are used for sibling - * entries. Value 256 is used for the retry entry. + * entries. Value 256 is used for zero entries. Value 257 is used for the + * retry entry. */ /* Private */ @@ -400,7 +401,19 @@ static inline bool xa_is_sibling(void *entry) (entry < xa_mk_sibling(XA_CHUNK_SIZE - 1)); } -#define XA_RETRY_ENTRY xa_mk_internal(256) +#define XA_ZERO_ENTRY xa_mk_internal(256) +#define XA_RETRY_ENTRY xa_mk_internal(257) + +/** + * xa_is_zero() - Is the entry a zero entry? + * @entry: Entry retrieved from the XArray + * + * Return: %true if the entry is a zero entry. + */ +static inline bool xa_is_zero(void *entry) +{ + return unlikely(entry == XA_ZERO_ENTRY); +} /** * xa_is_retry() - Is the entry a retry entry? @@ -562,18 +575,20 @@ static inline bool xas_top(struct xa_node *node) } /** - * xas_retry() - Handle a retry entry. + * xas_retry() - Retry the operation if appropriate. * @xas: XArray operation state. * @entry: Entry from xarray. * - * An RCU-protected read may see a retry entry as a side-effect of a - * simultaneous modification. This function sets up the @xas to retry - * the walk from the head of the array. + * The advanced functions may sometimes return an internal entry, such as + * a retry entry or a zero entry. This function sets up the @xas to restart + * the walk from the head of the array if needed. * * Return: true if the operation needs to be retried. */ static inline bool xas_retry(struct xa_state *xas, void *entry) { + if (xa_is_zero(entry)) + return true; if (!xa_is_retry(entry)) return false; xas->xa_node = XAS_RESTART; diff --git a/lib/idr.c b/lib/idr.c index b9aa08e198a2..e677d1869ead 100644 --- a/lib/idr.c +++ b/lib/idr.c @@ -8,67 +8,121 @@ DEFINE_PER_CPU(struct ida_bitmap *, ida_bitmap); static DEFINE_SPINLOCK(simple_ida_lock); +/* In radix-tree.c temporarily */ +extern bool idr_nomem(struct xa_state *, gfp_t); + /** - * idr_alloc_ul() - allocate a large ID - * @idr: idr handle - * @ptr: pointer to be associated with the new ID - * @nextid: Pointer to minimum ID to allocate - * @max: the maximum ID (inclusive) - * @gfp: memory allocation flags + * idr_alloc_ul() - Allocate a large ID. + * @idr: IDR handle. + * @ptr: Pointer to be associated with the new ID. + * @nextid: Pointer to minimum ID to allocate. + * @max: The maximum ID (inclusive). + * @gfp: Memory allocation flags. * * Allocates an unused ID in the range [*nextid, end] and stores it in * @nextid. Note that @max differs from the @end parameter to idr_alloc(). * - * Simultaneous modifications to the @idr are not allowed and should be - * prevented by the user, usually with a lock. idr_alloc_ul() may be called - * concurrently with read-only accesses to the @idr, such as idr_find() and - * idr_for_each_entry(). + * The IDR uses its own spinlock to protect against simultaneous + * modification. @nextid is assigned to before @ptr is stored in the IDR; + * if @nextid points into the object referenced by @ptr, it will not be + * possible for a simultaneous lookup to see the wrong value in @nextid. * - * Return: 0 on success or a negative errno on failure (ENOMEM or ENOSPC) + * Return: 0 on success or a negative errno on failure (ENOMEM or ENOSPC). */ int idr_alloc_ul(struct idr *idr, void *ptr, unsigned long *nextid, unsigned long max, gfp_t gfp) { - struct radix_tree_iter iter; - void __rcu **slot; + XA_STATE(xas, &idr->idr_xa, *nextid); + unsigned long flags; - if (WARN_ON_ONCE(radix_tree_is_internal_node(ptr))) + if (WARN_ON_ONCE(xa_is_internal(ptr))) return -EINVAL; + if (!ptr) + ptr = XA_ZERO_ENTRY; + + do { + xas_lock_irqsave(&xas, flags); + xas_find_tag(&xas, max, XA_FREE_TAG); + if (xas.xa_index > max) + xas_set_err(&xas, -ENOSPC); + else + *nextid = xas.xa_index; + xas_store(&xas, ptr); + xas_clear_tag(&xas, XA_FREE_TAG); + xas_unlock_irqrestore(&xas, flags); + } while (idr_nomem(&xas, gfp)); + + return xas_error(&xas); +} +EXPORT_SYMBOL_GPL(idr_alloc_ul); - if (WARN_ON_ONCE(!(idr->idr_rt.xa_flags & ROOT_IS_IDR))) - idr->idr_rt.xa_flags |= IDR_RT_MARKER; - - radix_tree_iter_init(&iter, *nextid); - slot = idr_get_free(&idr->idr_rt, &iter, gfp, max); - if (IS_ERR(slot)) - return PTR_ERR(slot); - - radix_tree_iter_replace(&idr->idr_rt, &iter, slot, ptr); - radix_tree_iter_tag_clear(&idr->idr_rt, &iter, IDR_FREE); +/** + * idr_alloc_u32() - Allocate an ID. + * @idr: IDR handle. + * @ptr: Pointer to be associated with the new ID. + * @nextid: Pointer to minimum ID to allocate. + * @max: The maximum ID (inclusive). + * @gfp: Memory allocation flags. + * + * Allocates an unused ID in the range [*nextid, end] and stores it in + * @nextid. Note that @max differs from the @end parameter to idr_alloc(). + * + * The IDR uses its own spinlock to protect against simultaneous + * modification. @nextid is assigned to before @ptr is stored in the IDR; + * if @nextid points into the object referenced by @ptr, it will not be + * possible for a simultaneous lookup to see the wrong value in @nextid. + * + * Return: 0 on success or a negative errno on failure (ENOMEM or ENOSPC). + */ +#ifdef CONFIG_64BIT +int idr_alloc_u32(struct idr *idr, void *ptr, unsigned int *nextid, + unsigned int max, gfp_t gfp) +{ + XA_STATE(xas, &idr->idr_xa, *nextid); + unsigned long flags; - *nextid = iter.index; - return 0; + if (WARN_ON_ONCE(xa_is_internal(ptr))) + return -EINVAL; + if (!ptr) + ptr = XA_ZERO_ENTRY; + + do { + xas_lock_irqsave(&xas, flags); + xas_find_tag(&xas, max, XA_FREE_TAG); + if (xas.xa_index > max) + xas_set_err(&xas, -ENOSPC); + else + *nextid = xas.xa_index; + xas_store(&xas, ptr); + xas_clear_tag(&xas, XA_FREE_TAG); + xas_unlock_irqrestore(&xas, flags); + } while (idr_nomem(&xas, gfp)); + + return xas_error(&xas); } -EXPORT_SYMBOL_GPL(idr_alloc_ul); +EXPORT_SYMBOL_GPL(idr_alloc_u32); +#endif /** - * idr_alloc - allocate an id - * @idr: idr handle - * @ptr: pointer to be associated with the new id - * @start: the minimum id (inclusive) - * @end: the maximum id (exclusive) - * @gfp: memory allocation flags + * idr_alloc() - Allocate an ID. + * @idr: IDR handle. + * @ptr: Pointer to be associated with the new ID. + * @start: The minimum id (inclusive). + * @end: The maximum id (exclusive). + * @gfp: Memory allocation flags. + * + * Allocates an unused ID >= start and < end. * - * Allocates an unused ID in the range [start, end). Returns -ENOSPC - * if there are no unused IDs in that range. + * If @end is <= 0, it is treated as %INT_MAX + 1. This is to always + * allow using @start + N as @end as long as N is <= %INT_MAX. This + * differs from the @max parameter to idr_alloc_ul() and idr_alloc_u32(). * - * Note that @end is treated as max when <= 0. This is to always allow - * using @start + N as @end as long as N is inside integer range. + * The IDR uses its own spinlock to protect against simultaneous + * modification. The @ptr is visible to other simultaneous readers + * like idr_find() before this function returns. * - * Simultaneous modifications to the @idr are not allowed and should be - * prevented by the user, usually with a lock. idr_alloc() may be called - * concurrently with read-only accesses to the @idr, such as idr_find() and - * idr_for_each_entry(). + * Return: The newly allocated ID on success. -ENOMEM for a memory + * allocation failure. -ENOSPC if there are no free IDs in the range. */ int idr_alloc(struct idr *idr, void *ptr, int start, int end, gfp_t gfp) { @@ -88,16 +142,22 @@ int idr_alloc(struct idr *idr, void *ptr, int start, int end, gfp_t gfp) EXPORT_SYMBOL_GPL(idr_alloc); /** - * idr_alloc_cyclic - allocate new idr entry in a cyclical fashion - * @idr: idr handle - * @ptr: pointer to be associated with the new id - * @start: the minimum id (inclusive) - * @end: the maximum id (exclusive) - * @gfp: memory allocation flags - * - * Allocates an ID larger than the last ID allocated if one is available. - * If not, it will attempt to allocate the smallest ID that is larger or - * equal to @start. + * idr_alloc_cyclic - Allocate an ID cyclically. + * @idr: IDR handle. + * @ptr: Pointer to be associated with the new ID. + * @start: The minimum id (inclusive). + * @end: The maximum id (exclusive). + * @gfp: Memory allocation flags. + * + * Allocates an unused ID >= @start and < @end. It will start searching + * after the last ID allocated and wrap back around to @start. + * + * The IDR uses its own spinlock to protect against simultaneous + * modification. The @ptr is visible to other simultaneous readers + * like idr_find() before this function returns. + * + * Return: The newly allocated ID on success. -ENOMEM for a memory + * allocation failure. -ENOSPC if there are no free IDs in the range. */ int idr_alloc_cyclic(struct idr *idr, void *ptr, int start, int end, gfp_t gfp) { @@ -119,88 +179,68 @@ int idr_alloc_cyclic(struct idr *idr, void *ptr, int start, int end, gfp_t gfp) idr->idr_next = id + 1U; return id; } -EXPORT_SYMBOL(idr_alloc_cyclic); +EXPORT_SYMBOL_GPL(idr_alloc_cyclic); /** - * idr_for_each - iterate through all stored pointers + * idr_for_each() - iterate through all stored pointers * @idr: idr handle * @fn: function to be called for each pointer * @data: data passed to callback function * - * The callback function will be called for each entry in @idr, passing - * the id, the pointer and the data pointer passed to this function. + * The callback function will be called for each non-NULL pointer in + * @idr, passing the id, the pointer and @data. No internal locks are + * held while @fn is called, so @fn may sleep unless otherwise prevented + * by your own locking. * * If @fn returns anything other than %0, the iteration stops and that * value is returned from this function. * - * idr_for_each() can be called concurrently with idr_alloc() and - * idr_remove() if protected by RCU. Newly added entries may not be - * seen and deleted entries may be seen, but adding and removing entries - * will not cause other entries to be skipped, nor spurious ones to be seen. + * idr_for_each() protects itself with the RCU read lock. Newly added + * entries may not be seen and deleted entries may be seen, but adding + * and removing entries will not cause other entries to be skipped, nor + * spurious ones to be seen. + * + * Return: The value returned by the last call to @fn. */ -int idr_for_each(const struct idr *idr, +int idr_for_each(struct idr *idr, int (*fn)(int id, void *p, void *data), void *data) { - struct radix_tree_iter iter; - void __rcu **slot; + unsigned long i = 0; + void *p; - radix_tree_for_each_slot(slot, &idr->idr_rt, &iter, 0) { - int ret = fn(iter.index, rcu_dereference_raw(*slot), data); + xa_for_each(&idr->idr_xa, p, i, INT_MAX) { + int ret = fn(i, p, data); if (ret) return ret; } return 0; } -EXPORT_SYMBOL(idr_for_each); +EXPORT_SYMBOL_GPL(idr_for_each); /** - * idr_get_next - Find next populated entry + * idr_get_next() - Find next populated entry * @idr: idr handle - * @nextid: Pointer to lowest possible ID to return + * @id: Pointer to lowest possible ID to return * * Returns the next populated entry in the tree with an ID greater than * or equal to the value pointed to by @nextid. On exit, @nextid is updated * to the ID of the found value. To use in a loop, the value pointed to by * nextid must be incremented by the user. - */ -void *idr_get_next(struct idr *idr, int *nextid) -{ - struct radix_tree_iter iter; - void __rcu **slot; - - slot = radix_tree_iter_find(&idr->idr_rt, &iter, *nextid); - if (!slot) - return NULL; - - *nextid = iter.index; - return rcu_dereference_raw(*slot); -} -EXPORT_SYMBOL(idr_get_next); - -/** - * idr_get_next_ul - Find next populated entry - * @idr: idr handle - * @nextid: Pointer to lowest possible ID to return * - * Returns the next populated entry in the tree with an ID greater than - * or equal to the value pointed to by @nextid. On exit, @nextid is updated - * to the ID of the found value. To use in a loop, the value pointed to by - * nextid must be incremented by the user. + * This function protects itself with the RCU read lock, so may return a + * stale entry or may skip a newly added entry unless synchronised with + * a lock. */ -void *idr_get_next_ul(struct idr *idr, unsigned long *nextid) +void *idr_get_next(struct idr *idr, int *id) { - struct radix_tree_iter iter; - void __rcu **slot; + unsigned long index = *id; + void *entry = xa_find(&idr->idr_xa, &index, INT_MAX); - slot = radix_tree_iter_find(&idr->idr_rt, &iter, *nextid); - if (!slot) - return NULL; - - *nextid = iter.index; - return rcu_dereference_raw(*slot); + *id = index; + return entry; } -EXPORT_SYMBOL(idr_get_next_ul); +EXPORT_SYMBOL_GPL(idr_get_next); /** * idr_replace - replace pointer for given id @@ -209,31 +249,35 @@ EXPORT_SYMBOL(idr_get_next_ul); * @id: Lookup key * * Replace the pointer registered with an ID and return the old value. - * This function can be called under the RCU read lock concurrently with - * idr_alloc() and idr_remove() (as long as the ID being removed is not - * the one being replaced!). + * This function protects itself with a spinlock. * * Returns: the old value on success. %-ENOENT indicates that @id was not * found. %-EINVAL indicates that @id or @ptr were not valid. */ void *idr_replace(struct idr *idr, void *ptr, unsigned long id) { - struct radix_tree_node *node; - void __rcu **slot = NULL; - void *entry; + XA_STATE(xas, &idr->idr_xa, id); + unsigned long flags; + void *curr; - if (WARN_ON_ONCE(radix_tree_is_internal_node(ptr))) + if (WARN_ON_ONCE(xa_is_internal(ptr))) return ERR_PTR(-EINVAL); - - entry = __radix_tree_lookup(&idr->idr_rt, id, &node, &slot); - if (!slot || radix_tree_tag_get(&idr->idr_rt, id, IDR_FREE)) - return ERR_PTR(-ENOENT); - - __radix_tree_replace(&idr->idr_rt, node, slot, ptr, NULL); - - return entry; + if (!ptr) + ptr = XA_ZERO_ENTRY; + + xas_lock_irqsave(&xas, flags); + curr = xas_load(&xas); + if (curr) + xas_store(&xas, ptr); + else + curr = ERR_PTR(-ENOENT); + xas_unlock_irqrestore(&xas, flags); + + if (xa_is_zero(curr)) + return NULL; + return curr; } -EXPORT_SYMBOL(idr_replace); +EXPORT_SYMBOL_GPL(idr_replace); /** * DOC: IDA description @@ -264,7 +308,7 @@ EXPORT_SYMBOL(idr_replace); * Developer's notes: * * The IDA uses the functionality provided by the IDR & radix tree to store - * bitmaps in each entry. The IDR_FREE tag means there is at least one bit + * bitmaps in each entry. The XA_FREE_TAG tag means there is at least one bit * free, unlike the IDR where it means at least one entry is free. * * I considered telling the radix tree that each slot is an order-10 node @@ -370,7 +414,7 @@ int ida_get_new_above(struct ida *ida, int start, int *id) __set_bit(bit, bitmap->bitmap); if (bitmap_full(bitmap->bitmap, IDA_BITMAP_BITS)) radix_tree_iter_tag_clear(root, &iter, - IDR_FREE); + XA_FREE_TAG); } else { new += bit; if (new < 0) @@ -426,7 +470,7 @@ void ida_remove(struct ida *ida, int id) goto err; __clear_bit(offset, btmp); - radix_tree_iter_tag_set(&ida->ida_rt, &iter, IDR_FREE); + radix_tree_iter_tag_set(&ida->ida_rt, &iter, XA_FREE_TAG); if (xa_is_value(bitmap)) { if (xa_to_value(rcu_dereference_raw(*slot)) == 0) radix_tree_iter_delete(&ida->ida_rt, &iter, slot); diff --git a/lib/radix-tree.c b/lib/radix-tree.c index cb7cb9e96a8b..3b63d1ce7fda 100644 --- a/lib/radix-tree.c +++ b/lib/radix-tree.c @@ -529,6 +529,30 @@ int radix_tree_maybe_preload_order(gfp_t gfp_mask, int order) return __radix_tree_preload(gfp_mask, nr_nodes); } +/* Once the IDR users abandon the preload API, we can use xas_nomem */ +bool idr_nomem(struct xa_state *xas, gfp_t gfp) +{ + if (xas->xa_node != XAS_ERROR(ENOMEM)) { + xas_destroy(xas); + return false; + } + xas->xa_alloc = kmem_cache_alloc(radix_tree_node_cachep, + gfp | __GFP_NOWARN); + if (!xas->xa_alloc) { + struct radix_tree_preload *rtp; + + rtp = this_cpu_ptr(&radix_tree_preloads); + if (!rtp->nr) + return false; + xas->xa_alloc = rtp->nodes; + rtp->nodes = xas->xa_alloc->parent; + rtp->nr--; + } + + xas->xa_node = XAS_RESTART; + return true; +} + static unsigned radix_tree_load_root(const struct radix_tree_root *root, struct radix_tree_node **nodep, unsigned long *maxindex) { @@ -562,7 +586,7 @@ static int radix_tree_extend(struct radix_tree_root *root, gfp_t gfp, maxshift += RADIX_TREE_MAP_SHIFT; entry = rcu_dereference_raw(root->xa_head); - if (!entry && (!is_idr(root) || root_tag_get(root, IDR_FREE))) + if (!entry && (!is_idr(root) || root_tag_get(root, XA_FREE_TAG))) goto out; do { @@ -572,10 +596,10 @@ static int radix_tree_extend(struct radix_tree_root *root, gfp_t gfp, return -ENOMEM; if (is_idr(root)) { - all_tag_set(node, IDR_FREE); - if (!root_tag_get(root, IDR_FREE)) { - rtag_clear(node, IDR_FREE, 0); - root_tag_set(root, IDR_FREE); + all_tag_set(node, XA_FREE_TAG); + if (!root_tag_get(root, XA_FREE_TAG)) { + rtag_clear(node, XA_FREE_TAG, 0); + root_tag_set(root, XA_FREE_TAG); } } else { /* Propagate the aggregated tag info to the new child */ @@ -646,8 +670,8 @@ static inline bool radix_tree_shrink(struct radix_tree_root *root, * one (root->xa_head) as far as dependent read barriers go. */ root->xa_head = (void __rcu *)child; - if (is_idr(root) && !rtag_get(node, IDR_FREE, 0)) - root_tag_clear(root, IDR_FREE); + if (is_idr(root) && !rtag_get(node, XA_FREE_TAG, 0)) + root_tag_clear(root, XA_FREE_TAG); /* * We have a dilemma here. The node's slot[0] must not be @@ -1074,7 +1098,7 @@ static bool node_tag_get(const struct radix_tree_root *root, /* * IDR users want to be able to store NULL in the tree, so if the slot isn't * free, don't adjust the count, even if it's transitioning between NULL and - * non-NULL. For the IDA, we mark slots as being IDR_FREE while they still + * non-NULL. For the IDA, we mark slots as being XA_FREE_TAG while they still * have empty bits, but it only stores NULL in slots when they're being * deleted. */ @@ -1084,7 +1108,7 @@ static int calculate_count(struct radix_tree_root *root, { if (is_idr(root)) { unsigned offset = get_slot_offset(node, slot); - bool free = node_tag_get(root, node, IDR_FREE, offset); + bool free = node_tag_get(root, node, XA_FREE_TAG, offset); if (!free) return 0; if (!old) @@ -1915,7 +1939,7 @@ static bool __radix_tree_delete(struct radix_tree_root *root, int tag; if (is_idr(root)) - node_tag_set(root, node, IDR_FREE, offset); + node_tag_set(root, node, XA_FREE_TAG, offset); else for (tag = 0; tag < RADIX_TREE_MAX_TAGS; tag++) node_tag_clear(root, node, tag, offset); @@ -1963,7 +1987,7 @@ void *radix_tree_delete_item(struct radix_tree_root *root, void *entry; entry = __radix_tree_lookup(root, index, &node, &slot); - if (!entry && (!is_idr(root) || node_tag_get(root, node, IDR_FREE, + if (!entry && (!is_idr(root) || node_tag_get(root, node, XA_FREE_TAG, get_slot_offset(node, slot)))) return NULL; @@ -2070,7 +2094,7 @@ void __rcu **idr_get_free(struct radix_tree_root *root, grow: shift = radix_tree_load_root(root, &child, &maxindex); - if (!radix_tree_tagged(root, IDR_FREE)) + if (!radix_tree_tagged(root, XA_FREE_TAG)) start = max(start, maxindex + 1); if (start > max) return ERR_PTR(-ENOSPC); @@ -2091,7 +2115,7 @@ void __rcu **idr_get_free(struct radix_tree_root *root, offset, 0, 0); if (!child) return ERR_PTR(-ENOMEM); - all_tag_set(child, IDR_FREE); + all_tag_set(child, XA_FREE_TAG); rcu_assign_pointer(*slot, node_to_entry(child)); if (node) node->count++; @@ -2100,8 +2124,8 @@ void __rcu **idr_get_free(struct radix_tree_root *root, node = entry_to_node(child); offset = radix_tree_descend(node, &child, start); - if (!rtag_get(node, IDR_FREE, offset)) { - offset = radix_tree_find_next_bit(node, IDR_FREE, + if (!rtag_get(node, XA_FREE_TAG, offset)) { + offset = radix_tree_find_next_bit(node, XA_FREE_TAG, offset + 1); start = rnext_index(start, node, offset); if (start > max) @@ -2125,32 +2149,11 @@ void __rcu **idr_get_free(struct radix_tree_root *root, iter->next_index = 1; iter->node = node; __set_iter_shift(iter, shift); - set_iter_tags(iter, node, offset, IDR_FREE); + set_iter_tags(iter, node, offset, XA_FREE_TAG); return slot; } -/** - * idr_destroy - release all internal memory from an IDR - * @idr: idr handle - * - * After this function is called, the IDR is empty, and may be reused or - * the data structure containing it may be freed. - * - * A typical clean-up sequence for objects stored in an idr tree will use - * idr_for_each() to free all objects, if necessary, then idr_destroy() to - * free the memory used to keep track of those objects. - */ -void idr_destroy(struct idr *idr) -{ - struct radix_tree_node *node = rcu_dereference_raw(idr->idr_rt.xa_head); - if (radix_tree_is_internal_node(node)) - radix_tree_free_nodes(node); - idr->idr_rt.xa_head = NULL; - root_tag_set(&idr->idr_rt, IDR_FREE); -} -EXPORT_SYMBOL(idr_destroy); - static void radix_tree_node_ctor(void *arg) { diff --git a/lib/xarray.c b/lib/xarray.c index cc88df7bd6df..baa425ba3ee1 100644 --- a/lib/xarray.c +++ b/lib/xarray.c @@ -1062,6 +1062,8 @@ void *xa_load(struct xarray *xa, unsigned long index) rcu_read_lock(); do { entry = xas_load(&xas); + if (xa_is_zero(entry)) + entry = NULL; } while (xas_retry(&xas, entry)); rcu_read_unlock(); @@ -1119,6 +1121,8 @@ void *xa_store(struct xarray *xa, unsigned long index, void *entry, gfp_t gfp) xa_lock_irqsave(xa, flags); curr = xas_store(&xas, entry); xa_unlock_irqrestore(xa, flags); + if (xa_is_zero(curr)) + curr = NULL; } while (xas_nomem(&xas, gfp)); if (xas_error(&xas)) @@ -1491,6 +1495,8 @@ void xa_dump_entry(void *entry, unsigned long index) printk("%lu: retry (%ld)\n", index, xa_to_internal(entry)); else if (xa_is_sibling(entry)) printk("%lu: sibling (%ld)\n", index, xa_to_sibling(entry)); + else if (xa_is_zero(entry)) + printk("%lu: zero (%ld)\n", index, xa_to_internal(entry)); else printk("%lu: UNKNOWN ENTRY (%p)\n", index, entry); } diff --git a/tools/testing/radix-tree/idr-test.c b/tools/testing/radix-tree/idr-test.c index 7499319e85f8..7b710145d2ae 100644 --- a/tools/testing/radix-tree/idr-test.c +++ b/tools/testing/radix-tree/idr-test.c @@ -177,6 +177,22 @@ void idr_get_next_test(void) idr_destroy(&idr); } +/* + * Check that growing the IDR works properly. + */ +void idr_alloc_far(struct idr *idr, unsigned long end) +{ + int i; + + for (i = 1; i < end; i++) + assert(idr_alloc(idr, idr, i, i + 1, GFP_KERNEL) == i); + + for (i = 1; i <= end; i++) { + assert(idr_alloc(idr, idr, 1, 0, GFP_KERNEL) == end); + idr_remove(idr, end); + } +} + void idr_checks(void) { unsigned long i; @@ -227,6 +243,11 @@ void idr_checks(void) idr_null_test(); idr_nowait_test(); idr_get_next_test(); + + for (i = 2; i < 18; i++) { + idr_alloc_far(&idr, 1UL << i); + idr_destroy(&idr); + } } /* @@ -505,7 +526,9 @@ void ida_thread_tests(void) int __weak main(void) { radix_tree_init(); + printv(0, "starting IDR checks\n"); idr_checks(); + printv(0, "starting IDA checks\n"); ida_checks(); ida_thread_tests(); radix_tree_cpu_dead(1);