[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] qemu-socket: Eliminate silly QERR_ macros
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH] qemu-socket: Eliminate silly QERR_ macros |
Date: |
Thu, 25 Sep 2014 10:53:24 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.0 |
Il 25/09/2014 08:49, Markus Armbruster ha scritto:
> The QERR_ macros are leftovers from the days of "rich" error objects.
> They're used with error_set() and qerror_report(), and expand into the
> first *two* arguments. This trickiness has become pointless. Clean
> up.
>
> Signed-off-by: Markus Armbruster <address@hidden>
> ---
> include/qapi/qmp/qerror.h | 12 ------------
> util/qemu-sockets.c | 26 +++++++++++++-------------
> 2 files changed, 13 insertions(+), 25 deletions(-)
>
> diff --git a/include/qapi/qmp/qerror.h b/include/qapi/qmp/qerror.h
> index 774e75d..0ca6cbd 100644
> --- a/include/qapi/qmp/qerror.h
> +++ b/include/qapi/qmp/qerror.h
> @@ -154,16 +154,4 @@ void qerror_report_err(Error *err);
> #define QERR_UNSUPPORTED \
> ERROR_CLASS_GENERIC_ERROR, "this feature or command is not currently
> supported"
>
> -#define QERR_SOCKET_CONNECT_FAILED \
> - ERROR_CLASS_GENERIC_ERROR, "Failed to connect socket"
> -
> -#define QERR_SOCKET_LISTEN_FAILED \
> - ERROR_CLASS_GENERIC_ERROR, "Failed to listen on socket"
> -
> -#define QERR_SOCKET_BIND_FAILED \
> - ERROR_CLASS_GENERIC_ERROR, "Failed to bind socket"
> -
> -#define QERR_SOCKET_CREATE_FAILED \
> - ERROR_CLASS_GENERIC_ERROR, "Failed to create socket"
> -
> #endif /* QERROR_H */
> diff --git a/util/qemu-sockets.c b/util/qemu-sockets.c
> index 4a25585..1eef590 100644
> --- a/util/qemu-sockets.c
> +++ b/util/qemu-sockets.c
> @@ -159,7 +159,7 @@ int inet_listen_opts(QemuOpts *opts, int port_offset,
> Error **errp)
> slisten = qemu_socket(e->ai_family, e->ai_socktype, e->ai_protocol);
> if (slisten < 0) {
> if (!e->ai_next) {
> - error_set_errno(errp, errno, QERR_SOCKET_CREATE_FAILED);
> + error_setg_errno(errp, errno, "Failed to create socket");
> }
> continue;
> }
> @@ -183,7 +183,7 @@ int inet_listen_opts(QemuOpts *opts, int port_offset,
> Error **errp)
> }
> if (p == port_max) {
> if (!e->ai_next) {
> - error_set_errno(errp, errno, QERR_SOCKET_BIND_FAILED);
> + error_setg_errno(errp, errno, "Failed to bind socket");
> }
> }
> }
> @@ -194,7 +194,7 @@ int inet_listen_opts(QemuOpts *opts, int port_offset,
> Error **errp)
>
> listen:
> if (listen(slisten,1) != 0) {
> - error_set_errno(errp, errno, QERR_SOCKET_LISTEN_FAILED);
> + error_setg_errno(errp, errno, "Failed to listen on socket");
> closesocket(slisten);
> freeaddrinfo(res);
> return -1;
> @@ -281,7 +281,7 @@ static int inet_connect_addr(struct addrinfo *addr, bool
> *in_progress,
>
> sock = qemu_socket(addr->ai_family, addr->ai_socktype,
> addr->ai_protocol);
> if (sock < 0) {
> - error_set_errno(errp, errno, QERR_SOCKET_CREATE_FAILED);
> + error_setg_errno(errp, errno, "Failed to create socket");
> return -1;
> }
> socket_set_fast_reuse(sock);
> @@ -302,7 +302,7 @@ static int inet_connect_addr(struct addrinfo *addr, bool
> *in_progress,
> connect_state);
> *in_progress = true;
> } else if (rc < 0) {
> - error_set_errno(errp, errno, QERR_SOCKET_CONNECT_FAILED);
> + error_setg_errno(errp, errno, "Failed to connect socket");
> closesocket(sock);
> return -1;
> }
> @@ -466,20 +466,20 @@ int inet_dgram_opts(QemuOpts *opts, Error **errp)
> /* create socket */
> sock = qemu_socket(peer->ai_family, peer->ai_socktype,
> peer->ai_protocol);
> if (sock < 0) {
> - error_set_errno(errp, errno, QERR_SOCKET_CREATE_FAILED);
> + error_setg_errno(errp, errno, "Failed to create socket");
> goto err;
> }
> socket_set_fast_reuse(sock);
>
> /* bind socket */
> if (bind(sock, local->ai_addr, local->ai_addrlen) < 0) {
> - error_set_errno(errp, errno, QERR_SOCKET_BIND_FAILED);
> + error_setg_errno(errp, errno, "Failed to bind socket");
> goto err;
> }
>
> /* connect to peer */
> if (connect(sock,peer->ai_addr,peer->ai_addrlen) < 0) {
> - error_set_errno(errp, errno, QERR_SOCKET_CONNECT_FAILED);
> + error_setg_errno(errp, errno, "Failed to connect socket");
> goto err;
> }
>
> @@ -684,7 +684,7 @@ int unix_listen_opts(QemuOpts *opts, Error **errp)
>
> sock = qemu_socket(PF_UNIX, SOCK_STREAM, 0);
> if (sock < 0) {
> - error_set_errno(errp, errno, QERR_SOCKET_CREATE_FAILED);
> + error_setg_errno(errp, errno, "Failed to create socket");
> return -1;
> }
>
> @@ -709,11 +709,11 @@ int unix_listen_opts(QemuOpts *opts, Error **errp)
>
> unlink(un.sun_path);
> if (bind(sock, (struct sockaddr*) &un, sizeof(un)) < 0) {
> - error_set_errno(errp, errno, QERR_SOCKET_BIND_FAILED);
> + error_setg_errno(errp, errno, "Failed to bind socket");
> goto err;
> }
> if (listen(sock, 1) < 0) {
> - error_set_errno(errp, errno, QERR_SOCKET_LISTEN_FAILED);
> + error_setg_errno(errp, errno, "Failed to listen on socket");
> goto err;
> }
>
> @@ -739,7 +739,7 @@ int unix_connect_opts(QemuOpts *opts, Error **errp,
>
> sock = qemu_socket(PF_UNIX, SOCK_STREAM, 0);
> if (sock < 0) {
> - error_set_errno(errp, errno, QERR_SOCKET_CREATE_FAILED);
> + error_setg_errno(errp, errno, "Failed to create socket");
> return -1;
> }
> if (callback != NULL) {
> @@ -774,7 +774,7 @@ int unix_connect_opts(QemuOpts *opts, Error **errp,
> }
>
> if (rc < 0) {
> - error_set_errno(errp, -rc, QERR_SOCKET_CONNECT_FAILED);
> + error_setg_errno(errp, -rc, "Failed to connect socket");
> close(sock);
> sock = -1;
> }
>
Reviewed-by: Paolo Bonzini <address@hidden>