[pve-devel] applied: [PATCH v2 qemu 4/4] vma: create: register all streams before entering coroutines

Thomas Lamprecht t.lamprecht at proxmox.com
Mon Feb 14 15:51:57 CET 2022


On 14.02.22 12:02, Fabian Ebner wrote:
> Otherwise, the header might already get written by a coroutine and
> registering further streams will fail after that.
> 
> Also adds a missing g_list_free call for the other GList that's used.

just as note:
it's cleaner this way, especially if we'd reuse `create_archive` in the future,
but currently this doesn't changes much, as it was the program exited immediately
afterwards, so returning all resources back to the OS, anyway.

> 
> Reported in the community forum:
> https://forum.proxmox.com/threads/104744/
> 
> Reproducer script (increase beyond 30 if the issue isn't triggered yet):
>> #!/usr/bin/perl
>>
>> my $dir = "./vma-create-bug";
>> mkdir $dir;
>>
>> my $archive_path = "$dir/vzdump-qemu-104-2202_02_02-00_00_00.vma";
>> unlink $archive_path;
>>
>> my $cmd = "vma create $archive_path -v";
>> for (my $i = 0; $i < 30; $i++) {
>>   system("truncate -s 1M $dir/drive-virtio$i.img");
>>   $cmd .= " drive-virtio$i=$dir/drive-virtio$i.img";
>> }
>> system($cmd);
> 
> Signed-off-by: Fabian Ebner <f.ebner at proxmox.com>
> ---
> 
> A late addition to v2.

Also later unrelated, so I applied it now already, avoiding the need to release
6.2 to repos already for the forum user to test this.

> 
>  ...VE-Backup-add-vma-backup-format-code.patch | 24 +++++++++++++++----
>  1 file changed, 19 insertions(+), 5 deletions(-)
> 
>

applied, thanks!





More information about the pve-devel mailing list