* [PATCH manager] ui: qemu: add detect zeroes disk option
@ 2026-06-08 8:36 Lukas Sichert
2026-06-16 13:01 ` Fabian Grünbichler
2026-06-16 13:16 ` superseded: " Lukas Sichert
0 siblings, 2 replies; 5+ messages in thread
From: Lukas Sichert @ 2026-06-08 8:36 UTC (permalink / raw)
To: pve-devel; +Cc: Lukas Sichert
When discard is enabled, qemu-server defaults to `detect-zeroes=unmap`.
This can have a significant performance impact for zero writes on some
setups, while users might still want to keep guest discard/TRIM support
enabled.
Allow configuring the option independently so discard can stay enabled
while zero detection is disabled. Persist the checkbox state explicitly,
as the backend treats an absent value as the default-enabled behavior.
Signed-off-by: Lukas Sichert <l.sichert@proxmox.com>
---
www/manager6/qemu/HDEdit.js | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/www/manager6/qemu/HDEdit.js b/www/manager6/qemu/HDEdit.js
index 1bb2bfda..d1d1b0fa 100644
--- a/www/manager6/qemu/HDEdit.js
+++ b/www/manager6/qemu/HDEdit.js
@@ -96,6 +96,7 @@ Ext.define('PVE.qemu.HDInputPanel', {
PVE.Utils.propertyStringSet(me.drive, values.readOnly, 'ro', 'on');
PVE.Utils.propertyStringSet(me.drive, values.cache, 'cache');
PVE.Utils.propertyStringSet(me.drive, values.aio, 'aio');
+ me.drive.detect_zeroes = values.detect_zeroes ? 1 : 0;
['mbps_rd', 'mbps_wr', 'iops_rd', 'iops_wr'].forEach((name) => {
let burst_name = `${name}_max`;
@@ -152,6 +153,7 @@ Ext.define('PVE.qemu.HDInputPanel', {
values.diskformat = drive.format || 'raw';
values.cache = drive.cache || '__default__';
values.discard = drive.discard === 'on';
+ values.detect_zeroes = PVE.Parser.parseBoolean(drive.detect_zeroes, 1);
values.ssd = PVE.Parser.parseBoolean(drive.ssd);
values.iothread = PVE.Parser.parseBoolean(drive.iothread);
values.readOnly = PVE.Parser.parseBoolean(drive.ro);
@@ -317,6 +319,12 @@ Ext.define('PVE.qemu.HDInputPanel', {
reference: 'discard',
name: 'discard',
},
+ {
+ xtype: 'proxmoxcheckbox',
+ fieldLabel: gettext('Detect zeroes'),
+ reference: 'detect_zeroes',
+ name: 'detect_zeroes',
+ },
{
xtype: 'proxmoxcheckbox',
name: 'iothread',
--
2.47.3
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH manager] ui: qemu: add detect zeroes disk option
2026-06-08 8:36 [PATCH manager] ui: qemu: add detect zeroes disk option Lukas Sichert
@ 2026-06-16 13:01 ` Fabian Grünbichler
2026-06-16 13:16 ` Lukas Sichert
2026-06-16 13:16 ` superseded: " Lukas Sichert
1 sibling, 1 reply; 5+ messages in thread
From: Fabian Grünbichler @ 2026-06-16 13:01 UTC (permalink / raw)
To: Lukas Sichert, pve-devel
if I am not mistaken, this is a fix for
https://bugzilla.proxmox.com/show_bug.cgi?id=4503 ?
On June 8, 2026 10:36 am, Lukas Sichert wrote:
> When discard is enabled, qemu-server defaults to `detect-zeroes=unmap`.
> This can have a significant performance impact for zero writes on some
> setups, while users might still want to keep guest discard/TRIM support
> enabled.
>
> Allow configuring the option independently so discard can stay enabled
> while zero detection is disabled. Persist the checkbox state explicitly,
> as the backend treats an absent value as the default-enabled behavior.
>
> Signed-off-by: Lukas Sichert <l.sichert@proxmox.com>
> ---
> www/manager6/qemu/HDEdit.js | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/www/manager6/qemu/HDEdit.js b/www/manager6/qemu/HDEdit.js
> index 1bb2bfda..d1d1b0fa 100644
> --- a/www/manager6/qemu/HDEdit.js
> +++ b/www/manager6/qemu/HDEdit.js
> @@ -96,6 +96,7 @@ Ext.define('PVE.qemu.HDInputPanel', {
> PVE.Utils.propertyStringSet(me.drive, values.readOnly, 'ro', 'on');
> PVE.Utils.propertyStringSet(me.drive, values.cache, 'cache');
> PVE.Utils.propertyStringSet(me.drive, values.aio, 'aio');
> + me.drive.detect_zeroes = values.detect_zeroes ? 1 : 0;
this should probably have a comment why this line is deviating from the
ones above (to not lose an explicit false value that clashes with the
implicit true default value of the backend).
>
> ['mbps_rd', 'mbps_wr', 'iops_rd', 'iops_wr'].forEach((name) => {
> let burst_name = `${name}_max`;
> @@ -152,6 +153,7 @@ Ext.define('PVE.qemu.HDInputPanel', {
> values.diskformat = drive.format || 'raw';
> values.cache = drive.cache || '__default__';
> values.discard = drive.discard === 'on';
> + values.detect_zeroes = PVE.Parser.parseBoolean(drive.detect_zeroes, 1);
> values.ssd = PVE.Parser.parseBoolean(drive.ssd);
> values.iothread = PVE.Parser.parseBoolean(drive.iothread);
> values.readOnly = PVE.Parser.parseBoolean(drive.ro);
> @@ -317,6 +319,12 @@ Ext.define('PVE.qemu.HDInputPanel', {
> reference: 'discard',
> name: 'discard',
> },
> + {
> + xtype: 'proxmoxcheckbox',
> + fieldLabel: gettext('Detect zeroes'),
> + reference: 'detect_zeroes',
> + name: 'detect_zeroes',
> + },
> {
> xtype: 'proxmoxcheckbox',
> name: 'iothread',
> --
> 2.47.3
>
>
>
>
>
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH manager] ui: qemu: add detect zeroes disk option
2026-06-16 13:01 ` Fabian Grünbichler
@ 2026-06-16 13:16 ` Lukas Sichert
2026-06-17 12:12 ` Fabian Grünbichler
0 siblings, 1 reply; 5+ messages in thread
From: Lukas Sichert @ 2026-06-16 13:16 UTC (permalink / raw)
To: Fabian Grünbichler, pve-devel
On 2026-06-16 15:01, Fabian Grünbichler <f.gruenbichler@proxmox.com> wrote:
> if I am not mistaken, this is a fix for
> https://bugzilla.proxmox.com/show_bug.cgi?id=4503 ?
Yes, I have also corrected this in a v2 already [1], but I forgot to send a
superseded-by. Sorry for the confusion.
[1] https://lore.proxmox.com/all/20260610131827.11092-1-l.sichert@proxmox.com/
>
> On June 8, 2026 10:36 am, Lukas Sichert wrote:
>> When discard is enabled, qemu-server defaults to `detect-zeroes=unmap`.
>> This can have a significant performance impact for zero writes on some
>> setups, while users might still want to keep guest discard/TRIM support
>> enabled.
>>
>> Allow configuring the option independently so discard can stay enabled
>> while zero detection is disabled. Persist the checkbox state explicitly,
>> as the backend treats an absent value as the default-enabled behavior.
>>
>> Signed-off-by: Lukas Sichert <l.sichert@proxmox.com>
>> ---
>> www/manager6/qemu/HDEdit.js | 8 ++++++++
>> 1 file changed, 8 insertions(+)
>>
>> diff --git a/www/manager6/qemu/HDEdit.js b/www/manager6/qemu/HDEdit.js
>> index 1bb2bfda..d1d1b0fa 100644
>> --- a/www/manager6/qemu/HDEdit.js
>> +++ b/www/manager6/qemu/HDEdit.js
>> @@ -96,6 +96,7 @@ Ext.define('PVE.qemu.HDInputPanel', {
>> PVE.Utils.propertyStringSet(me.drive, values.readOnly, 'ro', 'on');
>> PVE.Utils.propertyStringSet(me.drive, values.cache, 'cache');
>> PVE.Utils.propertyStringSet(me.drive, values.aio, 'aio');
>> + me.drive.detect_zeroes = values.detect_zeroes ? 1 : 0;
>
> this should probably have a comment why this line is deviating from the
> ones above (to not lose an explicit false value that clashes with the
> implicit true default value of the backend.
Thanks for looking over this. I will fix it in a v3.
>
>>
>> ['mbps_rd', 'mbps_wr', 'iops_rd', 'iops_wr'].forEach((name) => {
>> let burst_name = `${name}_max`;
>> @@ -152,6 +153,7 @@ Ext.define('PVE.qemu.HDInputPanel', {
>> values.diskformat = drive.format || 'raw';
>> values.cache = drive.cache || '__default__';
>> values.discard = drive.discard === 'on';
>> + values.detect_zeroes = PVE.Parser.parseBoolean(drive.detect_zeroes, 1);
>> values.ssd = PVE.Parser.parseBoolean(drive.ssd);
>> values.iothread = PVE.Parser.parseBoolean(drive.iothread);
>> values.readOnly = PVE.Parser.parseBoolean(drive.ro);
>> @@ -317,6 +319,12 @@ Ext.define('PVE.qemu.HDInputPanel', {
>> reference: 'discard',
>> name: 'discard',
>> },
>> + {
>> + xtype: 'proxmoxcheckbox',
>> + fieldLabel: gettext('Detect zeroes'),
>> + reference: 'detect_zeroes',
>> + name: 'detect_zeroes',
>> + },
>> {
>> xtype: 'proxmoxcheckbox',
>> name: 'iothread',
>> --
>> 2.47.3
>>
>>
>>
>>
>>
>>
^ permalink raw reply [flat|nested] 5+ messages in thread
* superseded: [PATCH manager] ui: qemu: add detect zeroes disk option
2026-06-08 8:36 [PATCH manager] ui: qemu: add detect zeroes disk option Lukas Sichert
2026-06-16 13:01 ` Fabian Grünbichler
@ 2026-06-16 13:16 ` Lukas Sichert
1 sibling, 0 replies; 5+ messages in thread
From: Lukas Sichert @ 2026-06-16 13:16 UTC (permalink / raw)
To: Lukas Sichert, pve-devel
superseded-by v2:
https://lore.proxmox.com/all/20260610131827.11092-1-l.sichert@proxmox.com/
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH manager] ui: qemu: add detect zeroes disk option
2026-06-16 13:16 ` Lukas Sichert
@ 2026-06-17 12:12 ` Fabian Grünbichler
0 siblings, 0 replies; 5+ messages in thread
From: Fabian Grünbichler @ 2026-06-17 12:12 UTC (permalink / raw)
To: Lukas Sichert, pve-devel
On June 16, 2026 3:16 pm, Lukas Sichert wrote:
> On 2026-06-16 15:01, Fabian Grünbichler <f.gruenbichler@proxmox.com> wrote:
>
>> if I am not mistaken, this is a fix for
>> https://bugzilla.proxmox.com/show_bug.cgi?id=4503 ?
>
> Yes, I have also corrected this in a v2 already [1], but I forgot to send a
> superseded-by. Sorry for the confusion.
>
> [1] https://lore.proxmox.com/all/20260610131827.11092-1-l.sichert@proxmox.com/
sorry, missed that (and so did b4 it seems)!
>> On June 8, 2026 10:36 am, Lukas Sichert wrote:
>>> When discard is enabled, qemu-server defaults to `detect-zeroes=unmap`.
>>> This can have a significant performance impact for zero writes on some
>>> setups, while users might still want to keep guest discard/TRIM support
>>> enabled.
>>>
>>> Allow configuring the option independently so discard can stay enabled
>>> while zero detection is disabled. Persist the checkbox state explicitly,
>>> as the backend treats an absent value as the default-enabled behavior.
>>>
>>> Signed-off-by: Lukas Sichert <l.sichert@proxmox.com>
>>> ---
>>> www/manager6/qemu/HDEdit.js | 8 ++++++++
>>> 1 file changed, 8 insertions(+)
>>>
>>> diff --git a/www/manager6/qemu/HDEdit.js b/www/manager6/qemu/HDEdit.js
>>> index 1bb2bfda..d1d1b0fa 100644
>>> --- a/www/manager6/qemu/HDEdit.js
>>> +++ b/www/manager6/qemu/HDEdit.js
>>> @@ -96,6 +96,7 @@ Ext.define('PVE.qemu.HDInputPanel', {
>>> PVE.Utils.propertyStringSet(me.drive, values.readOnly, 'ro', 'on');
>>> PVE.Utils.propertyStringSet(me.drive, values.cache, 'cache');
>>> PVE.Utils.propertyStringSet(me.drive, values.aio, 'aio');
>>> + me.drive.detect_zeroes = values.detect_zeroes ? 1 : 0;
>>
>> this should probably have a comment why this line is deviating from the
>> ones above (to not lose an explicit false value that clashes with the
>> implicit true default value of the backend.
>
> Thanks for looking over this. I will fix it in a v3.
ack.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2026-06-17 12:12 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-06-08 8:36 [PATCH manager] ui: qemu: add detect zeroes disk option Lukas Sichert
2026-06-16 13:01 ` Fabian Grünbichler
2026-06-16 13:16 ` Lukas Sichert
2026-06-17 12:12 ` Fabian Grünbichler
2026-06-16 13:16 ` superseded: " Lukas Sichert
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox