Commit 23304537 authored by Robin Murphy's avatar Robin Murphy Committed by Joerg Roedel

iommu/iova: Manage the depot list size

Automatically scaling the depot up to suit the peak capacity of a
workload is all well and good, but it would be nice to have a way to
scale it back down again if the workload changes. To that end, add
backround reclaim that will gradually free surplus magazines if the
depot size remains above a reasonable threshold for long enough.
Reviewed-by: default avatarJerry Snitselaar <jsnitsel@redhat.com>
Signed-off-by: default avatarRobin Murphy <robin.murphy@arm.com>
Link: https://lore.kernel.org/r/03170665c56d89c6ce6081246b47f68d4e483308.1694535580.git.robin.murphy@arm.comSigned-off-by: default avatarJoerg Roedel <jroedel@suse.de>
parent 911aa124
...@@ -11,6 +11,7 @@ ...@@ -11,6 +11,7 @@
#include <linux/smp.h> #include <linux/smp.h>
#include <linux/bitops.h> #include <linux/bitops.h>
#include <linux/cpu.h> #include <linux/cpu.h>
#include <linux/workqueue.h>
/* The anchor node sits above the top of the usable address space */ /* The anchor node sits above the top of the usable address space */
#define IOVA_ANCHOR ~0UL #define IOVA_ANCHOR ~0UL
...@@ -627,6 +628,8 @@ EXPORT_SYMBOL_GPL(reserve_iova); ...@@ -627,6 +628,8 @@ EXPORT_SYMBOL_GPL(reserve_iova);
*/ */
#define IOVA_MAG_SIZE 127 #define IOVA_MAG_SIZE 127
#define IOVA_DEPOT_DELAY msecs_to_jiffies(100)
struct iova_magazine { struct iova_magazine {
union { union {
unsigned long size; unsigned long size;
...@@ -644,8 +647,11 @@ struct iova_cpu_rcache { ...@@ -644,8 +647,11 @@ struct iova_cpu_rcache {
struct iova_rcache { struct iova_rcache {
spinlock_t lock; spinlock_t lock;
unsigned int depot_size;
struct iova_magazine *depot; struct iova_magazine *depot;
struct iova_cpu_rcache __percpu *cpu_rcaches; struct iova_cpu_rcache __percpu *cpu_rcaches;
struct iova_domain *iovad;
struct delayed_work work;
}; };
static struct iova_magazine *iova_magazine_alloc(gfp_t flags) static struct iova_magazine *iova_magazine_alloc(gfp_t flags)
...@@ -726,6 +732,7 @@ static struct iova_magazine *iova_depot_pop(struct iova_rcache *rcache) ...@@ -726,6 +732,7 @@ static struct iova_magazine *iova_depot_pop(struct iova_rcache *rcache)
rcache->depot = mag->next; rcache->depot = mag->next;
mag->size = IOVA_MAG_SIZE; mag->size = IOVA_MAG_SIZE;
rcache->depot_size--;
return mag; return mag;
} }
...@@ -733,6 +740,25 @@ static void iova_depot_push(struct iova_rcache *rcache, struct iova_magazine *ma ...@@ -733,6 +740,25 @@ static void iova_depot_push(struct iova_rcache *rcache, struct iova_magazine *ma
{ {
mag->next = rcache->depot; mag->next = rcache->depot;
rcache->depot = mag; rcache->depot = mag;
rcache->depot_size++;
}
static void iova_depot_work_func(struct work_struct *work)
{
struct iova_rcache *rcache = container_of(work, typeof(*rcache), work.work);
struct iova_magazine *mag = NULL;
unsigned long flags;
spin_lock_irqsave(&rcache->lock, flags);
if (rcache->depot_size > num_online_cpus())
mag = iova_depot_pop(rcache);
spin_unlock_irqrestore(&rcache->lock, flags);
if (mag) {
iova_magazine_free_pfns(mag, rcache->iovad);
iova_magazine_free(mag);
schedule_delayed_work(&rcache->work, IOVA_DEPOT_DELAY);
}
} }
int iova_domain_init_rcaches(struct iova_domain *iovad) int iova_domain_init_rcaches(struct iova_domain *iovad)
...@@ -752,6 +778,8 @@ int iova_domain_init_rcaches(struct iova_domain *iovad) ...@@ -752,6 +778,8 @@ int iova_domain_init_rcaches(struct iova_domain *iovad)
rcache = &iovad->rcaches[i]; rcache = &iovad->rcaches[i];
spin_lock_init(&rcache->lock); spin_lock_init(&rcache->lock);
rcache->iovad = iovad;
INIT_DELAYED_WORK(&rcache->work, iova_depot_work_func);
rcache->cpu_rcaches = __alloc_percpu(sizeof(*cpu_rcache), rcache->cpu_rcaches = __alloc_percpu(sizeof(*cpu_rcache),
cache_line_size()); cache_line_size());
if (!rcache->cpu_rcaches) { if (!rcache->cpu_rcaches) {
...@@ -812,6 +840,7 @@ static bool __iova_rcache_insert(struct iova_domain *iovad, ...@@ -812,6 +840,7 @@ static bool __iova_rcache_insert(struct iova_domain *iovad,
spin_lock(&rcache->lock); spin_lock(&rcache->lock);
iova_depot_push(rcache, cpu_rcache->loaded); iova_depot_push(rcache, cpu_rcache->loaded);
spin_unlock(&rcache->lock); spin_unlock(&rcache->lock);
schedule_delayed_work(&rcache->work, IOVA_DEPOT_DELAY);
cpu_rcache->loaded = new_mag; cpu_rcache->loaded = new_mag;
can_insert = true; can_insert = true;
...@@ -912,6 +941,7 @@ static void free_iova_rcaches(struct iova_domain *iovad) ...@@ -912,6 +941,7 @@ static void free_iova_rcaches(struct iova_domain *iovad)
iova_magazine_free(cpu_rcache->prev); iova_magazine_free(cpu_rcache->prev);
} }
free_percpu(rcache->cpu_rcaches); free_percpu(rcache->cpu_rcaches);
cancel_delayed_work_sync(&rcache->work);
while (rcache->depot) while (rcache->depot)
iova_magazine_free(iova_depot_pop(rcache)); iova_magazine_free(iova_depot_pop(rcache));
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment