From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:52485) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bjzDJ-0001vF-9p for qemu-devel@nongnu.org; Tue, 13 Sep 2016 21:42:14 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bjzDE-0001lB-4P for qemu-devel@nongnu.org; Tue, 13 Sep 2016 21:42:13 -0400 Received: from [59.151.112.132] (port=48583 helo=heian.cn.fujitsu.com) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bjzDD-0001k6-N9 for qemu-devel@nongnu.org; Tue, 13 Sep 2016 21:42:08 -0400 References: <1469696074-12744-1-git-send-email-caoj.fnst@cn.fujitsu.com> <5799CA68.7040503@cn.fujitsu.com> <7047097c-a969-98b0-7a9e-afa714b870d4@redhat.com> From: Cao jin Message-ID: <57D8AB04.1060604@cn.fujitsu.com> Date: Wed, 14 Sep 2016 09:42:28 +0800 MIME-Version: 1.0 In-Reply-To: <7047097c-a969-98b0-7a9e-afa714b870d4@redhat.com> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v2 0/3] Remove the obsolete non-blocking connect List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini , qemu-devel@nongnu.org Cc: "Daniel P. Berrange" , Gerd Hoffmann Hi Paolo, patch 1 seems conflicts with upstream when I rebase(conflicts with 616018352, but very tiny). and patch 2 isn't in your pull request. Do you want me to resend these two? On 08/01/2016 06:08 PM, Paolo Bonzini wrote: > > > On 28/07/2016 11:03, Cao jin wrote: >> forget to cc maintainers in cover-letter.. > > Patches 1 and 2 look good, thanks! > >> On 07/28/2016 04:54 PM, Cao jin wrote: >>> v2 changelog: >>> 1. revert the yoda-condition in patch 2 (Daniel) >>> >>> Cao jin (3): >>> util: remove the obsolete non-blocking connect >>> util: fix some coding style issue >>> migration/socket: fix typo in file header >>> >>> include/qemu/sockets.h | 7 +- >>> io/channel-socket.c | 2 +- >>> migration/socket.c | 4 +- >>> net/socket.c | 2 +- >>> util/qemu-sockets.c | 179 >>> ++++++++----------------------------------------- >>> 5 files changed, 33 insertions(+), 161 deletions(-) >>> >> > > > -- Yours Sincerely, Cao jin