Re: [pve-devel] [PATCH pve-storage] lvm: use blkdiscard instead cstream to saferemove drive

["DERUMIER, Alexandre via pve-devel" <pve-devel@lists.proxmox.com>]

[-- Attachment #1: Type: message/rfc822, Size: 12883 bytes --]

From: "DERUMIER, Alexandre" <alexandre.derumier@groupe-cyllene.com>
To: "w.bumiller@proxmox.com" <w.bumiller@proxmox.com>, "pve-devel@lists.proxmox.com" <pve-devel@lists.proxmox.com>
Subject: Re: [pve-devel] [PATCH pve-storage] lvm: use blkdiscard instead cstream to saferemove drive
Date: Thu, 14 Aug 2025 10:07:59 +0000
Message-ID: <9d5353263ee0aa3ee67d8d331f674f8a00044b1e.camel@groupe-cyllene.com>

> +        saferemove_discard => {
> +            description => "Wipe with discard instead zeroing.",
> +            type => 'boolean',

>>Not sure we need this (not sure when this is actually useful), but
>>it's
>>cheap enough to have around. Should add a `default => 0` for
>>documentation purposes, though.

Some storage allow overprovisioning. (create a lun bigger than the real
storage size), it can be interesting to discard instead zeroing (to
free space on storage side).
as snapshots currently use lvm volume with same size than the main
volume, it can be interesting to have overprovisioning.


btw, blockbridge guys have made a nice article here:
https://kb.blockbridge.com/technote/proxmox-qcow-snapshots-on-lvm/







[-- Attachment #2: Type: text/plain, Size: 160 bytes --]

_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel