From: Christoph Heiss <c.heiss@proxmox.com>
To: pve-devel@lists.proxmox.com
Subject: [pve-devel] [PATCH installer 0/6] fix #4829: set up a lower default limit for the ZFS ARC in the installer
Date: Tue, 22 Aug 2023 12:24:48 +0200 [thread overview]
Message-ID: <20230822102533.295530-1-c.heiss@proxmox.com> (raw)
Fixes #4829. Introduces a new ZFS install option `arc_max` (aptly named
after the corresponding module option `zfs_arc_max`).
For PVE installations, this can be adjusted when creating a ZFS RAID
under "Advanced Options". The default value is choosen as 10% of system
memory, clamped to between 64 MiB as lower limit and 16 GiB as upper
limit. For PBS and PMG, the option is (currently) hidden.
If the option is set to a non-zero value, a new file
/etc/modprobe.d/zfs.conf gets written during install, setting the
`zfs_arc_max` module option as appropriate.
Tested by installing PVE, PBS and PMG. For PVE, checked that the `zfs`
module option gets correctly written & applied, the latter by looking at
the output of `arc_summary`. For PBS and PMG, verified that no modprobe
options file is created and the ARC size is set to default.
Christoph Heiss (6):
fix #4829: install: add new ZFS `arc_max` setup option
fix #4829: proxinstall: expose new `arc_max` ZFS option for PVE
installations
fix #4829: test: add tests for new zfs_arc_max calculations
tui: views: add optional suffix label for `NumericEditView`s
fix #4829: tui: setup: add new ZFS `arc_max` option
fix #4829: tui: bootdisk: expose new `arc_max` ZFS option for PVE
installations
Makefile | 3 +
Proxmox/Install.pm | 4 +
Proxmox/Install/Config.pm | 1 +
Proxmox/Install/RunEnv.pm | 38 +++++++
debian/control | 1 +
proxinstall | 15 +++
proxmox-tui-installer/src/main.rs | 2 +-
proxmox-tui-installer/src/options.rs | 55 ++++++++++-
proxmox-tui-installer/src/setup.rs | 2 +
proxmox-tui-installer/src/views/bootdisk.rs | 59 ++++++++---
proxmox-tui-installer/src/views/mod.rs | 104 +++++++++++++++-----
test/Makefile | 10 ++
test/zfs-arc-max.pl | 81 +++++++++++++++
13 files changed, 333 insertions(+), 42 deletions(-)
create mode 100644 test/Makefile
create mode 100755 test/zfs-arc-max.pl
--
2.41.0
next reply other threads:[~2023-08-22 10:26 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-08-22 10:24 Christoph Heiss [this message]
2023-08-22 10:24 ` [pve-devel] [PATCH installer 1/6] fix #4829: install: add new ZFS `arc_max` setup option Christoph Heiss
2023-08-22 10:24 ` [pve-devel] [PATCH installer 2/6] fix #4829: proxinstall: expose new `arc_max` ZFS option for PVE installations Christoph Heiss
2023-08-22 10:24 ` [pve-devel] [PATCH installer 3/6] fix #4829: test: add tests for new zfs_arc_max calculations Christoph Heiss
2023-08-22 10:24 ` [pve-devel] [PATCH installer 4/6] fix #4829: tui: views: add optional suffix label for `NumericEditView`s Christoph Heiss
2023-08-22 10:24 ` [pve-devel] [PATCH installer 5/6] fix #4829: tui: setup: add new ZFS `arc_max` option Christoph Heiss
2023-08-22 10:24 ` [pve-devel] [PATCH installer 6/6] fix #4829: tui: bootdisk: expose new `arc_max` ZFS option for PVE installations Christoph Heiss
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20230822102533.295530-1-c.heiss@proxmox.com \
--to=c.heiss@proxmox.com \
--cc=pve-devel@lists.proxmox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox