qemu-server/PVE/QemuServer/Machine.pm
Thomas Lamprecht cbe9de99c5 machine: improve code style in get_pve_version
This makes it a bit more obvious what happens and having an actual
error for bogus $PVE_MACHINE_VERSION entries.

Note that there was no auto-vivification before, as we never directly
accessed $PVE_MACHINE_VERSION->{$verstr}->{highest} but used
get_machine_pve_revisions to query a specific QEMU machine version's
PVE revisions and then operated on the return value, and that method
returns undef if there is no entry at all.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
2025-04-07 17:30:16 +02:00

365 lines
11 KiB
Perl

package PVE::QemuServer::Machine;
use strict;
use warnings;
use PVE::QemuServer::Helpers;
use PVE::QemuServer::MetaInfo;
use PVE::QemuServer::Monitor;
use PVE::JSONSchema qw(get_standard_option parse_property_string print_property_string);
# The PVE machine versions allow rolling out (incompatibel) changes to the hardware layout and/or
# the QEMU command of a VM without requiring a newer QEMU upstream machine version.
#
# To use this find the newest available QEMU machine version, add and entry in this hash if it does
# not already exists and then bump the higherst version, or introduce a new one starting at 1, as
# the upstream version is counted as having a PVE revision of 0.
# Additionally you must describe in short what the basic changes done with such a new PVE machine
# revision in the respective subhash, use the full version including the pve one as key there.
#
# NOTE: Do not overuse this. one or two changes per upstream machine can be fine, if needed. But
# most of the time it's better to batch more together and if there is no time pressure then wait a
# few weeks/months until the next QEMU machine revision is ready. As it will get confusing otherwise
# and we lazily use some simple ascii sort when processing these, so more than 10 per entry require
# changes but should be avoided in the first place.
# TODO: add basic test to ensure the keys are correct and there's a change entry for each version.
our $PVE_MACHINE_VERSION = {
'4.1' => {
highest => 2,
revisions => {
'+pve1' => 'Introduction of pveX versioning, no changes.',
'+pve2' => 'Increases the number of SCSI drives supported.',
},
},
'9.2' => {
highest => 1,
revisions => {
'+pve1' => 'Disables S3/S4 power states by default.',
},
},
};
my $machine_fmt = {
type => {
default_key => 1,
description => "Specifies the QEMU machine type.",
type => 'string',
pattern => '(pc|pc(-i440fx)?-\d+(\.\d+)+(\+pve\d+)?(\.pxe)?|q35|pc-q35-\d+(\.\d+)+(\+pve\d+)?(\.pxe)?|virt(?:-\d+(\.\d+)+)?(\+pve\d+)?)',
maxLength => 40,
format_description => 'machine type',
optional => 1,
},
viommu => {
type => 'string',
description => "Enable and set guest vIOMMU variant (Intel vIOMMU needs q35 to be set as"
." machine type).",
enum => ['intel', 'virtio'],
optional => 1,
},
'enable-s3' => {
type => 'boolean',
description => "Enables S3 power state. Defaults to false beginning with machine types 9.2+pve1, true before.",
optional => 1,
},
'enable-s4' => {
type => 'boolean',
description => "Enables S4 power state. Defaults to false beginning with machine types 9.2+pve1, true before.",
optional => 1,
},
};
PVE::JSONSchema::register_format('pve-qemu-machine-fmt', $machine_fmt);
PVE::JSONSchema::register_standard_option('pve-qemu-machine', {
description => "Specify the QEMU machine.",
type => 'string',
optional => 1,
format => PVE::JSONSchema::get_format('pve-qemu-machine-fmt'),
});
sub parse_machine {
my ($value) = @_;
return if !$value;
my $res = parse_property_string($machine_fmt, $value);
return $res;
}
sub print_machine {
my ($machine_conf) = @_;
return print_property_string($machine_conf, $machine_fmt);
}
my $default_machines = {
x86_64 => 'pc',
aarch64 => 'virt',
};
sub default_machine_for_arch {
my ($arch) = @_;
my $machine = $default_machines->{$arch} or die "unsupported architecture '$arch'\n";
return $machine;
}
sub assert_valid_machine_property {
my ($machine_conf) = @_;
my $q35 = $machine_conf->{type} && ($machine_conf->{type} =~ m/q35/) ? 1 : 0;
if ($machine_conf->{viommu} && $machine_conf->{viommu} eq "intel" && !$q35) {
die "to use Intel vIOMMU please set the machine type to q35\n";
}
}
sub machine_type_is_q35 {
my ($conf) = @_;
my $machine_conf = parse_machine($conf->{machine});
return $machine_conf->{type} && ($machine_conf->{type} =~ m/q35/) ? 1 : 0;
}
# In list context, also returns whether the current machine is deprecated or not.
sub current_from_query_machines {
my ($machines) = @_;
my ($current, $default);
for my $machine ($machines->@*) {
$default = $machine->{name} if $machine->{'is-default'};
if ($machine->{'is-current'}) {
$current = $machine->{name};
# pve-version only exists for the current machine
$current .= "+$machine->{'pve-version'}" if $machine->{'pve-version'};
return wantarray ? ($current, $machine->{deprecated} ? 1 : 0) : $current;
}
}
# fallback to the default machine if current is not supported by qemu - assume never deprecated
my $fallback = $default || 'pc';
return wantarray ? ($fallback, 0) : $fallback;
}
# This only works if VM is running.
# In list context, also returns whether the current machine is deprecated or not.
sub get_current_qemu_machine {
my ($vmid) = @_;
my $res = PVE::QemuServer::Monitor::mon_cmd($vmid, 'query-machines');
return current_from_query_machines($res);
}
# returns a string with major.minor+pve<VERSION>, patch version-part is ignored
# as it's seldom resembling a real QEMU machine type, so it would be '0' 99% of
# the time anyway.. This explicitly separates pveversion from the machine.
sub extract_version {
my ($machine_type, $kvmversion) = @_;
if (defined($machine_type) && $machine_type =~
m/^(?:pc(?:-i440fx|-q35)?|virt)-(\d+)\.(\d+)(?:\.(\d+))?(\+pve\d+)?(?:\.pxe)?/)
{
my $versionstr = "$1.$2";
$versionstr .= $4 if $4;
return $versionstr;
} elsif (defined($kvmversion)) {
if ($kvmversion =~ m/^(\d+)\.(\d+)/) {
my $pvever = get_pve_version($kvmversion);
return "$1.$2+pve$pvever";
}
}
return;
}
sub is_machine_version_at_least {
my ($machine_type, $major, $minor, $pve) = @_;
return PVE::QemuServer::Helpers::min_version(
extract_version($machine_type), $major, $minor, $pve);
}
sub get_machine_pve_revisions {
my ($machine_version_str) = @_;
if ($machine_version_str =~ m/^(\d+\.\d+)/) {
return $PVE_MACHINE_VERSION->{$1};
}
die "internal error: cannot get pve version for invalid string '$machine_version_str'";
}
sub get_pve_version {
my ($verstr) = @_;
if (my $pve_machine = get_machine_pve_revisions($verstr)) {
return $pve_machine->{highest} || die "internal error - machine version '$verstr' missing 'highest'";
}
return 0;
}
sub can_run_pve_machine_version {
my ($machine_version, $kvmversion) = @_;
$machine_version =~ m/^(\d+)\.(\d+)(?:\+pve(\d+))?(?:\.pxe)?$/;
my $major = $1;
my $minor = $2;
my $pvever = $3;
$kvmversion =~ m/(\d+)\.(\d+)/;
return 0 if PVE::QemuServer::Helpers::version_cmp($1, $major, $2, $minor) < 0;
# if $pvever is missing or 0, we definitely support it as long as we didn't
# fail the QEMU version check above
return 1 if !$pvever;
my $max_supported = get_pve_version("$major.$minor");
return 1 if $max_supported >= $pvever;
return 0;
}
sub qemu_machine_pxe {
my ($vmid, $conf) = @_;
my $machine = get_current_qemu_machine($vmid);
my $machine_conf = parse_machine($conf->{machine});
if ($machine_conf->{type} && $machine_conf->{type} =~ m/\.pxe$/) {
$machine .= '.pxe';
}
return $machine;
}
sub get_installed_machine_version {
my ($kvmversion) = @_;
$kvmversion = PVE::QemuServer::Helpers::kvm_user_version() if !defined($kvmversion);
$kvmversion =~ m/^(\d+\.\d+)/;
return $1;
}
sub windows_get_pinned_machine_version {
my ($machine, $base_version, $kvmversion) = @_;
my $pin_version = $base_version;
if (!defined($base_version) || !can_run_pve_machine_version($base_version, $kvmversion)) {
$pin_version = get_installed_machine_version($kvmversion);
# pin to the current pveX version to make use of most current features if > 0
my $pvever = get_pve_version($pin_version);
if ($pvever > 0) {
$pin_version .= "+pve$pvever";
}
}
if (!$machine || $machine eq 'pc') {
$machine = "pc-i440fx-$pin_version";
} elsif ($machine eq 'q35') {
$machine = "pc-q35-$pin_version";
} elsif ($machine eq 'virt') {
$machine = "virt-$pin_version";
} else {
warn "unknown machine type '$machine', not touching that!\n";
}
return $machine;
}
sub get_vm_machine {
my ($conf, $forcemachine, $arch) = @_;
my $machine_conf = parse_machine($conf->{machine});
my $machine = $forcemachine || $machine_conf->{type};
if (!$machine || $machine =~ m/^(?:pc|q35|virt)$/) {
my $kvmversion = PVE::QemuServer::Helpers::kvm_user_version();
# we must pin Windows VMs without a specific version and no meta info about creation QEMU to
# 5.1, as 5.2 fixed a bug in ACPI layout which confuses windows quite a bit and may result
# in various regressions..
# see: https://lists.gnu.org/archive/html/qemu-devel/2021-02/msg08484.html
# Starting from QEMU 9.1, pin to the creation version instead. Support for 5.1 is expected
# to drop with QEMU 11.1 and it would still be good to handle Windows VMs that do not have
# an explicit machine version for whatever reason.
if (PVE::QemuServer::Helpers::windows_version($conf->{ostype})) {
my $base_version = '5.1';
# TODO PVE 10 - die early if there is a Windows VM both without explicit machine version
# and without meta info.
if (my $meta = PVE::QemuServer::MetaInfo::parse_meta_info($conf->{meta})) {
if (PVE::QemuServer::Helpers::min_version($meta->{'creation-qemu'}, 9, 1)) {
# need only major.minor
($base_version) = ($meta->{'creation-qemu'} =~ m/^(\d+.\d+)/);
}
}
$machine = windows_get_pinned_machine_version($machine, $base_version, $kvmversion);
} else {
$arch //= 'x86_64';
$machine ||= default_machine_for_arch($arch);
my $pvever = get_pve_version($kvmversion);
$machine .= "+pve$pvever";
}
}
if ($machine !~ m/\+pve\d+?(?:\.pxe)?$/) {
my $is_pxe = $machine =~ m/^(.*?)\.pxe$/;
$machine = $1 if $is_pxe;
# for version-pinned machines that do not include a pve-version (e.g.
# pc-q35-4.1), we assume 0 to keep them stable in case we bump
$machine .= '+pve0';
$machine .= '.pxe' if $is_pxe;
}
return $machine;
}
sub check_and_pin_machine_string {
my ($machine_string, $ostype) = @_;
my $machine_conf = parse_machine($machine_string);
my $machine = $machine_conf->{type};
if (!$machine || $machine =~ m/^(?:pc|q35|virt)$/) {
# always pin Windows' machine version on create, they get confused too easily
if (PVE::QemuServer::Helpers::windows_version($ostype)) {
$machine_conf->{type} = windows_get_pinned_machine_version($machine);
print "pinning machine type to '$machine_conf->{type}' for Windows guest OS\n";
}
}
assert_valid_machine_property($machine_conf);
return print_machine($machine_conf);
}
# disable s3/s4 by default for 9.2+pve1 machine types
# returns an arrayref of cmdline options for qemu or undef
sub get_power_state_flags {
my ($machine_conf, $version_guard) = @_;
my $object = $machine_conf->{type} && ($machine_conf->{type} =~ m/q35/) ? "ICH9-LPC" : "PIIX4_PM";
my $default = 1;
if ($version_guard->(9, 2, 1)) {
$default = 0;
}
my $s3 = $machine_conf->{'enable-s3'} // $default;
my $s4 = $machine_conf->{'enable-s4'} // $default;
my $options = [];
# they're enabled by default in QEMU, so only add the flags to disable them
if (!$s3) {
push $options->@*, '-global', "${object}.disable_s3=1";
}
if (!$s4) {
push $options->@*, '-global', "${object}.disable_s4=1";
}
if (scalar($options->@*)) {
return $options;
}
return;
}
1;