* [pve-devel] [PATCH pve-docs v2 0/2] PVE docs: Made small edits.
@ 2025-02-03 11:14 Alexander Abraham
2025-02-03 11:14 ` [pve-devel] [PATCH pve-docs v2 1/2] Section on ZFS and swap corrected Alexander Abraham
2025-02-03 11:14 ` [pve-devel] [PATCH pve-docs v2 2/2] PVE docs: Made small edits Alexander Abraham
0 siblings, 2 replies; 5+ messages in thread
From: Alexander Abraham @ 2025-02-03 11:14 UTC (permalink / raw)
To: pve-devel; +Cc: Alexander Abraham
Split a long sentence in the section on Swap on ZFS
and fixed some typos.
Alexander Abraham (2):
Section on ZFS and swap corrected.
PVE docs: Made small edits.
local-zfs.adoc | 37 ++-----------------------------------
1 file changed, 2 insertions(+), 35 deletions(-)
--
2.39.5
_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
^ permalink raw reply [flat|nested] 5+ messages in thread
* [pve-devel] [PATCH pve-docs v2 1/2] Section on ZFS and swap corrected.
2025-02-03 11:14 [pve-devel] [PATCH pve-docs v2 0/2] PVE docs: Made small edits Alexander Abraham
@ 2025-02-03 11:14 ` Alexander Abraham
2025-02-03 12:23 ` Fiona Ebner
2025-02-03 12:34 ` Alexander Zeidler
2025-02-03 11:14 ` [pve-devel] [PATCH pve-docs v2 2/2] PVE docs: Made small edits Alexander Abraham
1 sibling, 2 replies; 5+ messages in thread
From: Alexander Abraham @ 2025-02-03 11:14 UTC (permalink / raw)
To: pve-devel; +Cc: Alexander Abraham
---
local-zfs.adoc | 36 +-----------------------------------
1 file changed, 1 insertion(+), 35 deletions(-)
diff --git a/local-zfs.adoc b/local-zfs.adoc
index c64fb27..bcd02f7 100644
--- a/local-zfs.adoc
+++ b/local-zfs.adoc
@@ -627,41 +627,7 @@ You *must reboot* to activate these changes.
SWAP on ZFS
~~~~~~~~~~~
-Swap-space created on a zvol may generate some troubles, like blocking the
-server or generating a high IO load, often seen when starting a Backup
-to an external Storage.
-
-We strongly recommend to use enough memory, so that you normally do not
-run into low memory situations. Should you need or want to add swap, it is
-preferred to create a partition on a physical disk and use it as a swap device.
-You can leave some space free for this purpose in the advanced options of the
-installer. Additionally, you can lower the
-``swappiness'' value. A good value for servers is 10:
-
-----
-# sysctl -w vm.swappiness=10
-----
-
-To make the swappiness persistent, open `/etc/sysctl.conf` with
-an editor of your choice and add the following line:
-
---------
-vm.swappiness = 10
---------
-
-.Linux kernel `swappiness` parameter values
-[width="100%",cols="<m,2d",options="header"]
-|===========================================================
-| Value | Strategy
-| vm.swappiness = 0 | The kernel will swap only to avoid
-an 'out of memory' condition
-| vm.swappiness = 1 | Minimum amount of swapping without
-disabling it entirely.
-| vm.swappiness = 10 | This value is sometimes recommended to
-improve performance when sufficient memory exists in a system.
-| vm.swappiness = 60 | The default value.
-| vm.swappiness = 100 | The kernel will swap aggressively.
-|===========================================================
+It is strongly recommended to not use a ZFS volume for a swap partition because this could lead to deadlocks freezing thge affected system and other unpredictable behavior. The OpenZFS documentation, which can be viewed https://openzfs.github.io/openzfs-docs/Project%20and%20Community/FAQ.html#using-a-zvol-for-a-swap-device-on-linux[here], warns against using a ZFS volume for a swap partition.
[[zfs_encryption]]
Encrypted ZFS Datasets
--
2.39.5
_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
^ permalink raw reply [flat|nested] 5+ messages in thread
* [pve-devel] [PATCH pve-docs v2 2/2] PVE docs: Made small edits.
2025-02-03 11:14 [pve-devel] [PATCH pve-docs v2 0/2] PVE docs: Made small edits Alexander Abraham
2025-02-03 11:14 ` [pve-devel] [PATCH pve-docs v2 1/2] Section on ZFS and swap corrected Alexander Abraham
@ 2025-02-03 11:14 ` Alexander Abraham
1 sibling, 0 replies; 5+ messages in thread
From: Alexander Abraham @ 2025-02-03 11:14 UTC (permalink / raw)
To: pve-devel; +Cc: Alexander Abraham
A long sentence in the section about Swap on ZFS was split
into two sentences and some typos were corrected.
Signed-off-by: Alexander Abraham <a.abraham@proxmox.com>
---
local-zfs.adoc | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/local-zfs.adoc b/local-zfs.adoc
index bcd02f7..b662233 100644
--- a/local-zfs.adoc
+++ b/local-zfs.adoc
@@ -627,7 +627,8 @@ You *must reboot* to activate these changes.
SWAP on ZFS
~~~~~~~~~~~
-It is strongly recommended to not use a ZFS volume for a swap partition because this could lead to deadlocks freezing thge affected system and other unpredictable behavior. The OpenZFS documentation, which can be viewed https://openzfs.github.io/openzfs-docs/Project%20and%20Community/FAQ.html#using-a-zvol-for-a-swap-device-on-linux[here], warns against using a ZFS volume for a swap partition.
+It is strongly recommended to not use a ZFS volume for a swap partition because this could lead to deadlocks. These deadlocks could cause the affected system to freeze.
+The OpenZFS documentation, which can be viewed https://openzfs.github.io/openzfs-docs/Project%20and%20Community/FAQ.html#using-a-zvol-for-a-swap-device-on-linux[here], warns about using a ZFS volume for a swap partition.
[[zfs_encryption]]
Encrypted ZFS Datasets
--
2.39.5
_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [pve-devel] [PATCH pve-docs v2 1/2] Section on ZFS and swap corrected.
2025-02-03 11:14 ` [pve-devel] [PATCH pve-docs v2 1/2] Section on ZFS and swap corrected Alexander Abraham
@ 2025-02-03 12:23 ` Fiona Ebner
2025-02-03 12:34 ` Alexander Zeidler
1 sibling, 0 replies; 5+ messages in thread
From: Fiona Ebner @ 2025-02-03 12:23 UTC (permalink / raw)
To: Proxmox VE development discussion, Alexander Abraham
The subject prefix is correct now. But it seems like most of the other
feedback given for v1 still applies [0].
Also, there is no need to have two separate patches if you just fix up
the changes you just introduced in the second one. Then the patches can
be squashed together into a single one.
[0]:
https://lore.proxmox.com/pve-devel/71eb4140-1e06-43ba-bbe4-76a39d5e9582@proxmox.com/
Am 03.02.25 um 12:14 schrieb Alexander Abraham:
> ---
> local-zfs.adoc | 36 +-----------------------------------
> 1 file changed, 1 insertion(+), 35 deletions(-)
>
> diff --git a/local-zfs.adoc b/local-zfs.adoc
> index c64fb27..bcd02f7 100644
> --- a/local-zfs.adoc
> +++ b/local-zfs.adoc
> @@ -627,41 +627,7 @@ You *must reboot* to activate these changes.
> SWAP on ZFS
> ~~~~~~~~~~~
>
> -Swap-space created on a zvol may generate some troubles, like blocking the
> -server or generating a high IO load, often seen when starting a Backup
> -to an external Storage.
> -
> -We strongly recommend to use enough memory, so that you normally do not
> -run into low memory situations. Should you need or want to add swap, it is
> -preferred to create a partition on a physical disk and use it as a swap device.
> -You can leave some space free for this purpose in the advanced options of the
> -installer. Additionally, you can lower the
> -``swappiness'' value. A good value for servers is 10:
> -
> -----
> -# sysctl -w vm.swappiness=10
> -----
> -
> -To make the swappiness persistent, open `/etc/sysctl.conf` with
> -an editor of your choice and add the following line:
> -
> ---------
> -vm.swappiness = 10
> ---------
> -
> -.Linux kernel `swappiness` parameter values
> -[width="100%",cols="<m,2d",options="header"]
> -|===========================================================
> -| Value | Strategy
> -| vm.swappiness = 0 | The kernel will swap only to avoid
> -an 'out of memory' condition
> -| vm.swappiness = 1 | Minimum amount of swapping without
> -disabling it entirely.
> -| vm.swappiness = 10 | This value is sometimes recommended to
> -improve performance when sufficient memory exists in a system.
> -| vm.swappiness = 60 | The default value.
> -| vm.swappiness = 100 | The kernel will swap aggressively.
> -|===========================================================
> +It is strongly recommended to not use a ZFS volume for a swap partition because this could lead to deadlocks freezing thge affected system and other unpredictable behavior. The OpenZFS documentation, which can be viewed https://openzfs.github.io/openzfs-docs/Project%20and%20Community/FAQ.html#using-a-zvol-for-a-swap-device-on-linux[here], warns against using a ZFS volume for a swap partition.
>
> [[zfs_encryption]]
> Encrypted ZFS Datasets
_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [pve-devel] [PATCH pve-docs v2 1/2] Section on ZFS and swap corrected.
2025-02-03 11:14 ` [pve-devel] [PATCH pve-docs v2 1/2] Section on ZFS and swap corrected Alexander Abraham
2025-02-03 12:23 ` Fiona Ebner
@ 2025-02-03 12:34 ` Alexander Zeidler
1 sibling, 0 replies; 5+ messages in thread
From: Alexander Zeidler @ 2025-02-03 12:34 UTC (permalink / raw)
To: Proxmox VE development discussion; +Cc: Alexander Abraham
IIRC, there have been some considerations in the past to move the "SWAP
on ZFS" section to the chapter "Host System Administration".
If doing so:
1. It could be renamed to "SWAP" and, with the current layout, placed
between "Disk Health Monitoring" and "Logical Volume Manager (LVM)"?
2. Update the first paragraph accordingly, so that, for example, it
includes your new sentences and maybe starts with something like "If
using ZFS, [it is strongly recommended ...]".
On Mon Feb 3, 2025 at 12:14 PM CET, Alexander Abraham wrote:
> ---
> local-zfs.adoc | 36 +-----------------------------------
> 1 file changed, 1 insertion(+), 35 deletions(-)
>
> diff --git a/local-zfs.adoc b/local-zfs.adoc
> index c64fb27..bcd02f7 100644
> --- a/local-zfs.adoc
> +++ b/local-zfs.adoc
> @@ -627,41 +627,7 @@ You *must reboot* to activate these changes.
> SWAP on ZFS
> ~~~~~~~~~~~
>
> -Swap-space created on a zvol may generate some troubles, like blocking the
> -server or generating a high IO load, often seen when starting a Backup
> -to an external Storage.
> -
> -We strongly recommend to use enough memory, so that you normally do not
> -run into low memory situations. Should you need or want to add swap, it is
> -preferred to create a partition on a physical disk and use it as a swap device.
> -You can leave some space free for this purpose in the advanced options of the
> -installer. Additionally, you can lower the
> -``swappiness'' value. A good value for servers is 10:
> -
> -----
> -# sysctl -w vm.swappiness=10
> -----
> -
> -To make the swappiness persistent, open `/etc/sysctl.conf` with
> -an editor of your choice and add the following line:
> -
> ---------
> -vm.swappiness = 10
> ---------
> -
> -.Linux kernel `swappiness` parameter values
> -[width="100%",cols="<m,2d",options="header"]
> -|===========================================================
> -| Value | Strategy
> -| vm.swappiness = 0 | The kernel will swap only to avoid
> -an 'out of memory' condition
> -| vm.swappiness = 1 | Minimum amount of swapping without
> -disabling it entirely.
> -| vm.swappiness = 10 | This value is sometimes recommended to
> -improve performance when sufficient memory exists in a system.
> -| vm.swappiness = 60 | The default value.
> -| vm.swappiness = 100 | The kernel will swap aggressively.
> -|===========================================================
> +It is strongly recommended to not use a ZFS volume for a swap partition because this could lead to deadlocks freezing thge affected system and other unpredictable behavior. The OpenZFS documentation, which can be viewed https://openzfs.github.io/openzfs-docs/Project%20and%20Community/FAQ.html#using-a-zvol-for-a-swap-device-on-linux[here], warns against using a ZFS volume for a swap partition.
>
> [[zfs_encryption]]
> Encrypted ZFS Datasets
_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2025-02-03 12:35 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-02-03 11:14 [pve-devel] [PATCH pve-docs v2 0/2] PVE docs: Made small edits Alexander Abraham
2025-02-03 11:14 ` [pve-devel] [PATCH pve-docs v2 1/2] Section on ZFS and swap corrected Alexander Abraham
2025-02-03 12:23 ` Fiona Ebner
2025-02-03 12:34 ` Alexander Zeidler
2025-02-03 11:14 ` [pve-devel] [PATCH pve-docs v2 2/2] PVE docs: Made small edits Alexander Abraham
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox