From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:57522) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bLSqo-0001I8-CT for qemu-devel@nongnu.org; Fri, 08 Jul 2016 06:17:39 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bLSqm-0002kD-E3 for qemu-devel@nongnu.org; Fri, 08 Jul 2016 06:17:37 -0400 Date: Fri, 8 Jul 2016 12:17:28 +0200 From: Kevin Wolf Message-ID: <20160708101728.GI14684@noname.redhat.com> References: <20160622125320.31279-1-famz@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Subject: Re: [Qemu-devel] [Qemu-block] [PATCH v2 0/2] block: Add qemu_dup in osdep.c and use it List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: John Snow Cc: Fam Zheng , qemu-devel@nongnu.org, qemu-block@nongnu.org, Max Reitz Am 07.07.2016 um 23:14 hat John Snow geschrieben: > On 06/22/2016 08:53 AM, Fam Zheng wrote: > > v2: Fix patch 1 #else branch, and "r" => "ret". [Kevin] > > Add Kevin's r-b line in patch 2. > > > > This is an independent tiny change extracted from the image locking series, > > which can be processed separately. > > > > Improved according to Kevin's suggestion in v6 image locking series. > > > > Fam > > > > > > Fam Zheng (2): > > osdep: Introduce qemu_dup > > raw-posix: Use qemu_dup > > > > block/raw-posix.c | 10 +--------- > > include/qemu/osdep.h | 3 +++ > > util/osdep.c | 23 +++++++++++++++-------- > > 3 files changed, 19 insertions(+), 17 deletions(-) > > > > You submitted two versions of this, I think this is the right one. > > Reviewed-by: John Snow Oops, completely forgot about this one. Thanks for reminding, I've applied it now. Kevin