From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:44976) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dp3j2-0008DR-3G for qemu-devel@nongnu.org; Mon, 04 Sep 2017 22:36:32 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dp3ix-0007tu-D7 for qemu-devel@nongnu.org; Mon, 04 Sep 2017 22:36:28 -0400 Received: from mx1.redhat.com ([209.132.183.28]:39804) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1dp3ix-0007sv-3f for qemu-devel@nongnu.org; Mon, 04 Sep 2017 22:36:23 -0400 References: From: Jason Wang Message-ID: <0dbfac40-b8e4-9e13-78ee-6be038db8c4e@redhat.com> Date: Tue, 5 Sep 2017 10:36:01 +0800 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH v9 0/4] Improve error reporting List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Mao Zhongyi , qemu-devel@nongnu.org Cc: armbru@redhat.com, berrange@redhat.com, kraxel@redhat.com, pbonzini@redhat.com, eblake@redhat.com On 2017=E5=B9=B409=E6=9C=8804=E6=97=A5 22:35, Mao Zhongyi wrote: > v9: > * -rebase on upstream > -fix the commit message of patch04 > > v8: > * PATCH 02 & 04 > -resetting the error message for the user to read. [Markus Armbrus= ter] > -fix the indentation and commit message. [Markus Armbruster] > =20 > 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_p= ort(), > 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 Armbru= ster] > > 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 substanti= ally, > so that ensure each patch is a completed fix. [Markus Ar= mbruster] > > v4: > * PATCH 01 is redoing previous patch 1, replace the fprintf() with erro= r_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 conv= ert error_report() > in the function called by net_socket_*_init() to Error. Als= o add many error > handling information. > * PATCH 03 net_socket_mcast_create(), net_socket_fd_init_dgram() and ne= t_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: jasowang@redhat.com > Cc: armbru@redhat.com > Cc: berrange@redhat.com > Cc: kraxel@redhat.com > Cc: pbonzini@redhat.com > Cc: eblake@redhat.com > > 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(-) > Applied. Thanks