[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v9 0/4] Improve error reporting
From: |
Mao Zhongyi |
Subject: |
[Qemu-devel] [PATCH v9 0/4] Improve error reporting |
Date: |
Mon, 4 Sep 2017 22:35:36 +0800 |
v9:
* -rebase on upstream
-fix the commit message of patch04
v8:
* PATCH 02 & 04
-resetting the error message for the user to read. [Markus Armbruster]
-fix the indentation and commit message. [Markus Armbruster]
v7:
* PATCH 01
-fix the error message. [Daniel P. Berrange]
-adjust the indentation problem. [Eric Blake]
* PATCH 03
-print a generic message when gethostbyname() failed in parse_host_port(),
drop the misleading ": unkonwn host" part. [Markus Armbruster]
v6:
* PATCH 02
-rename the subject
-drop the "qemu: error: " prefix.
-correct inappropriate error information settings.
* PATCH 03,04
-correct inappropriate error information settings. [Markus Armbruster]
v5:
* PATCH 01 make the commit message more exact about the actual function.
[Markus Armbruster]
* PATCH 02, 03, 04 still retains the original function, but specific
content and order of each patch has been adjusted substantially,
so that ensure each patch is a completed fix. [Markus Armbruster]
v4:
* PATCH 01 is redoing previous patch 1, replace the fprintf() with
error_report()
in the 'default' case of net_socket_fd_init() [Markus
Armbruster]
v3:
* PATCH 01 is suggested by Markus and Daniel that removes the dubious 'default'
case
in the net_socket_fd_init(). Jason agreed.
* PATCH 02 is redoing previous patch 4.
* PATCH 04 is redoing previous patch 2, improves sort of error messages.
v2:
* PATCH 02 reworking of patch 2 following Markus's suggestion that convert
error_report()
in the function called by net_socket_*_init() to Error. Also add
many error
handling information.
* PATCH 03 net_socket_mcast_create(), net_socket_fd_init_dgram() and
net_socket_fd_init()
use the function such as fprintf, perror to report an error message.
Convert it
to Error.
* PATCH 04 parse_host_port() may fail without reporting an error. Now, fix it
to set an
error when it fails.
Cc: address@hidden
Cc: address@hidden
Cc: address@hidden
Cc: address@hidden
Cc: address@hidden
Cc: address@hidden
Mao Zhongyi (4):
net/socket: Don't treat odd socket type as SOCK_STREAM
net/socket: Convert several helper functions to Error
net/net: Convert parse_host_port() to Error
net/socket: Improve -net socket error reporting
include/qemu/sockets.h | 3 +-
net/net.c | 22 +++++--
net/socket.c | 156 ++++++++++++++++++++++++++++---------------------
3 files changed, 108 insertions(+), 73 deletions(-)
--
2.9.4
- [Qemu-devel] [PATCH v9 0/4] Improve error reporting,
Mao Zhongyi <=
- [Qemu-devel] [PATCH v9 1/4] net/socket: Don't treat odd socket type as SOCK_STREAM, Mao Zhongyi, 2017/09/04
- [Qemu-devel] [PATCH v9 4/4] net/socket: Improve -net socket error reporting, Mao Zhongyi, 2017/09/04
- [Qemu-devel] [PATCH v9 3/4] net/net: Convert parse_host_port() to Error, Mao Zhongyi, 2017/09/04
- [Qemu-devel] [PATCH v9 2/4] net/socket: Convert several helper functions to Error, Mao Zhongyi, 2017/09/04
- Re: [Qemu-devel] [PATCH v9 0/4] Improve error reporting, Jason Wang, 2017/09/04