[pve-devel] [PATCH qemu-server v8 1/2] add new public get_default_vga_type function
Aaron Lauterer
a.lauterer at proxmox.com
Fri Oct 3 17:00:42 CEST 2025
we resolve missing parameters if necessary to make it easier to call
from another module where we likely only have the VM config ready.
Signed-off-by: Aaron Lauterer <a.lauterer at proxmox.com>
Reviewed-by: Fiona Ebner <f.ebner at proxmox.com>
Tested-by: Hannes Duerr <h.duerr at proxmox.com>
Reviewed-by: Michael Köppl <m.koeppl at proxmox.com>
Tested-by: Michael Köppl <m.koeppl at proxmox.com>
---
changes since
v7:
* rebased
v6:
* instead of making get_vga_properties and extract_version public, we
create the new get_default_vga_type function and only make it public.
src/PVE/QemuServer.pm | 29 ++++++++++++++++++++++-------
1 file changed, 22 insertions(+), 7 deletions(-)
diff --git a/src/PVE/QemuServer.pm b/src/PVE/QemuServer.pm
index 9f35de2e..adcdac5b 100644
--- a/src/PVE/QemuServer.pm
+++ b/src/PVE/QemuServer.pm
@@ -3044,6 +3044,27 @@ my sub should_disable_smm {
&& $vga->{type} =~ m/^(serial\d+|none)$/;
}
+sub get_default_vga_type {
+ my ($conf, $arch, $machine_version, $winversion) = @_;
+
+ $arch //= PVE::QemuServer::Helpers::get_vm_arch($conf);
+ $winversion //= PVE::QemuServer::Helpers::windows_version($conf->{ostype});
+ if (!$machine_version) {
+ my $kvm_binary = PVE::QemuServer::Helpers::get_command_for_arch($arch);
+ my $kvmver = kvm_user_version($kvm_binary);
+ my $machine_type = PVE::QemuServer::Machine::get_vm_machine($conf, undef, $arch);
+ $machine_version = extract_version($machine_type, $kvmver);
+ }
+
+ if ($arch eq 'aarch64') {
+ return 'virtio';
+ } elsif (min_version($machine_version, 2, 9)) {
+ return (!$winversion || $winversion >= 6) ? 'std' : 'cirrus';
+ } else {
+ return ($winversion >= 6) ? 'std' : 'cirrus';
+ }
+}
+
my sub get_vga_properties {
my ($conf, $arch, $machine_version, $winversion) = @_;
@@ -3053,13 +3074,7 @@ my sub get_vga_properties {
$vga->{type} = 'qxl' if $qxlnum;
if (!$vga->{type}) {
- if ($arch eq 'aarch64') {
- $vga->{type} = 'virtio';
- } elsif (min_version($machine_version, 2, 9)) {
- $vga->{type} = (!$winversion || $winversion >= 6) ? 'std' : 'cirrus';
- } else {
- $vga->{type} = ($winversion >= 6) ? 'std' : 'cirrus';
- }
+ $vga->{type} = get_default_vga_type($conf, $arch, $machine_version, $winversion);
}
return ($vga, $qxlnum);
--
2.47.3
More information about the pve-devel
mailing list