Commit a8ec08bf authored by Vlastimil Babka's avatar Vlastimil Babka
Browse files

Merge branch 'slab/for-6.19/mempool_alloc_bulk' into slab/for-next

Merges series "mempool_alloc_bulk and various mempool improvements v3"
from Christoph Hellwig.

From the cover letter [1]:

This series adds a bulk version of mempool_alloc that makes allocating
multiple objects deadlock safe.

The initial users is the blk-crypto-fallback code:

  https://lore.kernel.org/linux-block/20251031093517.1603379-1-hch@lst.de/

with which v1 was posted, but I also have a few other users in mind.

Link: https://lore.kernel.org/all/20251113084022.1255121-1-hch@lst.de/ [1]
parents ed80cc75 48233291
Loading
Loading
Loading
Loading
+4 −4
Original line number Diff line number Diff line
@@ -8,6 +8,10 @@
struct dentry;
struct kmem_cache;

enum fault_flags {
	FAULT_NOWARN =	1 << 0,
};

#ifdef CONFIG_FAULT_INJECTION

#include <linux/atomic.h>
@@ -36,10 +40,6 @@ struct fault_attr {
	struct dentry *dname;
};

enum fault_flags {
	FAULT_NOWARN =	1 << 0,
};

#define FAULT_ATTR_INITIALIZER {					\
		.interval = 1,						\
		.times = ATOMIC_INIT(1),				\
+25 −33
Original line number Diff line number Diff line
@@ -27,32 +27,31 @@ typedef struct mempool {
	wait_queue_head_t wait;
} mempool_t;

static inline bool mempool_initialized(mempool_t *pool)
static inline bool mempool_initialized(struct mempool *pool)
{
	return pool->elements != NULL;
}

static inline bool mempool_is_saturated(mempool_t *pool)
static inline bool mempool_is_saturated(struct mempool *pool)
{
	return READ_ONCE(pool->curr_nr) >= pool->min_nr;
}

void mempool_exit(mempool_t *pool);
int mempool_init_node(mempool_t *pool, int min_nr, mempool_alloc_t *alloc_fn,
		      mempool_free_t *free_fn, void *pool_data,
		      gfp_t gfp_mask, int node_id);

int mempool_init_noprof(mempool_t *pool, int min_nr, mempool_alloc_t *alloc_fn,
		 mempool_free_t *free_fn, void *pool_data);
void mempool_exit(struct mempool *pool);
int mempool_init_node(struct mempool *pool, int min_nr,
		mempool_alloc_t *alloc_fn, mempool_free_t *free_fn,
		void *pool_data, gfp_t gfp_mask, int node_id);
int mempool_init_noprof(struct mempool *pool, int min_nr,
		mempool_alloc_t *alloc_fn, mempool_free_t *free_fn,
		void *pool_data);
#define mempool_init(...)						\
	alloc_hooks(mempool_init_noprof(__VA_ARGS__))

