[PATCH] zram: make default ZSTD compression level configurable

From: LiviaMedeiros
Date: Fri Dec 06 2024 - 15:23:51 EST


From: LiviaMedeiros <livia@xxxxxxxxxx>

Add support for configuring the default ZSTD compression level for zram
devices via the CONFIG_ZRAM_DEFAULT_ZSTD_LEVEL configuration option.
If this option is not set, fallback to zstd_default_clevel() is used.

Signed-off-by: LiviaMedeiros <livia@xxxxxxxxxx>
---
Disclaimer:
This probably should be implemented in userspace, I failed to find relevant options in zram-init (v11.1) scripts nor zramctl (v2.40.2) utility.
I also failed to make something like `echo "algo=zstd level=9" > /sys/block/zram3/algorithm_params` work in runtime, but I assume I'm just dumb and/or it requires extra steps.

Nevertheless IMHO it makes sense to set defaults in kernel config: when kernel is built for specific hardware, it would be handy to adjust it depending on amount of extra CPU power or extra RAM.
The main usecase for adjusting the compression level that I see is making zram device capable of holding _a lot_ of transparently compressed text data, fitting into RAM at the cost of additional CPU time.
Hence this patch implements this only for zstd backend, which provides the best compression ratio potential (perhaps implementing xz/LZMA2 backend would also help with it).
If this approach looks feasible, it can be expanded to other algos as well.

I'm open for any suggestions.
---
drivers/block/zram/Kconfig | 10 ++++++++++
drivers/block/zram/backend_zstd.c | 8 +++++++-
2 files changed, 17 insertions(+), 1 deletion(-)

diff --git a/drivers/block/zram/Kconfig b/drivers/block/zram/Kconfig
index 402b7b175863..8a5e6da39834 100644
--- a/drivers/block/zram/Kconfig
+++ b/drivers/block/zram/Kconfig
@@ -32,6 +32,16 @@ config ZRAM_BACKEND_ZSTD
select ZSTD_COMPRESS
select ZSTD_DECOMPRESS

+config ZRAM_DEFAULT_ZSTD_LEVEL
+ int "Default zstd compression level"
+ depends on ZRAM_BACKEND_ZSTD
+ range 1 22
+ default 3
+ help
+ Sets the default compression level for zstd compression in zram.
+ The value can range from 1 (fastest) to 22 (maximum compression).
+ If not set, the system uses the zstd default (typically 3).
+
config ZRAM_BACKEND_DEFLATE
bool "deflate compression support"
depends on ZRAM
diff --git a/drivers/block/zram/backend_zstd.c b/drivers/block/zram/backend_zstd.c
index 1184c0036f44..af7b919ec11c 100644
--- a/drivers/block/zram/backend_zstd.c
+++ b/drivers/block/zram/backend_zstd.c
@@ -7,6 +7,12 @@

#include "backend_zstd.h"

+#ifdef CONFIG_ZRAM_DEFAULT_ZSTD_LEVEL
+#define ZRAM_DEFAULT_ZSTD_LEVEL CONFIG_ZRAM_DEFAULT_ZSTD_LEVEL
+#else
+#define ZRAM_DEFAULT_ZSTD_LEVEL zstd_default_clevel()
+#endif
+
struct zstd_ctx {
zstd_cctx *cctx;
zstd_dctx *dctx;
@@ -68,7 +74,7 @@ static int zstd_setup_params(struct zcomp_params *params)

params->drv_data = zp;
if (params->level == ZCOMP_PARAM_NO_LEVEL)
- params->level = zstd_default_clevel();
+ params->level = ZRAM_DEFAULT_ZSTD_LEVEL;

zp->cprm = zstd_get_params(params->level, PAGE_SIZE);

--
2.47.1