From ce3f15aeac3a329c8ed844add672d3bb2a109eeb Mon Sep 17 00:00:00 2001 From: Stoiko Ivanov Date: Thu, 15 Jun 2023 12:42:56 +0200 Subject: [PATCH] file-restore: set zfs_arc_min to current minimum of 32M zfs_arc_min was raised to 32M (for linux) in zfs-commit 121b3cae742a0670d902a51bc61d49dc4a3e4445 while the current logic would still set the min_size to 32M (it's max(32M, allmem/32), which results to 32M for memory sizes up to 1024M), setting it explicitly to the minimum makes it clear, and will still be kept should the restore vm have more than 1G of memory at some point. Signed-off-by: Stoiko Ivanov --- proxmox-file-restore/src/qemu_helper.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/proxmox-file-restore/src/qemu_helper.rs b/proxmox-file-restore/src/qemu_helper.rs index 12c8cabf..0219a310 100644 --- a/proxmox-file-restore/src/qemu_helper.rs +++ b/proxmox-file-restore/src/qemu_helper.rs @@ -260,7 +260,7 @@ pub async fn start_vm( // NOTE: ZFS requires that the ARC can at least grow to the max transaction size of 64MB // also: setting any of min/max to zero will rather do the opposite of what one wants here &format!( - "{} panic=1 zfs.zfs_arc_min=16777216 zfs.zfs_arc_max=67108864 memhp_default_state=online_kernel", + "{} panic=1 zfs.zfs_arc_min=33554432 zfs.zfs_arc_max=67108864 memhp_default_state=online_kernel", if debug { "debug" } else { "quiet" } ), "-daemonize",