lkml.org 
[lkml]   [2013]   [Jul]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] mm: zswap: add runtime enable/disable
On 07/23/2013 03:34 AM, Seth Jennings wrote:
> Right now, zswap can only be enabled at boot time. This patch
> modifies zswap so that it can be dynamically enabled or disabled
> at runtime.
>
> In order to allow this ability, zswap unconditionally registers as a
> frontswap backend regardless of whether or not zswap.enabled=1 is passed
> in the boot parameters or not. This introduces a very small overhead
> for systems that have zswap disabled as calls to frontswap_store() will
> call zswap_frontswap_store(), but there is a fast path to immediately
> return if zswap is disabled.

There is also overhead in frontswap_load() after all pages are faulted
back into memory.

>
> Disabling zswap does not unregister zswap from frontswap. It simply
> blocks all future stores.
>
> Signed-off-by: Seth Jennings <sjenning@linux.vnet.ibm.com>
> ---
> Documentation/vm/zswap.txt | 18 ++++++++++++++++--
> mm/zswap.c | 9 +++------
> 2 files changed, 19 insertions(+), 8 deletions(-)
>
> diff --git a/Documentation/vm/zswap.txt b/Documentation/vm/zswap.txt
> index 7e492d8..d588477 100644
> --- a/Documentation/vm/zswap.txt
> +++ b/Documentation/vm/zswap.txt
> @@ -26,8 +26,22 @@ Zswap evicts pages from compressed cache on an LRU basis to the backing swap
> device when the compressed pool reaches it size limit. This requirement had
> been identified in prior community discussions.
>
> -To enabled zswap, the "enabled" attribute must be set to 1 at boot time. e.g.
> -zswap.enabled=1
> +Zswap is disabled by default but can be enabled at boot time by setting
> +the "enabled" attribute to 1 at boot time. e.g. zswap.enabled=1. Zswap
> +can also be enabled and disabled at runtime using the sysfs interface.
> +An exmaple command to enable zswap at runtime, assuming sysfs is mounted
> +at /sys, is:
> +
> +echo 1 > /sys/modules/zswap/parameters/enabled
> +
> +When zswap is disabled at runtime, it will stop storing pages that are
> +being swapped out. However, it will _not_ immediately write out or
> +fault back into memory all of the pages stored in the compressed pool.

I don't know what's you use case of adding this feature.
In my opinion I'd perfer to flush all the pages stored in zswap when
disabled it, so that I can run testing without rebooting the machine.

> +The pages stored in zswap will continue to remain in the compressed pool
> +until they are either invalidated or faulted back into memory. In order
> +to force all pages out of the compressed pool, a swapoff on the swap
> +device(s) will fault all swapped out pages, included those in the
> +compressed pool, back into memory.
>
> Design:
>
> diff --git a/mm/zswap.c b/mm/zswap.c
> index deda2b6..199b1b0 100644
> --- a/mm/zswap.c
> +++ b/mm/zswap.c
> @@ -75,9 +75,9 @@ static u64 zswap_duplicate_entry;
> /*********************************
> * tunables
> **********************************/
> -/* Enable/disable zswap (disabled by default, fixed at boot for now) */
> +/* Enable/disable zswap (disabled by default) */
> static bool zswap_enabled __read_mostly;
> -module_param_named(enabled, zswap_enabled, bool, 0);
> +module_param_named(enabled, zswap_enabled, bool, 0644);
>
> /* Compressor to be used by zswap (fixed at boot for now) */
> #define ZSWAP_COMPRESSOR_DEFAULT "lzo"
> @@ -612,7 +612,7 @@ static int zswap_frontswap_store(unsigned type, pgoff_t offset,
> u8 *src, *dst;
> struct zswap_header *zhdr;
>
> - if (!tree) {
> + if (!zswap_enabled || !tree) {
> ret = -ENODEV;
> goto reject;
> }
> @@ -908,9 +908,6 @@ static void __exit zswap_debugfs_exit(void) { }
> **********************************/
> static int __init init_zswap(void)
> {
> - if (!zswap_enabled)
> - return 0;
> -
> pr_info("loading zswap\n");
> if (zswap_entry_cache_create()) {
> pr_err("entry cache creation failed\n");
>

--
Regards,
-Bob


\
 
 \ /
  Last update: 2013-07-23 12:01    [W:0.068 / U:0.124 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site