[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 28/31] curl: Remove unnecessary explicit calls to
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PULL 28/31] curl: Remove unnecessary explicit calls to internal event handler |
Date: |
Tue, 6 May 2014 11:56:24 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Am 06.05.2014 um 11:44 hat Laurent Desnogues geschrieben:
> Hello,
>
> sorry for another late detection of the use of too recent features.
>
> On Wed, Apr 30, 2014 at 8:24 PM, Kevin Wolf <address@hidden> wrote:
> > From: Matthew Booth <address@hidden>
> >
> > Remove calls to curl_multi_do where the relevant handles are already
> > registered to the event loop.
> >
> > Ensure that we kick off socket handling with CURL_SOCKET_TIMEOUT after
> > adding a new handle.
> >
> > Signed-off-by: Matthew Booth <address@hidden>
> > Tested-by: Richard W.M. Jones <address@hidden>
> > Signed-off-by: Kevin Wolf <address@hidden>
> > ---
> > block/curl.c | 5 +++--
> > 1 file changed, 3 insertions(+), 2 deletions(-)
> >
> > diff --git a/block/curl.c b/block/curl.c
> > index 50bd05f..fd2756e 100644
> > --- a/block/curl.c
> > +++ b/block/curl.c
> > @@ -535,7 +535,6 @@ static int curl_open(BlockDriverState *bs, QDict
> > *options, int flags,
> > curl_multi_setopt(s->multi, CURLMOPT_TIMERDATA, s);
> > curl_multi_setopt(s->multi, CURLMOPT_TIMERFUNCTION, curl_timer_cb);
> > #endif
> > - curl_multi_do(s);
> >
> > qemu_opts_del(opts);
> > return 0;
> > @@ -564,6 +563,7 @@ static const AIOCBInfo curl_aiocb_info = {
> > static void curl_readv_bh_cb(void *p)
> > {
> > CURLState *state;
> > + int running;
> >
> > CURLAIOCB *acb = p;
> > BDRVCURLState *s = acb->common.bs->opaque;
> > @@ -612,8 +612,9 @@ static void curl_readv_bh_cb(void *p)
> > curl_easy_setopt(state->curl, CURLOPT_RANGE, state->range);
> >
> > curl_multi_add_handle(s->multi, state->curl);
> > - curl_multi_do(s);
> >
> > + /* Tell curl it needs to kick things off */
> > + curl_multi_socket_action(s->multi, CURL_SOCKET_TIMEOUT, 0, &running);
>
> curl_multi_socket_action isn't available on the CURL lib used
> by CentOS 5.6. The obvious workaround is to disable CURL
> when calling configure.
Matthew, can you please check whether this call can be avoided, or
otherwise change configure to check the presence of the function?
Kevin