[RFC PATCH v2 09/10] mm, swap: add debugfs knob for xswap per-device cluster limit
Baoquan He <[email protected]>
| Newsgroups | gmane.linux.kernel,gmane.linux.kernel.mm |
|---|---|
| Message-ID | <[email protected]> |
Add a per-device debugfs file for runtime adjustment of xswap cluster limit: /sys/kernel/debug/xswap/type<N>_cluster_limit Reading shows the current ceiling (in clusters); writing sets it (clamped to [0, nr_clusters_max]). Setting below nr_clusters_mapped triggers an immediate shrink check via xswap_try_shrink(). The debugfs entry is created at swapon and removed at swapoff. Signed-off-by: Baoquan He <[email protected]> --- include/linux/swap.h | 1 + mm/swapfile.c | 97 +++++++++++++++++++++++++++++++++++++++++++- 2 files changed, 97 insertions(+), 1 deletion(-) diff --git a/include/linux/swap.h b/include/linux/swap.h index 1159153459a1..4f4583f9a4e5 100644 --- a/include/linux/swap.h +++ b/include/linux/swap.h @@ -254,6 +254,7 @@ struct swap_info_struct { unsigned long nr_clusters; /* current growth ceiling (≤ nr_clusters_max) */ unsigned long nr_clusters_mapped; /* currently mapped cluster count */ unsigned long nr_free_tail; /* contiguous free clusters at tail */ + struct dentry *debugfs_entry; /* debugfs: type<N>_max_clusters */ struct mutex xswap_lock; /* serialize map/unmap operations */ #endif struct list_head free_clusters; /* free clusters list */ diff --git a/mm/swapfile.c b/mm/swapfile.c index 3037f428f217..265f2bac1a12 100644 --- a/mm/swapfile.c +++ b/mm/swapfile.c @@ -48,6 +48,9 @@ #include "swap_table.h" #include "internal.h" #include "swap.h" +#include <linux/debugfs.h> + +static DEFINE_SPINLOCK(swap_lock); #ifdef CONFIG_XSWAP /* @@ -63,6 +66,8 @@ #define XSWAP_GROW_CLUSTERS \ max_t(unsigned long, PAGE_SIZE / sizeof(struct swap_cluster_info), 16) +static struct dentry *xswap_debugfs_root; + static int xswap_map_clusters(struct swap_info_struct *si, unsigned long start_idx, unsigned long nr); static void xswap_unmap_clusters(struct swap_info_struct *si, @@ -73,6 +78,90 @@ static void xswap_update_free_tail(struct swap_info_struct *si, unsigned long freed_idx); static void xswap_try_shrink(struct swap_info_struct *si); +/* + * debugfs read/write for per-device max cluster count. + * Shows/sets si->nr_clusters (current growth ceiling), clamped to + * [0, si->nr_clusters_max]. + */ +static ssize_t xswap_max_clusters_read(struct file *file, char __user *buf, + size_t count, loff_t *ppos) +{ + struct swap_info_struct *si = file->private_data; + char tmp[32]; + int len; + + len = snprintf(tmp, sizeof(tmp), "%lu\n", READ_ONCE(si->nr_clusters)); + return simple_read_from_buffer(buf, count, ppos, tmp, len); +} + +static ssize_t xswap_max_clusters_write(struct file *file, + const char __user *buf, + size_t count, loff_t *ppos) +{ + struct swap_info_struct *si = file->private_data; + unsigned long val, new_pages; + int err; + + err = kstrtoul_from_user(buf, count, 0, &val); + if (err) + return err; + + if (val > si->nr_clusters_max) + val = si->nr_clusters_max; + + spin_lock(&si->lock); + si->nr_clusters = val; + spin_unlock(&si->lock); + + /* Keep the visible swap size in sync with the new ceiling. */ + new_pages = min_t(unsigned long, val * SWAPFILE_CLUSTER, si->max); + if (new_pages) + new_pages--; + if (new_pages != si->pages) { + long delta = (long)new_pages - (long)si->pages; + + spin_lock(&swap_lock); + si->pages = new_pages; + atomic_long_add(delta, &nr_swap_pages); + total_swap_pages += delta; + spin_unlock(&swap_lock); + } + + /* + * Lowering the ceiling may make tail clusters eligible for + * shrinking. Trigger an immediate check. + */ + xswap_try_shrink(si); + + return count; +} + +static const struct file_operations xswap_debugfs_fops = { + .read = xswap_max_clusters_read, + .write = xswap_max_clusters_write, + .open = simple_open, + .llseek = default_llseek, +}; + +static void xswap_debugfs_add(struct swap_info_struct *si) +{ + char name[32]; + + if (!xswap_debugfs_root) + return; + + snprintf(name, sizeof(name), "type%d_cluster_limit", si->type); + si->debugfs_entry = debugfs_create_file(name, 0644, xswap_debugfs_root, + si, &xswap_debugfs_fops); +} + +static void xswap_debugfs_del(struct swap_info_struct *si) +{ + debugfs_remove(si->debugfs_entry); + si->debugfs_entry = NULL; +} + + #endif static void swap_range_alloc(struct swap_info_struct *si, @@ -89,7 +178,6 @@ static void move_cluster(struct swap_info_struct *si, * * Also protects swap_active_head total_swap_pages, and the SWP_WRITEOK flag. */ -static DEFINE_SPINLOCK(swap_lock); static unsigned int nr_swapfiles; atomic_long_t nr_swap_pages; atomic_t nr_real_swapfiles; @@ -3147,6 +3235,7 @@ static void free_swap_cluster_info(struct swap_info_struct *si) #ifdef CONFIG_XSWAP if (si->flags & SWP_XSWAP) { + xswap_debugfs_del(si); /* Unmap all mapped clusters and free the VM_SPARSE area */ if (si->nr_clusters_mapped > 0) xswap_unmap_clusters(si, 0, si->nr_clusters_mapped); @@ -4075,6 +4164,7 @@ static int setup_swap_clusters_info(struct swap_info_struct *si, /* All mapped clusters except cluster 0 are free at the tail */ si->nr_free_tail = si->nr_clusters_mapped - 1; mutex_init(&si->xswap_lock); + xswap_debugfs_add(si); return 0; err_unmap: @@ -4498,6 +4588,11 @@ static int __init swapfile_init(void) if (swapfile_maximum_size >= (1UL << SWP_MIG_TOTAL_BITS)) swap_migration_ad_supported = true; #endif /* CONFIG_MIGRATION */ + +#ifdef CONFIG_XSWAP + xswap_debugfs_root = debugfs_create_dir("xswap", NULL); +#endif + return 0; } subsys_initcall(swapfile_init); -- 2.54.0