Am 01.03.21 um 11:18 schrieb Dietmar Maurer:
diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm index f401baf..e579cdf 100644 --- a/PVE/QemuServer.pm +++ b/PVE/QemuServer.pm @@ -6991,7 +6991,15 @@ sub clone_disk { # that is given by the OVMF_VARS.fd my $src_path = PVE::Storage::path($storecfg, $drive->{file}); my $dst_path = PVE::Storage::path($storecfg, $newvolid); - run_command(['qemu-img', 'dd', '-n', '-O', $dst_format, "bs=1", "count=$size", + + # Ceph doesn't like too small blocksize, see bug #3324 + my $bs = 1; + while ($bs < $size && $bs < 1024 && $size % $bs == 0) { + $bs *= 2; + }now, $size % $bs != 0 I guess this is wrong...
No, because of the $bs < $size check first. If we enter the loop, $size was a multiple of $bs and at least twice $bs. I wrote it like this because it'd avoid an endless loop if for whatever reason $size is 0.
+ my $count = $size / $bs; + + run_command(['qemu-img', 'dd', '-n', '-O', $dst_format, "bs=$bs", "count=$count", "if=$src_path", "of=$dst_path"]); } else { qemu_img_convert($drive->{file}, $newvolid, $size, $snapname, $sparseinit);_______________________________________________ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
_______________________________________________ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
