qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Richard W.M. Jones" <rjones@redhat.com>
To: qemu-devel@nongnu.org, kwolf@redhat.com, stefanha@redhat.com,
	jcody@redhat.com
Subject: Re: [Qemu-devel] [PATCH v7 00/13] curl: fix curl read
Date: Fri, 7 Jun 2013 08:27:07 +0100	[thread overview]
Message-ID: <20130607072707.GP4515@redhat.com> (raw)
In-Reply-To: <20130607015442.GA22188@localhost.nay.redhat.com>

On Fri, Jun 07, 2013 at 09:54:42AM +0800, Fam Zheng wrote:
> 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)

This doesn't work for me either.

*However* it only doesn't work if I use the Fedora version of curl,
which I'm convinced is broken.

That's why I'm doing all my testing with upstream curl from git,
including all the qemu tests I did.  The above command works fine with
curl from git:

$ LD_LIBRARY_PATH=~/d/curl/lib/.libs ./qemu-io http://192.168.0.249/scratch/winxp.img -c 'read -v 0 512'
00000000:  33 c0 8e d0 bc 00 7c fb 50 07 50 1f fc be 1b 7c  3.......P.P.....
00000010:  bf 1b 06 50 57 b9 e5 01 f3 a4 cb bd be 07 b1 04  ...PW...........
00000020:  38 6e 00 7c 09 75 13 83 c5 10 e2 f4 cd 18 8b f5  8n...u..........
00000030:  83 c6 10 49 74 19 38 2c 74 f6 a0 b5 07 b4 07 8b  ...It.8.t.......
00000040:  f0 ac 3c 00 74 fc bb 07 00 b4 0e cd 10 eb f2 88  ....t...........
00000050:  4e 10 e8 46 00 73 2a fe 46 10 80 7e 04 0b 74 0b  N..F.s..F.....t.
00000060:  80 7e 04 0c 74 05 a0 b6 07 75 d2 80 46 02 06 83  ....t....u..F...
00000070:  46 08 06 83 56 0a 00 e8 21 00 73 05 a0 b6 07 eb  F...V.....s.....
00000080:  bc 81 3e fe 7d 55 aa 74 0b 80 7e 10 00 74 c8 a0  .....U.t.....t..
00000090:  b7 07 eb a9 8b fc 1e 57 8b f5 cb bf 05 00 8a 56  .......W.......V
000000a0:  00 b4 08 cd 13 72 23 8a c1 24 3f 98 8a de 8a fc  .....r..........
000000b0:  43 f7 e3 8b d1 86 d6 b1 06 d2 ee 42 f7 e2 39 56  C..........B..9V
000000c0:  0a 77 23 72 05 39 46 08 73 1c b8 01 02 bb 00 7c  .w.r.9F.s.......
000000d0:  8b 4e 02 8b 56 00 cd 13 73 51 4f 74 4e 32 e4 8a  .N..V...sQOtN2..
000000e0:  56 00 cd 13 eb e4 8a 56 00 60 bb aa 55 b4 41 cd  V......V....U.A.
000000f0:  13 72 36 81 fb 55 aa 75 30 f6 c1 01 74 2b 61 60  .r6..U.u0...t.a.
00000100:  6a 00 6a 00 ff 76 0a ff 76 08 6a 00 68 00 7c 6a  j.j..v..v.j.h..j
00000110:  01 6a 10 b4 42 8b f4 cd 13 61 61 73 0e 4f 74 0b  .j..B....aas.Ot.
00000120:  32 e4 8a 56 00 cd 13 eb d6 61 f9 c3 49 6e 76 61  2..V.....a..Inva
00000130:  6c 69 64 20 70 61 72 74 69 74 69 6f 6e 20 74 61  lid.partition.ta
00000140:  62 6c 65 00 45 72 72 6f 72 20 6c 6f 61 64 69 6e  ble.Error.loadin
00000150:  67 20 6f 70 65 72 61 74 69 6e 67 20 73 79 73 74  g.operating.syst
00000160:  65 6d 00 4d 69 73 73 69 6e 67 20 6f 70 65 72 61  em.Missing.opera
00000170:  74 69 6e 67 20 73 79 73 74 65 6d 00 00 00 00 00  ting.system.....
00000180:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00  ................
00000190:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00  ................
000001a0:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00  ................
000001b0:  00 00 00 00 00 2c 44 63 1a dd 1a dd 00 00 80 01  ......Dc........
000001c0:  01 00 07 fe ff 0d 3f 00 00 00 4f b1 bf 00 00 00  ..........O.....
000001d0:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00  ................
000001e0:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00  ................
000001f0:  00 00 00 00 00 00 00 00 00 00 00 00 00 00 55 aa  ..............U.
read 512/512 bytes at offset 0
512 bytes, 1 ops; 0.0000 sec (32.552 MiB/sec and 66666.6667 ops/sec)

Rich.

-- 
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.
http://fedoraproject.org/wiki/MinGW

  reply	other threads:[~2013-06-07  7:27 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
2013-06-07  7:27     ` Richard W.M. Jones [this message]
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=20130607072707.GP4515@redhat.com \
    --to=rjones@redhat.com \
    --cc=jcody@redhat.com \
    --cc=kwolf@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).