From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:43724) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VoXqH-0005A2-KK for qemu-devel@nongnu.org; Thu, 05 Dec 2013 07:15:46 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VoXqC-0001Dn-4f for qemu-devel@nongnu.org; Thu, 05 Dec 2013 07:15:41 -0500 Received: from mail-ea0-x22d.google.com ([2a00:1450:4013:c01::22d]:33903) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VoXqB-0001Df-Hj for qemu-devel@nongnu.org; Thu, 05 Dec 2013 07:15:35 -0500 Received: by mail-ea0-f173.google.com with SMTP id o10so246748eaj.18 for ; Thu, 05 Dec 2013 04:15:34 -0800 (PST) Date: Thu, 5 Dec 2013 13:15:28 +0100 From: Stefan Hajnoczi Message-ID: <20131205121528.GA12608@stefanha-thinkpad.redhat.com> References: <1385546829-3839-1-git-send-email-pl@kamp.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1385546829-3839-1-git-send-email-pl@kamp.de> Subject: Re: [Qemu-devel] [PATCHv3 1.8 0/9] qemu-img convert optimizations List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Lieven Cc: kwolf@redhat.com, pbonzini@redhat.com, qemu-devel@nongnu.org, stefanha@redhat.com On Wed, Nov 27, 2013 at 11:07:00AM +0100, Peter Lieven wrote: > this series adds some optimizations for qemu-img during convert which > have been developed recently: > - skipping input based on get_block_status > - variable I/O buffer size > - align write requests to cluster_size > > v2->v3: > - added Paolos comments in Patch 1 > - changed the comment in patch 7 [Paolo] > - remove the patch to add sector progress output > - added a new patch to decrease the progress update interval. > > v1->v2: > - introduce opt_transfer_length in BlockLimits [Paolo] > - remove knobs for iobuffer_size and alignment and > use them unconditionally [Paolo] > - calculate I/O buffer size by BlockLimits information [Paolo] > - change the alignment patch to round down to the > last and not to the next aligned sector [Paolo] > - limit updates in the sector progress output > - new patch to increase the default for min_sparse [Paolo] > > Peter Lieven (9): > qemu-img: add support for skipping zeroes in input during convert > qemu-img: fix usage instruction for qemu-img convert > block/iscsi: set bdi->cluster_size > block: add opt_transfer_length to BlockLimits > block/iscsi: set bs->bl.opt_transfer_length > qemu-img: dynamically adjust iobuffer size during convert > qemu-img: round down request length to an aligned sector > qemu-img: increase min_sparse to 128 sectors (64kb) > qemu-img: decrease progress update interval on convert > > block/iscsi.c | 10 ++++ > include/block/block_int.h | 3 ++ > qemu-img.c | 131 +++++++++++++++++++++++++++------------------ > qemu-img.texi | 2 +- > 4 files changed, 93 insertions(+), 53 deletions(-) Merged all except patch 8/9. Thanks, applied to my block tree: https://github.com/stefanha/qemu/commits/block Stefan