blob: 8875120285226a91f35202968aaa2c269daf95f4 (
plain) (
blame)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
|
mistranslation: 0.921
instruction: 0.811
semantic: 0.777
graphic: 0.752
other: 0.735
device: 0.591
socket: 0.378
assembly: 0.309
vnc: 0.274
network: 0.265
boot: 0.190
KVM: 0.064
wrong error for blockdev-snapshot-sync
From Laszlo Ersek:
>> + proto_drv = bdrv_find_protocol(snapshot_file);
>> if (!proto_drv) {
>> - qerror_report(QERR_INVALID_BLOCK_FORMAT, format);
>> - ret = -1;
>> - goto out;
>> + error_set(errp, QERR_INVALID_BLOCK_FORMAT, format);
>> + return;
>> }
>
> I don't understand the logic here (based on the error message). We
> specified "format" for the case when a completely new snapshot file has
> to be created. If the file exists already, then bdrv_find_protocol()
> tries to find the driver for it. If that fails, then we must report an
> error indeed, but instead of referring to "format", we'd have to report
> the "scheme" from the beginning of "snapshot_file".
Which version of QEMU was this? Is this still a problem with the latest version of QEMU?
I can't find anything in the blockdev-snapshot-sync path that has this code in it still. Think it's a non-issue in 2017.
[Expired for QEMU because there has been no activity for 60 days.]
|