Re: [Qemu-devel] [PATCH v7 00/13] curl: fix curl read

["Richard W.M. Jones" <rjones@redhat.com>]

[-- Attachment #1: Type: text/plain, Size: 1141 bytes --]

On Thu, Jun 06, 2013 at 02:25:46PM +0800, Fam Zheng wrote:
> v7:
>   13: Added:
>       curl: change timeout to 30 seconds

I tested this against:

(1) HTTP to Apache server over slow but local wifi.

(2) HTTP to a remote ISO (on another continent).

Test (1) is fine.

Test (2) gives plenty of I/O errors.  I guess that 30 seconds isn't
sufficient here.

I should note that current upstream qemu *works* in both cases.
Although the timeout in current qemu is much shorter (5 seconds), for
some reason this does not affect the test.

I'm also confused about what problem this patch series is trying to
fix, since upstream qemu works fine for me with the latest curl.

The test script I'm using is attached.

I'm using: qemu v1.4.0-2150-g8819c10 with or without the v7 patch
series, curl curl-7_30_0-173-g5657c56 and libguestfs 1.23.2-15-g9585077

Rich.

-- 
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
virt-p2v converts physical machines to virtual machines.  Boot with a
live CD or over the network (PXE) and turn machines into KVM guests.
http://libguestfs.org/virt-v2v

[-- Attachment #2: test.sh --]
[-- Type: application/x-sh, Size: 1020 bytes --]