From: Fam Zheng <famz@redhat.com>
To: "Richard W.M. Jones" <rjones@redhat.com>
Cc: kwolf@redhat.com, jcody@redhat.com, qemu-devel@nongnu.org,
stefanha@redhat.com
Subject: Re: [Qemu-devel] [PATCH v7 00/13] curl: fix curl read
Date: Fri, 7 Jun 2013 09:54:42 +0800 [thread overview]
Message-ID: <20130607015442.GA22188@localhost.nay.redhat.com> (raw)
In-Reply-To: <20130606110157.GM4515@redhat.com>
On Thu, 06/06 12:01, Richard W.M. Jones wrote:
> 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 weird thing is it doesn't work for me, I'm sure something is wrong
with current upstream, although not totally broken.
$./qemu-io http://localhost/vm/arch.raw -c 'read -v 0 512'
(stuck here forever, no output)
Attach with gdb:
(gdb) bt
#0 aio_dispatch (ctx=0xaa1af230) at aio-posix.c:128
#1 0x00007f6fa96a18f7 in aio_poll (ctx=0x7f6faa1afa00,
blocking=true) at aio-posix.c:190
#2 0x00007f6fa96ecabc in qemu_aio_wait ()
at main-loop.c:484
#3 0x00007f6fa96a7530 in bdrv_rwv_co (bs=0x7f6faa1b9540,
sector_num=0, qiov=0x7fffacf403b0, is_write=false)
at block.c:2216
#4 0x00007f6fa96a75ca in bdrv_rw_co (bs=0x7f6faa1b9540,
sector_num=0, buf=0x7fffacf406f0 "", nb_sectors=4,
is_write=false) at block.c:2235
#5 0x00007f6fa96a7629 in bdrv_read (bs=0x7f6faa1b9540,
sector_num=0, buf=0x7fffacf406f0 "", nb_sectors=4)
at block.c:2242
#6 0x00007f6fa96a78f5 in bdrv_pread (bs=0x7f6faa1b9540,
offset=0, buf=0x7fffacf406f0, count1=2048)
at block.c:2304
#7 0x00007f6fa96a36f4 in find_image_format (
bs=0x7f6faa1b9540,
filename=0x7fffacf4441d "http://localhost/vm/arch.raw",
pdrv=0x7fffacf40f28) at block.c:542
#8 0x00007f6fa96a4a8a in bdrv_open (bs=0x7f6faa1b5b00,
filename=0x7fffacf4441d "http://localhost/vm/arch.raw",
options=0x7f6faa1b7500, flags=24578, drv=0x0)
at block.c:1055
#9 0x00007f6fa970e803 in openfile (
name=0x7fffacf4441d "http://localhost/vm/arch.raw",
flags=16386, growable=0) at qemu-io.c:1812
#10 0x00007f6fa970ef5b in main (argc=4,
argv=0x7fffacf43308) at qemu-io.c:2055
The BlockDriverState looks normal, just that libcurl not making any
progress (no callback, no timeout)
--
Fam
next prev parent reply other threads:[~2013-06-07 1:54 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-06-06 6:25 [Qemu-devel] [PATCH v7 00/13] curl: fix curl read Fam Zheng
2013-06-06 6:25 ` [Qemu-devel] [PATCH v7 01/13] curl: introduce CURLSockInfo to BDRVCURLState Fam Zheng
2013-06-06 6:25 ` [Qemu-devel] [PATCH v7 02/13] curl: change magic number to sizeof Fam Zheng
2013-06-06 6:25 ` [Qemu-devel] [PATCH v7 03/13] curl: change curl_multi_do to curl_fd_handler Fam Zheng
2013-06-06 6:25 ` [Qemu-devel] [PATCH v7 04/13] curl: fix curl_open Fam Zheng
2013-06-06 6:25 ` [Qemu-devel] [PATCH v7 05/13] curl: add timer to BDRVCURLState Fam Zheng
2013-06-06 6:25 ` [Qemu-devel] [PATCH v7 06/13] curl: introduce CURLDataCache Fam Zheng
2013-06-06 6:25 ` [Qemu-devel] [PATCH v7 07/13] curl: make use of CURLDataCache Fam Zheng
2013-06-06 6:25 ` [Qemu-devel] [PATCH v7 08/13] curl: use list to store CURLState Fam Zheng
2013-06-06 6:25 ` [Qemu-devel] [PATCH v7 09/13] curl: add cache quota Fam Zheng
2013-06-06 6:25 ` [Qemu-devel] [PATCH v7 10/13] curl: introduce ssl_no_cert runtime option Fam Zheng
2013-06-06 6:25 ` [Qemu-devel] [PATCH v7 11/13] block/curl.c: Refuse to open the handle for writes Fam Zheng
2013-06-06 6:25 ` [Qemu-devel] [PATCH v7 12/13] curl: set s->url to NULL after free Fam Zheng
2013-06-06 6:25 ` [Qemu-devel] [PATCH v7 13/13] curl: change timeout to 30 seconds Fam Zheng
2013-06-06 11:01 ` [Qemu-devel] [PATCH v7 00/13] curl: fix curl read Richard W.M. Jones
2013-06-07 1:54 ` Fam Zheng [this message]
2013-06-07 7:27 ` Richard W.M. Jones
2013-06-07 7:47 ` Fam Zheng
2013-06-07 9:41 ` Richard W.M. Jones
2013-06-07 7:46 ` Stefan Hajnoczi
2013-06-07 7:53 ` Fam Zheng
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20130607015442.GA22188@localhost.nay.redhat.com \
--to=famz@redhat.com \
--cc=jcody@redhat.com \
--cc=kwolf@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=rjones@redhat.com \
--cc=stefanha@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.