From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51460) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Xbx93-0005En-8V for qemu-devel@nongnu.org; Wed, 08 Oct 2014 15:43:39 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Xbx8x-0001HX-4o for qemu-devel@nongnu.org; Wed, 08 Oct 2014 15:43:33 -0400 Received: from mx1.redhat.com ([209.132.183.28]:59711) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Xbx8w-0001HL-Tj for qemu-devel@nongnu.org; Wed, 08 Oct 2014 15:43:27 -0400 Message-ID: <543593D7.4070909@redhat.com> Date: Wed, 08 Oct 2014 21:43:19 +0200 From: Max Reitz MIME-Version: 1.0 References: <1411400179-10605-1-git-send-email-mreitz@redhat.com> In-Reply-To: <1411400179-10605-1-git-send-email-mreitz@redhat.com> Content-Type: text/plain; charset=iso-8859-15; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v2 0/2] raw-posix: Fix raw_co_get_block_status() List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Kevin Wolf , Paolo Bonzini , =?ISO-8859-15?Q?Beno=EEt_Canet?= , Stefan Hajnoczi On 22.09.2014 17:36, Max Reitz wrote: > raw_co_get_block_status() should return 0 and set *pnum to 0 after the > EOF; currently it does this merely by accident, so implement it > directly. Also, nb_sectors should be clamped against the image end. > > While doing that, centralize the generation of > raw_co_get_block_status()'s return value along the way. > > > v2: > - Patch 1: Clamp nb_sectors against image end > - Patch 2: Fix alignment issue > > > Max Reitz (2): > raw-posix: Fix raw_co_get_block_status() after EOF > raw-posix: raw_co_get_block_status() return value > > block/raw-posix.c | 36 ++++++++++++++++++++++-------------- > 1 file changed, 22 insertions(+), 14 deletions(-) Ping. (This should be rather simple to review)