From: "DERUMIER, Alexandre via pve-devel" <pve-devel@lists.proxmox.com>
To: "pve-devel@lists.proxmox.com" <pve-devel@lists.proxmox.com>,
"f.gruenbichler@proxmox.com" <f.gruenbichler@proxmox.com>
Cc: "DERUMIER, Alexandre" <alexandre.derumier@groupe-cyllene.com>
Subject: Re: [pve-devel] [PATCH qemu-server 18/31] blockdev: add change_medium() helper
Date: Tue, 1 Jul 2025 10:01:47 +0000 [thread overview]
Message-ID: <mailman.760.1751364141.395.pve-devel@lists.proxmox.com> (raw)
In-Reply-To: <856434301.10776.1751359098160@webmail.proxmox.com>
[-- Attachment #1: Type: message/rfc822, Size: 14731 bytes --]
From: "DERUMIER, Alexandre" <alexandre.derumier@groupe-cyllene.com>
To: "pve-devel@lists.proxmox.com" <pve-devel@lists.proxmox.com>, "f.gruenbichler@proxmox.com" <f.gruenbichler@proxmox.com>
Subject: Re: [pve-devel] [PATCH qemu-server 18/31] blockdev: add change_medium() helper
Date: Tue, 1 Jul 2025 10:01:47 +0000
Message-ID: <5b5e89fd824c431a57b8fcf9a5eb6cf30aeb24e3.camel@groupe-cyllene.com>
>>if you start a VM with cdrom=none, throttle group object is
>>generated:
sorry, I wanted to said : the throttle top node (which is using the
throttle group)
I think it's just related to blockdev-medium-change, as we can't
specify the node chain, qemu is simply autocreating a format+file node
with autogenerated node-name, and attach it to the top.
here the blockdev dump after going from none -> iso
ide2 => {
device => "",
inserted => {
active => $VAR1->{ide0}{inserted}{active},
backing_file_depth => 0,
bps => 0,
bps_rd => 0,
bps_wr => 0,
cache => {
direct => $VAR1->{ide0}{inserted}{cache}{direct},
"no-flush" => $VAR1->{ide0}{inserted}{cache}{direct},
writeback => $VAR1->{ide0}{inserted}{active},
},
detect_zeroes => "off",
drv => "raw",
encrypted => $VAR1->{ide0}{inserted}{cache}{direct},
file => "/var/lib/vz/template/iso/debian-12.5.0-amd64-
netinst.iso",
image => {
"actual-size" => 659554304,
"dirty-flag" => $VAR1->{ide0}{inserted}{cache}{direct},
filename => "/var/lib/vz/template/iso/isofile.iso",
format => "raw",
"virtual-size" => 659554304,
},
iops => 0,
iops_rd => 0,
iops_wr => 0,
"node-name" => "#block127",
ro => $VAR1->{ide0}{inserted}{active},
write_threshold => 0,
},
"io-status" => "ok",
locked => $VAR1->{ide0}{inserted}{active},
qdev => "ide2",
removable => $VAR1->{ide0}{inserted}{active},
tray_open => $VAR1->{ide0}{inserted}{cache}{direct},
type => "unknown",
},
>>@alexandre are you testing with all three patch series applied (qemu-
>>server, pve-qemu-kvm, pve-storage)?
yes sure !
[-- 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
next prev parent reply other threads:[~2025-07-01 10:01 UTC|newest]
Thread overview: 63+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-06-27 15:56 [pve-devel] [PATCH-SERIES qemu-server 00/31] let's switch to blockdev, blockdev, blockdev, part four (final) Fiona Ebner
2025-06-27 15:56 ` [pve-devel] [PATCH qemu-server 01/31] mirror: code style: avoid masking earlier declaration of $op Fiona Ebner
2025-06-27 15:56 ` [pve-devel] [PATCH qemu-server 02/31] test: collect mocked functions for QemuServer module Fiona Ebner
2025-06-27 15:56 ` [pve-devel] [PATCH qemu-server 03/31] drive: add helper to parse drive interface Fiona Ebner
2025-06-27 15:57 ` [pve-devel] [PATCH qemu-server 04/31] drive: drop invalid export of get_scsi_devicetype Fiona Ebner
2025-06-27 15:57 ` [pve-devel] [PATCH qemu-server 05/31] blockdev: add helpers for attaching and detaching block devices Fiona Ebner
2025-06-30 10:15 ` Fabian Grünbichler
2025-06-30 10:35 ` DERUMIER, Alexandre via pve-devel
[not found] ` <6575d8fe67659098d2bbd533c9063bcbd44c0a21.camel@groupe-cyllene.com>
2025-06-30 11:43 ` DERUMIER, Alexandre via pve-devel
2025-06-30 11:58 ` Fiona Ebner
2025-06-30 11:45 ` Fiona Ebner
2025-06-30 11:55 ` Fabian Grünbichler
2025-06-30 15:11 ` Fiona Ebner
2025-06-27 15:57 ` [pve-devel] [PATCH qemu-server 06/31] blockdev: add missing include for JSON module Fiona Ebner
2025-06-27 15:57 ` [pve-devel] [PATCH qemu-server 07/31] backup: use blockdev for fleecing images Fiona Ebner
2025-06-30 10:15 ` Fabian Grünbichler
2025-07-01 8:20 ` Fiona Ebner
2025-06-27 15:57 ` [pve-devel] [PATCH qemu-server 08/31] backup: use blockdev for TPM state file Fiona Ebner
2025-06-30 10:15 ` Fabian Grünbichler
2025-07-01 8:22 ` Fiona Ebner
2025-06-27 15:57 ` [pve-devel] [PATCH qemu-server 09/31] blockdev: introduce qdev_id_to_drive_id() helper Fiona Ebner
2025-06-27 15:57 ` [pve-devel] [PATCH qemu-server 10/31] blockdev: introduce and use get_block_info() helper Fiona Ebner
2025-06-27 15:57 ` [pve-devel] [PATCH qemu-server 11/31] blockdev: move helper for resize into module Fiona Ebner
2025-06-27 15:57 ` [pve-devel] [PATCH qemu-server 12/31] blockdev: add helper to get node below throttle node Fiona Ebner
2025-06-27 15:57 ` [pve-devel] [PATCH qemu-server 13/31] blockdev: resize: query and use node name for resize operation Fiona Ebner
2025-06-30 6:23 ` DERUMIER, Alexandre via pve-devel
2025-06-30 7:52 ` Fiona Ebner
2025-06-30 11:38 ` Fiona Ebner
2025-06-27 15:57 ` [pve-devel] [PATCH qemu-server 14/31] blockdev: support using zeroinit filter Fiona Ebner
2025-06-27 15:57 ` [pve-devel] [PATCH qemu-server 15/31] blockdev: make some functions private Fiona Ebner
2025-06-27 15:57 ` [pve-devel] [PATCH qemu-server 16/31] block job: allow specifying a block node that should be detached upon completion Fiona Ebner
2025-06-27 15:57 ` [pve-devel] [PATCH qemu-server 17/31] block job: add blockdev mirror Fiona Ebner
2025-06-30 10:15 ` Fabian Grünbichler
2025-07-01 9:21 ` Fiona Ebner
2025-06-27 15:57 ` [pve-devel] [PATCH qemu-server 18/31] blockdev: add change_medium() helper Fiona Ebner
2025-06-30 14:29 ` DERUMIER, Alexandre via pve-devel
[not found] ` <cd933fed020383019705045025d38c509042c267.camel@groupe-cyllene.com>
2025-06-30 14:42 ` DERUMIER, Alexandre via pve-devel
2025-07-01 7:30 ` DERUMIER, Alexandre via pve-devel
2025-07-01 8:38 ` Fabian Grünbichler
2025-07-01 10:01 ` DERUMIER, Alexandre via pve-devel [this message]
2025-07-01 8:42 ` Fiona Ebner
2025-07-01 10:05 ` Fiona Ebner
2025-07-01 10:20 ` DERUMIER, Alexandre via pve-devel
2025-07-01 10:25 ` Fiona Ebner
2025-07-01 11:51 ` DERUMIER, Alexandre via pve-devel
2025-06-27 15:57 ` [pve-devel] [PATCH qemu-server 19/31] blockdev: add blockdev_change_medium() helper Fiona Ebner
2025-06-27 15:57 ` [pve-devel] [PATCH qemu-server 20/31] blockdev: move helper for configuring throttle limits to module Fiona Ebner
2025-06-27 15:57 ` [pve-devel] [PATCH qemu-server 21/31] clone disk: skip check for aio=default (io_uring) compatibility starting with machine version 10.0 Fiona Ebner
2025-06-27 15:57 ` [pve-devel] [PATCH qemu-server 22/31] print drive device: don't reference any drive for 'none' " Fiona Ebner
2025-06-27 15:57 ` [pve-devel] [PATCH qemu-server 23/31] blockdev: add support for NBD paths Fiona Ebner
2025-06-27 15:57 ` [pve-devel] [PATCH qemu-server 24/31] blockdev: add helper to generate PBS block device for live restore Fiona Ebner
2025-06-27 15:57 ` [pve-devel] [PATCH qemu-server 25/31] blockdev: support alloc-track driver for live-{import, restore} Fiona Ebner
2025-06-27 15:57 ` [pve-devel] [PATCH qemu-server 26/31] live import: also record volid information Fiona Ebner
2025-06-27 15:57 ` [pve-devel] [PATCH qemu-server 27/31] live import/restore: query which node to use for operation Fiona Ebner
2025-06-27 15:57 ` [pve-devel] [PATCH qemu-server 28/31] live import/restore: use Blockdev::detach helper Fiona Ebner
2025-06-27 15:57 ` [pve-devel] [PATCH qemu-server 29/31] command line: switch to blockdev starting with machine version 10.0 Fiona Ebner
2025-06-30 10:15 ` Fabian Grünbichler
2025-06-30 10:57 ` Fiona Ebner
2025-06-27 15:57 ` [pve-devel] [PATCH qemu-server 30/31] test: migration: update running machine to 10.0 Fiona Ebner
2025-06-27 15:57 ` [pve-devel] [PATCH qemu-server 31/31] partially fix #3227: ensure that target image for mirror has the same size for EFI disks Fiona Ebner
2025-06-27 16:00 ` [pve-devel] [PATCH-SERIES qemu-server 00/31] let's switch to blockdev, blockdev, blockdev, part four (final) Fiona Ebner
2025-06-30 8:19 ` DERUMIER, Alexandre via pve-devel
2025-06-30 8:24 ` Fiona Ebner
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=mailman.760.1751364141.395.pve-devel@lists.proxmox.com \
--to=pve-devel@lists.proxmox.com \
--cc=alexandre.derumier@groupe-cyllene.com \
--cc=f.gruenbichler@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal