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

[Laurent Vivier]

Le vendredi 22 mai 2009 à 16:57 +0200, Alexander Graf a écrit :
> 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.
> 
> Signed-off-by: Alexander Graf <address@hidden>

Just a comment on the form: changelog should appear after the "---" with
diffstat to not be commited with the patch comment.
(they work like that on the linux kernel mailing list...)

> v2 changes:
> 
> - fix the segfault (yay!)
> - implement AIO
> 
> v3 changes:
> 
> - remove synchronous API
> - implement caching
> 
> v4 changes:
> 
> - enable other protocols (HTTPS, FTP, FTPS, TFTP, SFTP, SCP)
>   (I only tested FTP so far, but they _should_ work)
> 
> v5 changes:
> 
> - address Anthony's comments
> - deactivate ssh protocols again
> 
> v6 changes:
> 
> - s/^I/        /g
> - s/http/curl/g
> 
> v7 changes:
> 
> - add Signed-off-by again
> 
> ---
>  Makefile        |    6 +
>  Makefile.target |    2 +-
>  block-curl.c    |  540 
> +++++++++++++++++++++++++++++++++++++++++++++++++++++++
>  block.c         |   12 ++
>  block.h         |    7 +
>  configure       |   25 +++
>  6 files changed, 591 insertions(+), 1 deletions(-)
>  create mode 100644 block-curl.c
> 
> diff --git a/Makefile b/Makefile
> index 3c70068..421cd41 100644
> --- a/Makefile
> +++ b/Makefile
[...]