[Qemu-devel] Re: [PATCH] Add HTTP protocol using curl

[Alexander Graf]

On 09.04.2009, at 23:37, Anthony Liguori wrote:

> Alexander Graf wrote:
> > Currently Qemu can read from posix I/O and NBD. This patch adds a
> > third protocol to the game: HTTP.
> >
> > In certain situations it can be useful to access HTTP data directly,
> > for example if you want to try out an http provided OS image, but
> > don't know if you want to download it yet.
> >
> > Using this patch you can now try it on on the fly. Just use it like:
> >
> > qemu -cdrom http://host/path/my.iso
> >
> > In order to not reinvent the wheel, this patch uses libcurl.
>
> I have one of these too that reinvents the wheel.  The main reason  
> was to enable bdrv_aio_* implementations.
>
> I'm happy to merge a version that uses libcurl but I don't want to  
> merge it unless it's implementing bdrv_aio_.  As far as I'm  
> concerned, backends implementing bdrv_read/bdrv_write is deprecated.

I agree. Libcurl provides means to download asynchronously as well, so  
AIO is possible with it. I didn't implement it because I needed  
something quick before I leave for 3 weeks of vacation (today ;-)).

So in case you don't merge it, that's fine. I wanted to improve it  
next month anyways.

Alex