From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:52355) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UkXxI-0008R2-P3 for qemu-devel@nongnu.org; Thu, 06 Jun 2013 07:02:10 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UkXxD-0005kR-Mp for qemu-devel@nongnu.org; Thu, 06 Jun 2013 07:02:08 -0400 Received: from mx1.redhat.com ([209.132.183.28]:51436) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UkXxD-0005j9-FK for qemu-devel@nongnu.org; Thu, 06 Jun 2013 07:02:03 -0400 Received: from int-mx09.intmail.prod.int.phx2.redhat.com (int-mx09.intmail.prod.int.phx2.redhat.com [10.5.11.22]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id r56B21Fh015802 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Thu, 6 Jun 2013 07:02:01 -0400 Date: Thu, 6 Jun 2013 12:01:57 +0100 From: "Richard W.M. Jones" Message-ID: <20130606110157.GM4515@redhat.com> References: <1370499959-8916-1-git-send-email-famz@redhat.com> MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="nrM5Z5VIJgwP9LWp" Content-Disposition: inline In-Reply-To: <1370499959-8916-1-git-send-email-famz@redhat.com> Subject: Re: [Qemu-devel] [PATCH v7 00/13] curl: fix curl read List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Fam Zheng Cc: kwolf@redhat.com, jcody@redhat.com, qemu-devel@nongnu.org, stefanha@redhat.com --nrM5Z5VIJgwP9LWp Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable 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. --=20 Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjon= es 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 --nrM5Z5VIJgwP9LWp Content-Type: application/x-sh Content-Disposition: attachment; filename="test.sh" Content-Transfer-Encoding: quoted-printable #!/bin/bash -=0A=0A# Enable core dumps.=0Aulimit -c unlimited=0A=0A# Locati= on of self-compiled Curl (from git):=0Aexport LD_LIBRARY_PATH=3D~/d/curl/li= b/.libs=0A=0A# Location of self-compiled qemu (from git):=0Aexport LIBGUEST= FS_QEMU=3D~/d/qemu/qemu.wrapper=0Aexport PATH=3D~/d/qemu:$PATH=0A=0A# Set u= p libguestfs to run qemu directly instead of using libvirt:=0Aexport LIBGUE= STFS_BACKEND=3Ddirect=0A=0A# Enable libguestfs debugging (optional):=0Aexpo= rt LIBGUESTFS_DEBUG=3D1=0Aexport LIBGUESTFS_TRACE=3D1=0A=0A# Location of re= mote disk image over http:=0A#disk=3D"http://192.168.0.249/scratch/plan9.is= o"=0A#disk=3D"http://mirrorservice.org/sites/dl.fedoraproject.org/pub/fedor= a/linux/releases/18/Fedora/x86_64/iso/Fedora-18-x86_64-DVD.iso"=0Adisk=3D"h= ttp://libguestfs.org/plan9.iso"=0Aformat=3D"raw"=0A=0A# For ISOs:=0Aargs=3D= "-m /dev/sda"=0A# For regular OS disks:=0A#args=3D"-i"=0A=0A# Location of l= ibguestfs directory (compiled from git):=0Acd ~/d/libguestfs=0A=0A# Run gue= stfish script.=0A./run ./fish/guestfish --ro --format=3D"$format" -a "$disk= " $args <