zram: make default ZSTD compression level configurable

Commit Message

LiviaMedeiros Dec. 6, 2024, 8:16 p.m. UTC

From: LiviaMedeiros <[e-mail]>

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 <[e-mail]>
---
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(-)

Comments

Sergey Senozhatsky Dec. 7, 2024, 3:54 a.m. UTC | #1

On (24/12/07 04:16), LiviaMedeiros wrote:
> From: LiviaMedeiros <[e-mail]>
> 
> 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.

Please let's not.  Then somebody will need options to .config c/d dicts
for zstd/lz4/lz4hc, acceleration levels for lz4, compression level for
deflate and so on and on and on.

[..]
> 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.

This should work, but you need to configure algo params before you init
zram device (that is _before_ write to disksize device attr).

E.g.

% modprobe zram
% echo zstd > /sys/block/zram0/comp_algorithm
% echo "algo=zstd level=9 dict=/etc/zstd-dict-amd64" > /sys/block/zram0/algorithm_params
% echo 1G > /sys/block/zram0/disksize

Patch

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);