[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v6 00/12] curl: fix curl read
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH v6 00/12] curl: fix curl read |
Date: |
Wed, 29 May 2013 09:07:25 +0800 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Tue, 05/28 12:32, Richard W.M. Jones wrote:
>
> This fixes the obvious bug.
Thanks for figuring out this. Mainline had this 5s timeout so I kept it,
but you don't experience this bug, right? Since master doesn't setup a
timer to get curl notified about the timing, the option is just not
effective.
> I wonder if it should be even larger? One use for curl is to install
> guests using ISOs from websites without having to download the ISO,
> and I imagine that even a 30 second timeout could be conservative for
> that task.
>
Long latency network is common in practice, as well as low bandwidth,
the meaning of the timeout is to complete the request, in extreme cases
if it is a 1Kbps link, downloading 256k takes minutes. Anyway, I think
making it larger won't hurt.
--
Fam
- Re: [Qemu-devel] [PATCH v6 00/12] curl: fix curl read, (continued)
- Re: [Qemu-devel] [PATCH v6 00/12] curl: fix curl read, Fam Zheng, 2013/05/28
- Re: [Qemu-devel] [PATCH v6 00/12] curl: fix curl read, Richard W.M. Jones, 2013/05/28
- Re: [Qemu-devel] [PATCH v6 00/12] curl: fix curl read, Richard W.M. Jones, 2013/05/28
- Re: [Qemu-devel] [PATCH v6 00/12] curl: fix curl read, Fam Zheng, 2013/05/28
- Re: [Qemu-devel] [PATCH v6 00/12] curl: fix curl read, Richard W.M. Jones, 2013/05/28
Re: [Qemu-devel] [PATCH v6 00/12] curl: fix curl read, Richard W.M. Jones, 2013/05/28
Re: [Qemu-devel] [PATCH v6 00/12] curl: fix curl read, Stefan Hajnoczi, 2013/05/28