From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:58091) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1W26wv-00059J-7p for qemu-devel@nongnu.org; Sat, 11 Jan 2014 17:22:43 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1W26wo-0002UJ-5w for qemu-devel@nongnu.org; Sat, 11 Jan 2014 17:22:37 -0500 Received: from mx1.redhat.com ([209.132.183.28]:9431) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1W26wn-0002UE-Um for qemu-devel@nongnu.org; Sat, 11 Jan 2014 17:22:30 -0500 Message-ID: <52D1C484.5070608@redhat.com> Date: Sat, 11 Jan 2014 23:24:04 +0100 From: Max Reitz MIME-Version: 1.0 References: <1386940979-3824-1-git-send-email-kwolf@redhat.com> <1386940979-3824-2-git-send-email-kwolf@redhat.com> In-Reply-To: <1386940979-3824-2-git-send-email-kwolf@redhat.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v2 01/24] block: Move initialisation of BlockLimits to bdrv_refresh_limits() List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Kevin Wolf , qemu-devel@nongnu.org Cc: pbonzini@redhat.com, pl@kamp.de, stefanha@redhat.com On 13.12.2013 14:22, Kevin Wolf wrote: > This function separates filling the BlockLimits from bdrv_open(), which > allows it to call it from other operations which may change the limits > (e.g. modifications to the backing file chain or bdrv_reopen) > > Signed-off-by: Kevin Wolf > --- > block.c | 19 +++++++++++++++++++ > block/iscsi.c | 46 +++++++++++++++++++++++++++++----------------- > block/qcow2.c | 11 ++++++++++- > block/qed.c | 11 ++++++++++- > block/vmdk.c | 22 ++++++++++++++++++---- > include/block/block_int.h | 2 ++ > 6 files changed, 88 insertions(+), 23 deletions(-) Reviewed-by: Max Reitz