Since commit e65c67e4 & d984464e, they are not used anymore, and all non-blocking connect now goes through inet_connect_addr().
Cc: Daniel P. Berrange <berra...@redhat.com> Cc: Gerd Hoffmann <kra...@redhat.com> Cc: Paolo Bonzini <pbonz...@redhat.com> Cc: Eric Blake <ebl...@redhat.com> Reviewed-by: Eric Blake <ebl...@redhat.com> Signed-off-by: Cao jin <caoj.f...@cn.fujitsu.com> --- include/qemu/sockets.h | 6 ------ util/qemu-sockets.c | 46 ---------------------------------------------- 2 files changed, 52 deletions(-) This patch is squashed as Eric suggested. As v1 has been reviewed-by, so I take the liberty to add Eric's R-b in this v2. diff --git a/include/qemu/sockets.h b/include/qemu/sockets.h index 5fe01fb..28a28c0 100644 --- a/include/qemu/sockets.h +++ b/include/qemu/sockets.h @@ -36,17 +36,11 @@ InetSocketAddress *inet_parse(const char *str, Error **errp); int inet_listen(const char *str, char *ostr, int olen, int socktype, int port_offset, Error **errp); int inet_connect(const char *str, Error **errp); -int inet_nonblocking_connect(const char *str, - NonBlockingConnectHandler *callback, - void *opaque, Error **errp); NetworkAddressFamily inet_netfamily(int family); int unix_listen(const char *path, char *ostr, int olen, Error **errp); int unix_connect(const char *path, Error **errp); -int unix_nonblocking_connect(const char *str, - NonBlockingConnectHandler *callback, - void *opaque, Error **errp); SocketAddress *socket_parse(const char *str, Error **errp); int socket_connect(SocketAddress *addr, Error **errp, diff --git a/util/qemu-sockets.c b/util/qemu-sockets.c index fb83d48..0962646 100644 --- a/util/qemu-sockets.c +++ b/util/qemu-sockets.c @@ -674,36 +674,6 @@ int inet_connect(const char *str, Error **errp) return sock; } -/** - * Create a non-blocking socket and connect it to an address. - * Calls the callback function with fd in case of success or -1 in case of - * error. - * - * @str: address string - * @callback: callback function that is called when connect completes, - * cannot be NULL. - * @opaque: opaque for callback function - * @errp: set in case of an error - * - * Returns: -1 on immediate error, file descriptor on success. - **/ -int inet_nonblocking_connect(const char *str, - NonBlockingConnectHandler *callback, - void *opaque, Error **errp) -{ - int sock = -1; - InetSocketAddress *addr; - - g_assert(callback != NULL); - - addr = inet_parse(str, errp); - if (addr != NULL) { - sock = inet_connect_saddr(addr, errp, callback, opaque); - qapi_free_InetSocketAddress(addr); - } - return sock; -} - #ifndef _WIN32 static int unix_listen_saddr(UnixSocketAddress *saddr, @@ -893,22 +863,6 @@ int unix_connect(const char *path, Error **errp) } -int unix_nonblocking_connect(const char *path, - NonBlockingConnectHandler *callback, - void *opaque, Error **errp) -{ - UnixSocketAddress *saddr; - int sock = -1; - - g_assert(callback != NULL); - - saddr = g_new0(UnixSocketAddress, 1); - saddr->path = g_strdup(path); - sock = unix_connect_saddr(saddr, errp, callback, opaque); - qapi_free_UnixSocketAddress(saddr); - return sock; -} - SocketAddress *socket_parse(const char *str, Error **errp) { SocketAddress *addr; -- 2.1.0