提交 9a8f39d0 编写于 作者: C Christophe Fergeau

storage: Ensure 'qemu-img resize' size arg is a 512 multiple

qemu-img resize will fail with "The new size must be a multiple of 512"
if libvirt doesn't round it first.
This fixes rhbz#951495
Signed-off-by: NChristophe Fergeau <cfergeau@redhat.com>
上级 a7c4202c
...@@ -339,6 +339,9 @@ ...@@ -339,6 +339,9 @@
/* divide value by size, rounding up */ /* divide value by size, rounding up */
# define VIR_DIV_UP(value, size) (((value) + (size) - 1) / (size)) # define VIR_DIV_UP(value, size) (((value) + (size) - 1) / (size))
/* round up value to the closest multiple of size */
# define VIR_ROUND_UP(value, size) (VIR_DIV_UP(value, size) * (size))
# if WITH_DTRACE_PROBES # if WITH_DTRACE_PROBES
# ifndef LIBVIRT_PROBES_H # ifndef LIBVIRT_PROBES_H
......
...@@ -14131,7 +14131,8 @@ error: ...@@ -14131,7 +14131,8 @@ error:
* Changes the capacity of the storage volume @vol to @capacity. The * Changes the capacity of the storage volume @vol to @capacity. The
* operation will fail if the new capacity requires allocation that would * operation will fail if the new capacity requires allocation that would
* exceed the remaining free space in the parent pool. The contents of * exceed the remaining free space in the parent pool. The contents of
* the new capacity will appear as all zero bytes. * the new capacity will appear as all zero bytes. The capacity value will
* be rounded to the granularity supported by the hypervisor.
* *
* Normally, the operation will attempt to affect capacity with a minimum * Normally, the operation will attempt to affect capacity with a minimum
* impact on allocation (that is, the default operation favors a sparse * impact on allocation (that is, the default operation favors a sparse
......
...@@ -1219,6 +1219,10 @@ virStorageBackendFilesystemResizeQemuImg(const char *path, ...@@ -1219,6 +1219,10 @@ virStorageBackendFilesystemResizeQemuImg(const char *path,
return -1; return -1;
} }
/* Round capacity as qemu-img resize errors out on sizes which are not
* a multiple of 512 */
capacity = VIR_ROUND_UP(capacity, 512);
cmd = virCommandNew(img_tool); cmd = virCommandNew(img_tool);
virCommandAddArgList(cmd, "resize", path, NULL); virCommandAddArgList(cmd, "resize", path, NULL);
virCommandAddArgFormat(cmd, "%llu", capacity); virCommandAddArgFormat(cmd, "%llu", capacity);
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册