* [Qemu-devel] [PATCH] qemu-socket: Eliminate silly QERR_ macros
@ 2014-09-25 6:49 Markus Armbruster
2014-09-25 6:50 ` Markus Armbruster
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Markus Armbruster @ 2014-09-25 6:49 UTC (permalink / raw)
To: qemu-devel; +Cc: lcapitulino
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 <armbru@redhat.com>
---
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;
}
--
1.9.3
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [Qemu-devel] [PATCH] qemu-socket: Eliminate silly QERR_ macros
2014-09-25 6:49 [Qemu-devel] [PATCH] qemu-socket: Eliminate silly QERR_ macros Markus Armbruster
@ 2014-09-25 6:50 ` Markus Armbruster
2014-09-25 8:53 ` Paolo Bonzini
2014-09-26 17:37 ` Luiz Capitulino
2 siblings, 0 replies; 4+ messages in thread
From: Markus Armbruster @ 2014-09-25 6:50 UTC (permalink / raw)
To: qemu-devel; +Cc: lcapitulino
Markus Armbruster <armbru@redhat.com> writes:
> 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 <armbru@redhat.com>
Forgot to mention: this applies on top of "[PATCH] qemu-socket: Polish
errors for connect() and listen() failure".
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Qemu-devel] [PATCH] qemu-socket: Eliminate silly QERR_ macros
2014-09-25 6:49 [Qemu-devel] [PATCH] qemu-socket: Eliminate silly QERR_ macros Markus Armbruster
2014-09-25 6:50 ` Markus Armbruster
@ 2014-09-25 8:53 ` Paolo Bonzini
2014-09-26 17:37 ` Luiz Capitulino
2 siblings, 0 replies; 4+ messages in thread
From: Paolo Bonzini @ 2014-09-25 8:53 UTC (permalink / raw)
To: Markus Armbruster, qemu-devel; +Cc: lcapitulino
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 <armbru@redhat.com>
> ---
> 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 <pbonzini@redhat.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Qemu-devel] [PATCH] qemu-socket: Eliminate silly QERR_ macros
2014-09-25 6:49 [Qemu-devel] [PATCH] qemu-socket: Eliminate silly QERR_ macros Markus Armbruster
2014-09-25 6:50 ` Markus Armbruster
2014-09-25 8:53 ` Paolo Bonzini
@ 2014-09-26 17:37 ` Luiz Capitulino
2 siblings, 0 replies; 4+ messages in thread
From: Luiz Capitulino @ 2014-09-26 17:37 UTC (permalink / raw)
To: Markus Armbruster; +Cc: qemu-devel
On Thu, 25 Sep 2014 08:49:31 +0200
Markus Armbruster <armbru@redhat.com> wrote:
> 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 <armbru@redhat.com>
Applied to the qmp branch, thanks.
> ---
> 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;
> }
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2014-09-26 18:56 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-09-25 6:49 [Qemu-devel] [PATCH] qemu-socket: Eliminate silly QERR_ macros Markus Armbruster
2014-09-25 6:50 ` Markus Armbruster
2014-09-25 8:53 ` Paolo Bonzini
2014-09-26 17:37 ` Luiz Capitulino
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).