extern mempool_t *mempool_create(int min_nr, mempool_alloc_t *alloc_fn,
struct mempool *mempool_create(int min_nr, mempool_alloc_t *alloc_fn,
		mempool_free_t *free_fn, void *pool_data);

extern mempool_t *mempool_create_node_noprof(int min_nr, mempool_alloc_t *alloc_fn,
			mempool_free_t *free_fn, void *pool_data,
			gfp_t gfp_mask, int nid);
struct mempool *mempool_create_node_noprof(int min_nr,
		mempool_alloc_t *alloc_fn, mempool_free_t *free_fn,
		void *pool_data, gfp_t gfp_mask, int nid);
#define mempool_create_node(...)					\
	alloc_hooks(mempool_create_node_noprof(__VA_ARGS__))

@@ -60,15 +59,21 @@ extern mempool_t *mempool_create_node_noprof(int min_nr, mempool_alloc_t *alloc_
	mempool_create_node(_min_nr, _alloc_fn, _free_fn, _pool_data,	\
			    GFP_KERNEL, NUMA_NO_NODE)

extern int mempool_resize(mempool_t *pool, int new_min_nr);
extern void mempool_destroy(mempool_t *pool);
int mempool_resize(struct mempool *pool, int new_min_nr);
void mempool_destroy(struct mempool *pool);

extern void *mempool_alloc_noprof(mempool_t *pool, gfp_t gfp_mask) __malloc;
void *mempool_alloc_noprof(struct mempool *pool, gfp_t gfp_mask) __malloc;
#define mempool_alloc(...)						\
	alloc_hooks(mempool_alloc_noprof(__VA_ARGS__))
int mempool_alloc_bulk_noprof(struct mempool *pool, void **elem,
		unsigned int count, unsigned int allocated);
#define mempool_alloc_bulk(...)						\
	alloc_hooks(mempool_alloc_bulk_noprof(__VA_ARGS__))

extern void *mempool_alloc_preallocated(mempool_t *pool) __malloc;
extern void mempool_free(void *element, mempool_t *pool);
void *mempool_alloc_preallocated(struct mempool *pool) __malloc;
void mempool_free(void *element, struct mempool *pool);
unsigned int mempool_free_bulk(struct mempool *pool, void **elem,
		unsigned int count);

/*
 * A mempool_alloc_t and mempool_free_t that get the memory from
@@ -97,19 +102,6 @@ void mempool_kfree(void *element, void *pool_data);
	mempool_create((_min_nr), mempool_kmalloc, mempool_kfree,	\
		       (void *)(unsigned long)(_size))

void *mempool_kvmalloc(gfp_t gfp_mask, void *pool_data);
void mempool_kvfree(void *element, void *pool_data);

static inline int mempool_init_kvmalloc_pool(mempool_t *pool, int min_nr, size_t size)
{
	return mempool_init(pool, min_nr, mempool_kvmalloc, mempool_kvfree, (void *) size);
}

static inline mempool_t *mempool_create_kvmalloc_pool(int min_nr, size_t size)
{
	return mempool_create(min_nr, mempool_kvmalloc, mempool_kvfree, (void *) size);
}

/*
 * A mempool_alloc_t and mempool_free_t for a simple page allocator that
 * allocates pages of the order specified by pool_data
+256 −153
Original line number Diff line number Diff line
// SPDX-License-Identifier: GPL-2.0
/*
 *  linux/mm/mempool.c
 *
 *  memory buffer pool support. Such pools are mostly used
 *  for guaranteed, deadlock-free memory allocations during
 *  extreme VM load.
@@ -9,7 +7,7 @@
 *  started by Ingo Molnar, Copyright (C) 2001
 *  debugging by David Rientjes, Copyright (C) 2015
 */

#include <linux/fault-inject.h>
#include <linux/mm.h>
#include <linux/slab.h>
#include <linux/highmem.h>
@@ -20,8 +18,27 @@
#include <linux/writeback.h>
#include "slab.h"

static DECLARE_FAULT_ATTR(fail_mempool_alloc);
static DECLARE_FAULT_ATTR(fail_mempool_alloc_bulk);

static int __init mempool_faul_inject_init(void)
{
	int error;

	error = PTR_ERR_OR_ZERO(fault_create_debugfs_attr("fail_mempool_alloc",
			NULL, &fail_mempool_alloc));
	if (error)
		return error;

	/* booting will fail on error return here, don't bother to cleanup */
	return PTR_ERR_OR_ZERO(
		fault_create_debugfs_attr("fail_mempool_alloc_bulk", NULL,
		&fail_mempool_alloc_bulk));
}
late_initcall(mempool_faul_inject_init);

#ifdef CONFIG_SLUB_DEBUG_ON
static void poison_error(mempool_t *pool, void *element, size_t size,
static void poison_error(struct mempool *pool, void *element, size_t size,
			 size_t byte)
{
	const int nr = pool->curr_nr;
@@ -38,7 +55,7 @@ static void poison_error(mempool_t *pool, void *element, size_t size,
	dump_stack();
}

static void __check_element(mempool_t *pool, void *element, size_t size)
static void __check_element(struct mempool *pool, void *element, size_t size)
{
	u8 *obj = element;
	size_t i;
@@ -54,7 +71,7 @@ static void __check_element(mempool_t *pool, void *element, size_t size)
	memset(obj, POISON_INUSE, size);
}

static void check_element(mempool_t *pool, void *element)
static void check_element(struct mempool *pool, void *element)
{
	/* Skip checking: KASAN might save its metadata in the element. */
	if (kasan_enabled())
@@ -83,7 +100,7 @@ static void __poison_element(void *element, size_t size)
	obj[size - 1] = POISON_END;
}

static void poison_element(mempool_t *pool, void *element)
static void poison_element(struct mempool *pool, void *element)
{
	/* Skip poisoning: KASAN might save its metadata in the element. */
	if (kasan_enabled())
@@ -104,15 +121,16 @@ static void poison_element(mempool_t *pool, void *element)
	}
}
#else /* CONFIG_SLUB_DEBUG_ON */
static inline void check_element(mempool_t *pool, void *element)
static inline void check_element(struct mempool *pool, void *element)
{
}
static inline void poison_element(mempool_t *pool, void *element)
static inline void poison_element(struct mempool *pool, void *element)
{
}
#endif /* CONFIG_SLUB_DEBUG_ON */

static __always_inline bool kasan_poison_element(mempool_t *pool, void *element)
static __always_inline bool kasan_poison_element(struct mempool *pool,
		void *element)
{
	if (pool->alloc == mempool_alloc_slab || pool->alloc == mempool_kmalloc)
		return kasan_mempool_poison_object(element);
@@ -122,7 +140,7 @@ static __always_inline bool kasan_poison_element(mempool_t *pool, void *element)
	return true;
}

static void kasan_unpoison_element(mempool_t *pool, void *element)
static void kasan_unpoison_element(struct mempool *pool, void *element)
{
	if (pool->alloc == mempool_kmalloc)
		kasan_mempool_unpoison_object(element, (size_t)pool->pool_data);
@@ -134,7 +152,7 @@ static void kasan_unpoison_element(mempool_t *pool, void *element)
					     (unsigned long)pool->pool_data);
}

static __always_inline void add_element(mempool_t *pool, void *element)
static __always_inline void add_element(struct mempool *pool, void *element)
{
	BUG_ON(pool->min_nr != 0 && pool->curr_nr >= pool->min_nr);
	poison_element(pool, element);
@@ -142,7 +160,7 @@ static __always_inline void add_element(mempool_t *pool, void *element)
		pool->elements[pool->curr_nr++] = element;
}

static void *remove_element(mempool_t *pool)
static void *remove_element(struct mempool *pool)
{
	void *element = pool->elements[--pool->curr_nr];

@@ -163,7 +181,7 @@ static void *remove_element(mempool_t *pool)
 * May be called on a zeroed but uninitialized mempool (i.e. allocated with
 * kzalloc()).
 */
void mempool_exit(mempool_t *pool)
void mempool_exit(struct mempool *pool)
{
	while (pool->curr_nr) {
		void *element = remove_element(pool);
@@ -182,7 +200,7 @@ EXPORT_SYMBOL(mempool_exit);
 * Free all reserved elements in @pool and @pool itself.  This function
 * only sleeps if the free_fn() function sleeps.
 */
void mempool_destroy(mempool_t *pool)
void mempool_destroy(struct mempool *pool)
{
	if (unlikely(!pool))
		return;
@@ -192,9 +210,9 @@ void mempool_destroy(mempool_t *pool)
}
EXPORT_SYMBOL(mempool_destroy);

int mempool_init_node(mempool_t *pool, int min_nr, mempool_alloc_t *alloc_fn,
		      mempool_free_t *free_fn, void *pool_data,
		      gfp_t gfp_mask, int node_id)
int mempool_init_node(struct mempool *pool, int min_nr,
		mempool_alloc_t *alloc_fn, mempool_free_t *free_fn,
		void *pool_data, gfp_t gfp_mask, int node_id)
{
	spin_lock_init(&pool->lock);
	pool->min_nr	= min_nr;
@@ -244,8 +262,9 @@ EXPORT_SYMBOL(mempool_init_node);
 *
 * Return: %0 on success, negative error code otherwise.
 */
int mempool_init_noprof(mempool_t *pool, int min_nr, mempool_alloc_t *alloc_fn,
			mempool_free_t *free_fn, void *pool_data)
int mempool_init_noprof(struct mempool *pool, int min_nr,
		mempool_alloc_t *alloc_fn, mempool_free_t *free_fn,
		void *pool_data)
{
	return mempool_init_node(pool, min_nr, alloc_fn, free_fn,
				 pool_data, GFP_KERNEL, NUMA_NO_NODE);
@@ -271,11 +290,11 @@ EXPORT_SYMBOL(mempool_init_noprof);
 *
 * Return: pointer to the created memory pool object or %NULL on error.
 */
mempool_t *mempool_create_node_noprof(int min_nr, mempool_alloc_t *alloc_fn,
				      mempool_free_t *free_fn, void *pool_data,
				      gfp_t gfp_mask, int node_id)
struct mempool *mempool_create_node_noprof(int min_nr,
		mempool_alloc_t *alloc_fn, mempool_free_t *free_fn,
		void *pool_data, gfp_t gfp_mask, int node_id)
{
	mempool_t *pool;
	struct mempool *pool;

	pool = kmalloc_node_noprof(sizeof(*pool), gfp_mask | __GFP_ZERO, node_id);
	if (!pool)
@@ -309,7 +328,7 @@ EXPORT_SYMBOL(mempool_create_node_noprof);
 *
 * Return: %0 on success, negative error code otherwise.
 */
int mempool_resize(mempool_t *pool, int new_min_nr)
int mempool_resize(struct mempool *pool, int new_min_nr)
{
	void *element;
	void **new_elements;
@@ -371,140 +390,227 @@ int mempool_resize(mempool_t *pool, int new_min_nr)
}
EXPORT_SYMBOL(mempool_resize);

/**
 * mempool_alloc - allocate an element from a specific memory pool
 * @pool:      pointer to the memory pool which was allocated via
 *             mempool_create().
 * @gfp_mask:  the usual allocation bitmask.
 *
 * this function only sleeps if the alloc_fn() function sleeps or
 * returns NULL. Note that due to preallocation, this function
 * *never* fails when called from process contexts. (it might
 * fail if called from an IRQ context.)
 * Note: using __GFP_ZERO is not supported.
 *
 * Return: pointer to the allocated element or %NULL on error.
 */
void *mempool_alloc_noprof(mempool_t *pool, gfp_t gfp_mask)
static unsigned int mempool_alloc_from_pool(struct mempool *pool, void **elems,
		unsigned int count, unsigned int allocated,
		gfp_t gfp_mask)
{
	void *element;
	unsigned long flags;
	wait_queue_entry_t wait;
	gfp_t gfp_temp;
	unsigned int i;

	VM_WARN_ON_ONCE(gfp_mask & __GFP_ZERO);
	might_alloc(gfp_mask);
	spin_lock_irqsave(&pool->lock, flags);
	if (unlikely(pool->curr_nr < count - allocated))
		goto fail;
	for (i = 0; i < count; i++) {
		if (!elems[i]) {
			elems[i] = remove_element(pool);
			allocated++;
		}
	}
	spin_unlock_irqrestore(&pool->lock, flags);

	gfp_mask |= __GFP_NOMEMALLOC;	/* don't allocate emergency reserves */
	gfp_mask |= __GFP_NORETRY;	/* don't loop in __alloc_pages */
	gfp_mask |= __GFP_NOWARN;	/* failures are OK */
	/* Paired with rmb in mempool_free(), read comment there. */
	smp_wmb();

	gfp_temp = gfp_mask & ~(__GFP_DIRECT_RECLAIM|__GFP_IO);
	/*
	 * Update the allocation stack trace as this is more useful for
	 * debugging.
	 */
	for (i = 0; i < count; i++)
		kmemleak_update_trace(elems[i]);
	return allocated;

repeat_alloc:
fail:
	if (gfp_mask & __GFP_DIRECT_RECLAIM) {
		DEFINE_WAIT(wait);

	element = pool->alloc(gfp_temp, pool->pool_data);
	if (likely(element != NULL))
		return element;
		prepare_to_wait(&pool->wait, &wait, TASK_UNINTERRUPTIBLE);
		spin_unlock_irqrestore(&pool->lock, flags);

	spin_lock_irqsave(&pool->lock, flags);
	if (likely(pool->curr_nr)) {
		element = remove_element(pool);
		/*
		 * Wait for someone else to return an element to @pool, but wake
		 * up occasionally as memory pressure might have reduced even
		 * and the normal allocation in alloc_fn could succeed even if
		 * no element was returned.
		 */
		io_schedule_timeout(5 * HZ);
		finish_wait(&pool->wait, &wait);
	} else {
		/* We must not sleep if __GFP_DIRECT_RECLAIM is not set. */
		spin_unlock_irqrestore(&pool->lock, flags);
		/* paired with rmb in mempool_free(), read comment there */
		smp_wmb();
	}

	return allocated;
}

/*
		 * Update the allocation stack trace as this is more useful
		 * for debugging.
 * Adjust the gfp flags for mempool allocations, as we never want to dip into
 * the global emergency reserves or retry in the page allocator.
 *
 * The first pass also doesn't want to go reclaim, but the next passes do, so
 * return a separate subset for that first iteration.
 */
		kmemleak_update_trace(element);
		return element;
static inline gfp_t mempool_adjust_gfp(gfp_t *gfp_mask)
{
	*gfp_mask |= __GFP_NOMEMALLOC | __GFP_NORETRY | __GFP_NOWARN;
	return *gfp_mask & ~(__GFP_DIRECT_RECLAIM | __GFP_IO);
}

/**
 * mempool_alloc_bulk - allocate multiple elements from a memory pool
 * @pool:	pointer to the memory pool
 * @elems:	partially or fully populated elements array
 * @count:	number of entries in @elem that need to be allocated
 * @allocated:	number of entries in @elem already allocated
 *
 * Allocate elements for each slot in @elem that is non-%NULL. This is done by
 * first calling into the alloc_fn supplied at pool initialization time, and
 * dipping into the reserved pool when alloc_fn fails to allocate an element.
 *
 * On return all @count elements in @elems will be populated.
 *
 * Return: Always 0.  If it wasn't for %$#^$ alloc tags, it would return void.
 */
int mempool_alloc_bulk_noprof(struct mempool *pool, void **elems,
		unsigned int count, unsigned int allocated)
{
	gfp_t gfp_mask = GFP_KERNEL;
	gfp_t gfp_temp = mempool_adjust_gfp(&gfp_mask);
	unsigned int i = 0;

	VM_WARN_ON_ONCE(count > pool->min_nr);
	might_alloc(gfp_mask);

	/*
	 * We use gfp mask w/o direct reclaim or IO for the first round.  If
	 * alloc failed with that and @pool was empty, retry immediately.
	 * If an error is injected, fail all elements in a bulk allocation so
	 * that we stress the multiple elements missing path.
	 */
	if (gfp_temp != gfp_mask) {
		spin_unlock_irqrestore(&pool->lock, flags);
	if (should_fail_ex(&fail_mempool_alloc_bulk, 1, FAULT_NOWARN)) {
		pr_info("forcing mempool usage for %pS\n",
				(void *)_RET_IP_);
		goto use_pool;
	}

repeat_alloc:
	/*
	 * Try to allocate the elements using the allocation callback first as
	 * that might succeed even when the caller's bulk allocation did not.
	 */
	for (i = 0; i < count; i++) {
		if (elems[i])
			continue;
		elems[i] = pool->alloc(gfp_temp, pool->pool_data);
		if (unlikely(!elems[i]))
			goto use_pool;
		allocated++;
	}

	return 0;

use_pool:
	allocated = mempool_alloc_from_pool(pool, elems, count, allocated,
			gfp_temp);
	gfp_temp = gfp_mask;
	goto repeat_alloc;
}
EXPORT_SYMBOL_GPL(mempool_alloc_bulk_noprof);

	/* We must not sleep if !__GFP_DIRECT_RECLAIM */
	if (!(gfp_mask & __GFP_DIRECT_RECLAIM)) {
		spin_unlock_irqrestore(&pool->lock, flags);
		return NULL;
	}
/**
 * mempool_alloc - allocate an element from a memory pool
 * @pool:	pointer to the memory pool
 * @gfp_mask:	GFP_* flags.  %__GFP_ZERO is not supported.
 *
 * Allocate an element from @pool.  This is done by first calling into the
 * alloc_fn supplied at pool initialization time, and dipping into the reserved
 * pool when alloc_fn fails to allocate an element.
 *
 * This function only sleeps if the alloc_fn callback sleeps, or when waiting
 * for elements to become available in the pool.
 *
 * Return: pointer to the allocated element or %NULL when failing to allocate
 * an element.  Allocation failure can only happen when @gfp_mask does not
 * include %__GFP_DIRECT_RECLAIM.
 */
void *mempool_alloc_noprof(struct mempool *pool, gfp_t gfp_mask)
{
	gfp_t gfp_temp = mempool_adjust_gfp(&gfp_mask);
	void *element;

	/* Let's wait for someone else to return an element to @pool */
	init_wait(&wait);
	prepare_to_wait(&pool->wait, &wait, TASK_UNINTERRUPTIBLE);
	VM_WARN_ON_ONCE(gfp_mask & __GFP_ZERO);
	might_alloc(gfp_mask);

	spin_unlock_irqrestore(&pool->lock, flags);
repeat_alloc:
	if (should_fail_ex(&fail_mempool_alloc, 1, FAULT_NOWARN)) {
		pr_info("forcing mempool usage for %pS\n",
				(void *)_RET_IP_);
		element = NULL;
	} else {
		element = pool->alloc(gfp_temp, pool->pool_data);
	}

	if (unlikely(!element)) {
		/*
	 * FIXME: this should be io_schedule().  The timeout is there as a
	 * workaround for some DM problems in 2.6.18.
		 * Try to allocate an element from the pool.
		 *
		 * The first pass won't have __GFP_DIRECT_RECLAIM and won't
		 * sleep in mempool_alloc_from_pool.  Retry the allocation
		 * with all flags set in that case.
		 */
	io_schedule_timeout(5*HZ);

	finish_wait(&pool->wait, &wait);
		if (!mempool_alloc_from_pool(pool, &element, 1, 0, gfp_temp)) {
			if (gfp_temp != gfp_mask) {
				gfp_temp = gfp_mask;
				goto repeat_alloc;
			}
			if (gfp_mask & __GFP_DIRECT_RECLAIM) {
				goto repeat_alloc;
			}
		}
	}

	return element;
}
EXPORT_SYMBOL(mempool_alloc_noprof);

/**
 * mempool_alloc_preallocated - allocate an element from preallocated elements
 *                              belonging to a specific memory pool
 * @pool:      pointer to the memory pool which was allocated via
 *             mempool_create().
 *                              belonging to a memory pool
 * @pool:	pointer to the memory pool
 *
 * This function is similar to mempool_alloc, but it only attempts allocating
 * an element from the preallocated elements. It does not sleep and immediately
 * returns if no preallocated elements are available.
 * This function is similar to mempool_alloc(), but it only attempts allocating
 * an element from the preallocated elements. It only takes a single spinlock_t
 * and immediately returns if no preallocated elements are available.
 *
 * Return: pointer to the allocated element or %NULL if no elements are
 * available.
 */
void *mempool_alloc_preallocated(mempool_t *pool)
void *mempool_alloc_preallocated(struct mempool *pool)
{
	void *element;
	unsigned long flags;
	void *element = NULL;

	spin_lock_irqsave(&pool->lock, flags);
	if (likely(pool->curr_nr)) {
		element = remove_element(pool);
		spin_unlock_irqrestore(&pool->lock, flags);
		/* paired with rmb in mempool_free(), read comment there */
		smp_wmb();
		/*
		 * Update the allocation stack trace as this is more useful
		 * for debugging.
		 */
		kmemleak_update_trace(element);
	mempool_alloc_from_pool(pool, &element, 1, 0, GFP_NOWAIT);
	return element;
}
	spin_unlock_irqrestore(&pool->lock, flags);

	return NULL;
}
EXPORT_SYMBOL(mempool_alloc_preallocated);

/**
 * mempool_free - return an element to the pool.
 * @element:   pool element pointer.
 * @pool:      pointer to the memory pool which was allocated via
 *             mempool_create().
 * mempool_free_bulk - return elements to a mempool
 * @pool:	pointer to the memory pool
 * @elems:	elements to return
 * @count:	number of elements to return
 *
 * this function only sleeps if the free_fn() function sleeps.
 * Returns a number of elements from the start of @elem to @pool if @pool needs
 * replenishing and sets their slots in @elem to NULL.  Other elements are left
 * in @elem.
 *
 * Return: number of elements transferred to @pool.  Elements are always
 * transferred from the beginning of @elem, so the return value can be used as
 * an offset into @elem for the freeing the remaining elements in the caller.
 */
void mempool_free(void *element, mempool_t *pool)
unsigned int mempool_free_bulk(struct mempool *pool, void **elems,
		unsigned int count)
{
	unsigned long flags;

	if (unlikely(element == NULL))
		return;
	unsigned int freed = 0;
	bool added = false;

	/*
	 * Paired with the wmb in mempool_alloc().  The preceding read is
@@ -538,21 +644,6 @@ void mempool_free(void *element, mempool_t *pool)
	 * Waiters happen iff curr_nr is 0 and the above guarantee also
	 * ensures that there will be frees which return elements to the
	 * pool waking up the waiters.
	 */
	if (unlikely(READ_ONCE(pool->curr_nr) < pool->min_nr)) {
		spin_lock_irqsave(&pool->lock, flags);
		if (likely(pool->curr_nr < pool->min_nr)) {
			add_element(pool, element);
			spin_unlock_irqrestore(&pool->lock, flags);
			if (wq_has_sleeper(&pool->wait))
				wake_up(&pool->wait);
			return;
		}
		spin_unlock_irqrestore(&pool->lock, flags);
	}

	/*
	 * Handle the min_nr = 0 edge case:
	 *
	 * For zero-minimum pools, curr_nr < min_nr (0 < 0) never succeeds,
	 * so waiters sleeping on pool->wait would never be woken by the
@@ -560,19 +651,44 @@ void mempool_free(void *element, mempool_t *pool)
	 * allocation of element when both min_nr and curr_nr are 0, and
	 * any active waiters are properly awakened.
	 */
	if (unlikely(pool->min_nr == 0 &&
	if (unlikely(READ_ONCE(pool->curr_nr) < pool->min_nr)) {
		spin_lock_irqsave(&pool->lock, flags);
		while (pool->curr_nr < pool->min_nr && freed < count) {
			add_element(pool, elems[freed++]);
			added = true;
		}
		spin_unlock_irqrestore(&pool->lock, flags);
	} else if (unlikely(pool->min_nr == 0 &&
		     READ_ONCE(pool->curr_nr) == 0)) {
		/* Handle the min_nr = 0 edge case: */
		spin_lock_irqsave(&pool->lock, flags);
		if (likely(pool->curr_nr == 0)) {
			add_element(pool, element);
			spin_unlock_irqrestore(&pool->lock, flags);
			if (wq_has_sleeper(&pool->wait))
				wake_up(&pool->wait);
			return;
			add_element(pool, elems[freed++]);
			added = true;
		}
		spin_unlock_irqrestore(&pool->lock, flags);
	}

	if (unlikely(added) && wq_has_sleeper(&pool->wait))
		wake_up(&pool->wait);

	return freed;
}
EXPORT_SYMBOL_GPL(mempool_free_bulk);

/**
 * mempool_free - return an element to the pool.
 * @element:	element to return
 * @pool:	pointer to the memory pool
 *
 * Returns @element to @pool if it needs replenishing, else frees it using
 * the free_fn callback in @pool.
 *
 * This function only sleeps if the free_fn callback sleeps.
 */
void mempool_free(void *element, struct mempool *pool)
{
	if (likely(element) && !mempool_free_bulk(pool, &element, 1))
		pool->free(element, pool->pool_data);
}
EXPORT_SYMBOL(mempool_free);
@@ -612,19 +728,6 @@ void mempool_kfree(void *element, void *pool_data)
}
EXPORT_SYMBOL(mempool_kfree);

void *mempool_kvmalloc(gfp_t gfp_mask, void *pool_data)
{
	size_t size = (size_t)pool_data;
	return kvmalloc(size, gfp_mask);
}
EXPORT_SYMBOL(mempool_kvmalloc);

void mempool_kvfree(void *element, void *pool_data)
{
	kvfree(element);
}
EXPORT_SYMBOL(mempool_kvfree);

/*
 * A simple mempool-backed page allocator that allocates pages
 * of the order specified by pool_data.
+10 −5
Original line number Diff line number Diff line
@@ -4982,13 +4982,18 @@ static inline bool prepare_alloc_pages(gfp_t gfp_mask, unsigned int order,
 * @nr_pages: The number of pages desired in the array
 * @page_array: Array to store the pages
 *
 * This is a batched version of the page allocator that attempts to
 * allocate nr_pages quickly. Pages are added to the page_array.
 * This is a batched version of the page allocator that attempts to allocate
 * @nr_pages quickly.  Pages are added to @page_array.
 *
 * Note that only NULL elements are populated with pages and nr_pages
 * is the maximum number of pages that will be stored in the array.
 * Note that only the elements in @page_array that were cleared to %NULL on
 * entry are populated with newly allocated pages. @nr_pages is the maximum
 * number of pages that will be stored in the array.
 *
 * Returns the number of pages in the array.
 * Returns the number of pages in @page_array, including ones already
 * allocated on entry.  This can be less than the number requested in @nr_pages,
 * but all empty slots are filled from the beginning.  I.e., if all slots in
 * @page_array were set to %NULL on entry, the slots from 0 to the return value
 * - 1 will be filled.
 */
unsigned long alloc_pages_bulk_noprof(gfp_t gfp, int preferred_nid,
			nodemask_t *nodemask, int nr_pages,