yesBad/pve-storage
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

plugin: rbd: pass rxbounce when mapping Windows VM guest volumes

Browse Source 
When mapping a volume (e.g., because KRBD is enabled) and the hint
'guest-is-windows' is given and true, pass the rxbounce option. This
is to avoid "bad crc/signature" warnings in the journal, retransmits
and degraded performance, see [1]. If the volume is already mapped
without rxbounce (this can be determined from the map options exposed
in sysfs), and it should be mapped with rxbounce, and the
'plugin-may-deactivate-volume' hint denotes it is currently safe to
deactivate the volume, unmap the volume and re-map it with rxbounce.

If 'guest-is-windows' is not given or not true, and the volume is
already mapped, take no action. This also means that guest volumes
that are mapped with rxbounce, but do not have to be (because they do
not belong to a Windows guest), are not deactivated. This can be the
case if a user applied the workaround of adding rxbounce to
'rbd_default_map_options', since this applies to all volumes.

[1] https://bugzilla.proxmox.com/show_bug.cgi?id=5779
[2] https://forum.proxmox.com/threads/155741/post-710845

Signed-off-by: Friedrich Weber <f.weber@proxmox.com>
Link: https://lore.proxmox.com/20251031103709.60233-6-f.weber@proxmox.com
This commit is contained in:
Friedrich Weber
2025-10-31 11:36:12 +01:00
committed by Thomas Lamprecht
parent 738897852c
commit 24367c07d3
1 changed files with 38 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

42
src/PVE/Storage/RBDPlugin.pm
View File

@ -90,6 +90,19 @@ my sub get_rbd_dev_path {
return $pve_path; return $pve_path;
} }
my sub read_rbd_map_options {
my ($rbd_id) = @_;
my $sysfs_config_info = file_read_firstline("/sys/devices/rbd/${rbd_id}/config_info");
return if !defined($sysfs_config_info);
my $config_info = [split(/\s+/, $sysfs_config_info)];
my $options = @$config_info[1];
return if !defined($options);
return [split(/,/, $options)];
}
my $rbd_cmd = sub { my $rbd_cmd = sub {
my ($scfg, $storeid, $op, @options) = @_; my ($scfg, $storeid, $op, @options) = @_;
@ -799,7 +812,7 @@ sub deactivate_storage {
} }
sub map_volume { sub map_volume {
my ($class, $storeid, $scfg, $volname, $snapname) = @_; my ($class, $storeid, $scfg, $volname, $snapname, $hints) = @_;
my ($vtype, $img_name, $vmid) = $class->parse_volname($volname); my ($vtype, $img_name, $vmid) = $class->parse_volname($volname);
@ -808,12 +821,33 @@ sub map_volume {
my $kerneldev = get_rbd_dev_path($scfg, $storeid, $name); my $kerneldev = get_rbd_dev_path($scfg, $storeid, $name);
my @extra_options = ();
if (defined($hints) && $hints->{'guest-is-windows'}) {
# make sure to pass rxbounce for Windows guest volumes to avoid degraded performance
@extra_options = ('--options', 'rxbounce');
# if already mapped without rxbounce and deactivation is safe, try to unmap
if (-b $kerneldev) {
my $mapped_options = read_rbd_map_options(get_rbd_id($kerneldev));
if ($mapped_options && scalar(grep { /rxbounce/ } @$mapped_options) == 0) {
if ($hints->{'plugin-may-deactivate-volume'}) {
eval { $class->unmap_volume($storeid, $scfg, $volname, $snapname); };
warn "could not unmap to apply rxbounce - $@\n" if $@;
} else {
warn "not unmapping volume $volname to apply rxbounce since it is not safe\n";
}
}
}
}
return $kerneldev if -b $kerneldev; # already mapped return $kerneldev if -b $kerneldev; # already mapped
# features can only be enabled/disabled for image, not for snapshot! # features can only be enabled/disabled for image, not for snapshot!
$krbd_feature_update->($scfg, $storeid, $img_name); $krbd_feature_update->($scfg, $storeid, $img_name);
my $cmd = $rbd_cmd->($scfg, $storeid, 'map', $name); my $cmd = $rbd_cmd->($scfg, $storeid, 'map', $name, @extra_options);
run_rbd_command($cmd, errmsg => "can't map rbd volume $name"); run_rbd_command($cmd, errmsg => "can't map rbd volume $name");
return $kerneldev; return $kerneldev;
@ -836,9 +870,9 @@ sub unmap_volume {
} }
sub activate_volume { sub activate_volume {
my ($class, $storeid, $scfg, $volname, $snapname, $cache) = @_; my ($class, $storeid, $scfg, $volname, $snapname, $cache, $hints) = @_;
$class->map_volume($storeid, $scfg, $volname, $snapname) if $scfg->{krbd}; $class->map_volume($storeid, $scfg, $volname, $snapname, $hints) if $scfg->{krbd};
return 1; return 1;
} }