Re: [Qemu-devel] [PATCH 14/20] qcow2: Factor out count_cow_clusters

2012-04-20 Thread Kevin Wolf
Am 20.04.2012 15:06, schrieb Stefan Hajnoczi: > On Fri, Apr 20, 2012 at 1:27 PM, Stefan Hajnoczi wrote: >> On Fri, Apr 20, 2012 at 9:24 AM, Kevin Wolf wrote: >>> Am 19.04.2012 23:18, schrieb Marcelo Tosatti: On Thu, Apr 19, 2012 at 05:14:20PM -0300, Marcelo Tosatti wrote: >> There is one

Re: [Qemu-devel] [PATCH 14/20] qcow2: Factor out count_cow_clusters

2012-04-20 Thread Stefan Hajnoczi
On Fri, Apr 20, 2012 at 1:27 PM, Stefan Hajnoczi wrote: > On Fri, Apr 20, 2012 at 9:24 AM, Kevin Wolf wrote: >> Am 19.04.2012 23:18, schrieb Marcelo Tosatti: >>> On Thu, Apr 19, 2012 at 05:14:20PM -0300, Marcelo Tosatti wrote: > There is one intended change in functionality in this patch, whi

Re: [Qemu-devel] [PATCH 14/20] qcow2: Factor out count_cow_clusters

2012-04-20 Thread Stefan Hajnoczi
On Fri, Apr 20, 2012 at 9:24 AM, Kevin Wolf wrote: > Am 19.04.2012 23:18, schrieb Marcelo Tosatti: >> On Thu, Apr 19, 2012 at 05:14:20PM -0300, Marcelo Tosatti wrote: There is one intended change in functionality in this patch, which is that it allocates new clusters even when it could s

Re: [Qemu-devel] [PATCH 14/20] qcow2: Factor out count_cow_clusters

2012-04-20 Thread Kevin Wolf
Am 19.04.2012 23:18, schrieb Marcelo Tosatti: > On Thu, Apr 19, 2012 at 05:14:20PM -0300, Marcelo Tosatti wrote: >>> There is one intended change in functionality in this patch, which is >>> that it allocates new clusters even when it could satisfy the first part >>> of the request with already all

Re: [Qemu-devel] [PATCH 14/20] qcow2: Factor out count_cow_clusters

2012-04-19 Thread Marcelo Tosatti
On Thu, Apr 19, 2012 at 05:14:20PM -0300, Marcelo Tosatti wrote: > > There is one intended change in functionality in this patch, which is > > that it allocates new clusters even when it could satisfy the first part > > of the request with already allocated clusters. In order to check if > > there

Re: [Qemu-devel] [PATCH 14/20] qcow2: Factor out count_cow_clusters

2012-04-19 Thread Marcelo Tosatti
On Thu, Apr 19, 2012 at 09:38:29AM +0200, Kevin Wolf wrote: > Am 19.04.2012 04:44, schrieb Marcelo Tosatti: > > On Mon, Mar 12, 2012 at 04:19:45PM +0100, Kevin Wolf wrote: > >> Signed-off-by: Kevin Wolf > >> Reviewed-by: Stefan Hajnoczi > >> --- > >> block/qcow2-cluster.c | 55 > >> ++

Re: [Qemu-devel] [PATCH 14/20] qcow2: Factor out count_cow_clusters

2012-04-19 Thread Kevin Wolf
Am 19.04.2012 04:44, schrieb Marcelo Tosatti: > On Mon, Mar 12, 2012 at 04:19:45PM +0100, Kevin Wolf wrote: >> Signed-off-by: Kevin Wolf >> Reviewed-by: Stefan Hajnoczi >> --- >> block/qcow2-cluster.c | 55 >> - >> 1 files changed, 36 insertions

Re: [Qemu-devel] [PATCH 14/20] qcow2: Factor out count_cow_clusters

2012-04-18 Thread Marcelo Tosatti
On Mon, Mar 12, 2012 at 04:19:45PM +0100, Kevin Wolf wrote: > Signed-off-by: Kevin Wolf > Reviewed-by: Stefan Hajnoczi > --- > block/qcow2-cluster.c | 55 > - > 1 files changed, 36 insertions(+), 19 deletions(-) Kevin, Autotest installed Fedor

[Qemu-devel] [PATCH 14/20] qcow2: Factor out count_cow_clusters

2012-03-12 Thread Kevin Wolf
Signed-off-by: Kevin Wolf Reviewed-by: Stefan Hajnoczi --- block/qcow2-cluster.c | 55 - 1 files changed, 36 insertions(+), 19 deletions(-) diff --git a/block/qcow2-cluster.c b/block/qcow2-cluster.c index a791bbe..903454d 100644 --- a/block/qcow