[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v6 00/12] curl: fix curl read

From: Richard W.M. Jones
Subject: Re: [Qemu-devel] [PATCH v6 00/12] curl: fix curl read
Date: Tue, 28 May 2013 12:25:39 +0100
User-agent: Mutt/1.5.20 (2009-12-10)

[I chopped the CC list down]

On Tue, May 28, 2013 at 07:14:01PM +0800, Fam Zheng wrote:
> There seems no easy way to me to inject sleep to io reqs in curl.c.
> Another option might be configure & compile qemu with
> CFLAGS=-DDEBUG_CURL and grab the stdout, so that we can see how requests
> are processed.

No problems.

I started with v6 patch.  I did:

$ CFLAGS=-DDEBUG_CURL ./configure --enable-werror --target-list="i386-softmmu 
$ make

Note there is a bug!

block/curl.c: In function ‘curl_complete_io’:
block/curl.c:211:5: error: too many arguments for format 
cc1: all warnings being treated as errors
make: *** [block/curl.o] Error 1

Please add the attached patch to your patch set.

After fixing that and continuing the build, I then ran:

$ /tmp/test.sh >& /tmp/curl-debug.log

This shows the error and produces an absolutely vast amount of
debugging (attached).  Note that I killed it after a while but there
should be enough to show the error.


Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Fedora Windows cross-compiler. Compile Windows programs, test, and
build Windows installers. Over 100 libraries supported.

Attachment: 0001-curl-Fix-format-string-in-debugging-statement.patch
Description: Text document

Attachment: curl-debug.log.xz
Description: application/xz

reply via email to

[Prev in Thread] Current Thread [Next in Thread]