qemu-block
[Top][All Lists]
Advanced

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

Re: [Qemu-block] [PATCH] util: remove the obsolete non-blocking connect


From: Juan Quintela
Subject: Re: [Qemu-block] [PATCH] util: remove the obsolete non-blocking connect
Date: Thu, 15 Jun 2017 09:34:40 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux)

Mao Zhongyi <address@hidden> wrote:
> From: Cao jin <address@hidden>
>
> The non-blocking connect mechanism is obsolete, and it doesn't
> work well in inet connection, because it will call getaddrinfo
> first and getaddrinfo will blocks on DNS lookups. Since commit
> e65c67e4 & d984464e, the non-blocking connect of migration goes
> through QIOChannel in a different manner(using a thread), and
> nobody use this old non-blocking connect anymore.
>
> Any newly written code which needs a non-blocking connect should
> use the QIOChannel code, so we can drop NonBlockingConnectHandler


Reviewed-by: Juan Quintela <address@hidden>


...

>  int inet_connect_saddr(InetSocketAddress *saddr,
> -                       NonBlockingConnectHandler *callback, void *opaque,
>                         Error **errp);

In case you have to respin it.

Now you can write the prototype in a single line, no?
Same for the rest of appearances.




reply via email to

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