[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v7 1/4] qerror: add five qerror strings
From: |
Michael Roth |
Subject: |
Re: [Qemu-devel] [PATCH v7 1/4] qerror: add five qerror strings |
Date: |
Thu, 19 Apr 2012 20:20:44 -0500 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Thu, Apr 19, 2012 at 03:11:25PM +0800, Amos Kong wrote:
> Add five new qerror strings, they are about listen/connect socket:
> QERR_SOCKET_CONNECT_IN_PROGRESS
> QERR_SOCKET_CONNECT_FAILED
> QERR_SOCKET_LISTEN_FAILED
> QERR_SOCKET_BIND_FAILED
> QERR_SOCKET_CREATE_FAILED
>
> Signed-off-by: Amos Kong <address@hidden>
I had some comments on the last series regarding the wording of the error
messages that aren't addressed here.
> ---
> qerror.c | 20 ++++++++++++++++++++
> qerror.h | 15 +++++++++++++++
> 2 files changed, 35 insertions(+), 0 deletions(-)
>
> diff --git a/qerror.c b/qerror.c
> index 96fbe71..2963b8c 100644
> --- a/qerror.c
> +++ b/qerror.c
> @@ -304,6 +304,26 @@ static const QErrorStringTable qerror_table[] = {
> .error_fmt = QERR_VNC_SERVER_FAILED,
> .desc = "Could not start VNC server on %(target)",
> },
> + {
> + .error_fmt = QERR_SOCKET_CONNECT_IN_PROGRESS,
> + .desc = "Connection can not be completed immediately",
> + },
> + {
> + .error_fmt = QERR_SOCKET_CONNECT_FAILED,
> + .desc = "Failed to connect socket",
> + },
> + {
> + .error_fmt = QERR_SOCKET_LISTEN_FAILED,
> + .desc = "Failed to listen socket",
> + },
> + {
> + .error_fmt = QERR_SOCKET_BIND_FAILED,
> + .desc = "Failed to bind socket",
> + },
> + {
> + .error_fmt = QERR_SOCKET_CREATE_FAILED,
> + .desc = "Failed to create socket",
> + },
> {}
> };
>
> diff --git a/qerror.h b/qerror.h
> index 5c23c1f..4cbba48 100644
> --- a/qerror.h
> +++ b/qerror.h
> @@ -248,4 +248,19 @@ QError *qobject_to_qerror(const QObject *obj);
> #define QERR_VNC_SERVER_FAILED \
> "{ 'class': 'VNCServerFailed', 'data': { 'target': %s } }"
>
> +#define QERR_SOCKET_CONNECT_IN_PROGRESS \
> + "{ 'class': 'SockConnectInprogress', 'data': {} }"
> +
> +#define QERR_SOCKET_CONNECT_FAILED \
> + "{ 'class': 'SockConnectFailed', 'data': {} }"
> +
> +#define QERR_SOCKET_LISTEN_FAILED \
> + "{ 'class': 'SockListenFailed', 'data': {} }"
> +
> +#define QERR_SOCKET_BIND_FAILED \
> + "{ 'class': 'SockBindFailed', 'data': {} }"
> +
> +#define QERR_SOCKET_CREATE_FAILED \
> + "{ 'class': 'SockCreateFailed', 'data': {} }"
> +
> #endif /* QERROR_H */
>
- [Qemu-devel] [PATCH v7 0/5] support to migrate with IPv6 address, Amos Kong, 2012/04/19
- [Qemu-devel] [PATCH v7 2/4] sockets: change inet_connect() to support nonblock socket, Amos Kong, 2012/04/19
- [Qemu-devel] [PATCH v7 1/4] qerror: add five qerror strings, Amos Kong, 2012/04/19
- Re: [Qemu-devel] [PATCH v7 1/4] qerror: add five qerror strings,
Michael Roth <=
- [Qemu-devel] [PATCH v7 3/4] sockets: use error class to pass listen error, Amos Kong, 2012/04/19
- [Qemu-devel] [PATCH v7 4/4] use inet_listen()/inet_connect() to support ipv6 migration, Amos Kong, 2012/04/19
- [Qemu-devel] [PATCH v7 0/4] support to migrate with IPv6 address, Amos Kong, 2012/04/19
- Re: [Qemu-devel] [PATCH v7 0/5] support to migrate with IPv6 address, Orit Wasserman, 2012/04/19