[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-stable] [Qemu-devel] [PATCH] block/curl: Implement the libcurl
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-stable] [Qemu-devel] [PATCH] block/curl: Implement the libcurl timer callback interface |
Date: |
Thu, 16 Jan 2014 10:52:11 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Am 16.01.2014 um 10:24 hat Richard W.M. Jones geschrieben:
>
> On further investigation, the "No such file or directory" error occurs
> when using snapshot=on.
>
> ie:
>
> This fails:
>
> ./x86_64-softmmu/qemu-system-x86_64 -drive
> 'file=http://127.0.0.1/~rjones/cirros-0.3.1-x86_64-disk.img,if=virtio,snapshot=on'
>
> This works:
>
> ./x86_64-softmmu/qemu-system-x86_64 -drive
> 'file=http://127.0.0.1/~rjones/cirros-0.3.1-x86_64-disk.img,if=virtio,readonly=on'
Whoops, sorry, I think it was me who broke this when I rewrote
snapshot=on support to fit better in the whole blockdev-add
infrastructure (commit 9fd3171a).
The reason is that instead of writing the file name into the backing
file header field of the temporary image, it now uses the file.filename
runtime option, which doesn't parse protocol names from the filename.
I believe Max was going to look into some related issues (basically how
to get rid of the "magic" filenames in the core code without breaking
compatibility), perhaps a fix comes out naturally from such changes.
Workaround for now: Add an explicit file.driver=http
Kevin