External/libvirt
1
0
Fork 0
mirror of https://gitlab.com/libvirt/libvirt.git synced 2025-02-21 19:02:25 +00:00
Code Issues Packages Projects Releases Wiki Activity

qemu: tighten semantics of 'size' when resizing block devices

Browse Source 
When VIR_DOMAIN_BLOCK_RESIZE_CAPACITY is set, the 'size' parameter
is currently ignored. Since applications must none the less pass a
value for this parameter, it is preferrable to declare some explicit
semantics for it.

This declare that the parameter must be 0, or the exact size of the
underlying block device. The latter gives the management application
the ability to sanity check that the block device size matches what
they think it should be.

Reviewed-by: Peter Krempa <pkrempa@redhat.com>
Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
This commit is contained in:
Daniel P. Berrangé 2024-01-03 16:36:45 +00:00
parent 784d90fe68
commit 024d6dc263
2 changed files with 11 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/libvirt-domain.c
View File

@ -6389,8 +6389,9 @@ virDomainBlockPeek(virDomainPtr dom,
* to the next alignment boundary.
*
* If @flag contains VIR_DOMAIN_BLOCK_RESIZE_CAPACITY (since 10.0.0) the
* hypervisor will resize the guest block device to fully fill the source,
* ignoring @size. This is possible only for image formats with no metadata
* hypervisor will resize the guest block device to fully fill the source.
* @size must be either set to zero, or to the exact size of the block
* device source. This is possible only for image formats with no metadata
* ('raw') and for source devices with limited capacity such as block devices.
*
* The @disk parameter is either an unambiguous source name of the

9
src/qemu/qemu_driver.c
View File

@ -9324,7 +9324,14 @@ qemuDomainBlockResize(virDomainPtr dom,
goto endjob;
}
size = disk->src->physical;
if (size == 0) {
size = disk->src->physical;
} else if (size != disk->src->physical) {
virReportError(VIR_ERR_INVALID_ARG,
_("Requested resize to '%1$llu' but device size is '%2$llu'"),
size, disk->src->physical);
goto endjob;
}
}
/* qcow2 and qed must be sized on 512 byte blocks/sectors,