From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:37632) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1W3Y9R-00074x-Qw for qemu-devel@nongnu.org; Wed, 15 Jan 2014 16:37:34 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1W3Y9N-0006u4-47 for qemu-devel@nongnu.org; Wed, 15 Jan 2014 16:37:29 -0500 Date: Wed, 15 Jan 2014 21:37:19 +0000 From: "Richard W.M. Jones" Message-ID: <20140115213719.GC13802@redhat.com> References: <1389806638-3114-1-git-send-email-peter.maydell@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1389806638-3114-1-git-send-email-peter.maydell@linaro.org> Subject: Re: [Qemu-devel] [PATCH] block/curl: Implement the libcurl timer callback interface List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell Cc: Kevin Wolf , patches@linaro.org, qemu-devel@nongnu.org, Stefan Hajnoczi , qemu-stable@nongnu.org On Wed, Jan 15, 2014 at 05:23:58PM +0000, Peter Maydell wrote: > libcurl versions 7.16.0 and later have a timer callback interface which > must be implemented in order for libcurl to make forward progress (it > will sometimes rely on being called back on the timeout if there are > no file descriptors registered). Implement the callback, and use a > QEMU AIO timer to ensure we prod libcurl again when it asks us to. Somewhat off topic, but does the curl driver work for you at all? Especially if you have a disk image on a remote server, but one with relatively low latency (eg. on a LAN but not localhost). We have had endless problems with it, including upstream discussions with curl people, summarised in this bug: https://bugzilla.redhat.com/show_bug.cgi?id=971790 Just now I tried the latest qemu from git with and without your patch and still cannot make it work, although the bugginess is now different from RHBZ#971790. I opened a new bug about what I'm seeing today (https://bugs.launchpad.net/qemu/+bug/1269606). Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones Read my programming blog: http://rwmj.wordpress.com Fedora now supports 80 OCaml packages (the OPEN alternative to F#)