From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:50173) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YNRg5-0005UF-7M for qemu-devel@nongnu.org; Mon, 16 Feb 2015 14:49:57 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YNRg2-0001wU-2O for qemu-devel@nongnu.org; Mon, 16 Feb 2015 14:49:57 -0500 Received: from mx1.redhat.com ([209.132.183.28]:51414) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YNRg1-0001wG-Qe for qemu-devel@nongnu.org; Mon, 16 Feb 2015 14:49:53 -0500 Message-ID: <54E249DE.70307@redhat.com> Date: Mon, 16 Feb 2015 14:49:50 -0500 From: Max Reitz MIME-Version: 1.0 References: <1423865338-8576-1-git-send-email-jsnow@redhat.com> <1423865338-8576-3-git-send-email-jsnow@redhat.com> In-Reply-To: <1423865338-8576-3-git-send-email-jsnow@redhat.com> Content-Type: text/plain; charset=iso-8859-15; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v13 02/17] qmp: Ensure consistent granularity type List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: John Snow , qemu-devel@nongnu.org Cc: kwolf@redhat.com, famz@redhat.com, armbru@redhat.com, vsementsov@parallels.com, stefanha@redhat.com On 2015-02-13 at 17:08, John Snow wrote: > We treat this field with a variety of different types everywhere > in the code. Now it's just uint32_t. > > Signed-off-by: John Snow > --- > block.c | 11 ++++++----- > block/mirror.c | 4 ++-- > include/block/block.h | 2 +- > include/block/block_int.h | 2 +- > qapi/block-core.json | 2 +- > 5 files changed, 11 insertions(+), 10 deletions(-) Considering you called this series the "$MY_NAME Fixup issue", I'm assuming this patch is the response to something I said, maybe that Coverity might complain about one right-shift of a 64 bit value with the result stored in a 32 bit integer. If so, that would have been preventable by assert((x >> 9) <= INT_MAX), so it's not that you would have to force everything to be 32 bits wide. Anyway, I'm fine with either 32 or 64 bits (2 GB maximum granularity should be good for now): Reviewed-by: Max Reitz