From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:35697) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VfuWd-0001HY-Vs for qemu-devel@nongnu.org; Mon, 11 Nov 2013 11:39:49 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VfuWV-0000xu-An for qemu-devel@nongnu.org; Mon, 11 Nov 2013 11:39:43 -0500 Received: from mx1.redhat.com ([209.132.183.28]:55365) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VfuWV-0000xq-2i for qemu-devel@nongnu.org; Mon, 11 Nov 2013 11:39:35 -0500 Date: Mon, 11 Nov 2013 17:39:26 +0100 From: Kevin Wolf Message-ID: <20131111163926.GB24359@dhcp-200-207.str.redhat.com> References: <1383753568-15844-1-git-send-email-charlie@ctshepherd.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1383753568-15844-1-git-send-email-charlie@ctshepherd.com> Subject: Re: [Qemu-devel] [PATCH v3 0/2] COW: Speed up writes List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Charlie Shepherd Cc: pbonzini@redhat.com, gabriel@kerneis.info, qemu-devel@nongnu.org, stefanha@gmail.com Am 06.11.2013 um 16:59 hat Charlie Shepherd geschrieben: > v3: > - Refix cow_update_bitmap and squash patches 1 & 3 together to ensuring that we only flush if > necessary, patch 1 on its own would change this causing a regression. > v2: > - Fix bit position calculations in cow_update_bitmap > - Add necessary check in cow_set_bits > > Following on from Paolo's commits 26ae980 and 276cbc7, this patchset > implements some changes he recommended earlier which I didn't previously have > time to do while on GSoC. > > Charlie Shepherd (2): > COW: Speed up writes > COW: Extend checking allocated bits to beyond one sector > > block/cow.c | 123 ++++++++++++++++++++++++++++++++++++------------------------ > 1 file changed, 75 insertions(+), 48 deletions(-) Thanks, applied to the block-next branch for 1.8. Your lines in the commit message were a bit too long, please try to have a line break after ~72 characters in future patches (I fixed it up manually for this one). Kevin