Re: [Qemu-block] [Qemu-devel] [PATCH 1/2] block: sync bdrv_co_get_block_status_above() with bdrv_is_allocated_above()

2016-09-12 Thread Vladimir Sementsov-Ogievskiy
On 09.09.2016 15:31, Denis V. Lunev wrote: They should work very similar, covering same areas if backing store is shorter than the image. This change is necessary for the followup patch switching to bdrv_get_block_status_above() in mirror to avoid assert in check_block. Signed-off-by: Denis V.

Re: [Qemu-block] bdrv_reopen() and backing images

2016-09-12 Thread Alberto Garcia
On Fri 09 Sep 2016 07:17:22 PM CEST, Kevin Wolf wrote: >> So the other alternative is to add "read-only" as an option and keep >> it in sync with the BDRV_O_RDWR flag. This is similar to what commit >> 91a097e7478940483 did with the cache options. >> >> There's a problem here, mostly related to

Re: [Qemu-block] [PATCH v24 11/12] support replication driver in blockdev-add

2016-09-12 Thread Stefan Hajnoczi
On Mon, Aug 15, 2016 at 05:32:19PM +0800, Changlong Xie wrote: > On 08/15/2016 04:37 PM, Kevin Wolf wrote: > > Am 15.08.2016 um 03:49 hat Changlong Xie geschrieben: > > > On 08/09/2016 05:08 PM, Kevin Wolf wrote: > > > > Am 27.07.2016 um 09:01 hat Changlong Xie geschrieben: > > > > > From: Wen

Re: [Qemu-block] [Qemu-devel] [PATCH 1/2] block: sync bdrv_co_get_block_status_above() with bdrv_is_allocated_above()

2016-09-12 Thread Roman Kagan
On Fri, Sep 09, 2016 at 03:31:47PM +0300, Denis V. Lunev wrote: > They should work very similar, covering same areas if backing store is > shorter than the image. This change is necessary for the followup patch > switching to bdrv_get_block_status_above() in mirror to avoid assert > in

Re: [Qemu-block] [Qemu-devel] [PATCH 2/2] mirror: fix improperly filled copy_bitmap for mirror block job

2016-09-12 Thread Eric Blake
On 09/09/2016 07:31 AM, Denis V. Lunev wrote: > bdrv_is_allocated_above() returns true in the case even for completel s/completel/completely/ > zeroed areas as BDRV_BLOCK_ALLOCATED flag is set in both cases. > > The patch stops using bdrv_is_allocated_above() wrapper and switches to >

Re: [Qemu-block] [Qemu-devel] [PATCH 2/2] mirror: fix improperly filled copy_bitmap for mirror block job

2016-09-12 Thread Jeff Cody
On Fri, Sep 09, 2016 at 03:31:48PM +0300, Denis V. Lunev wrote: > bdrv_is_allocated_above() returns true in the case even for completel > zeroed areas as BDRV_BLOCK_ALLOCATED flag is set in both cases. > > The patch stops using bdrv_is_allocated_above() wrapper and switches to >

[Qemu-block] [PULL 1/2] block/gluster: add support to choose libgfapi logfile

2016-09-12 Thread Jeff Cody
From: Prasanna Kumar Kalever currently all the libgfapi logs defaults to '/dev/stderr' as it was hardcoded in a call to glfs logging api. When the debug level is chosen to DEBUG/TRACE, gfapi logs will be huge and fill/overflow the console view. This patch provides a

[Qemu-block] [PULL 2/2] qapi/block-core: add doc describing GlusterServer vs. SocketAddress

2016-09-12 Thread Jeff Cody
From: Prasanna Kumar Kalever Added documentation describing relation between GlusterServer and SocketAddress qapi schemas. Thanks to Markus Armbruster Reviewed-by: Markus Armbruster Reviewed-by: Jeff Cody

[Qemu-block] [PULL 0/2] Block patches

2016-09-12 Thread Jeff Cody
The following changes since commit 7263da78045dc91cc207f350911efe4259e99b3c: Merge remote-tracking branch 'remotes/mcayland/tags/qemu-openbios-signed' into staging (2016-09-12 15:09:47 +0100) are available in the git repository at: g...@github.com:codyprime/qemu-kvm-jtc.git

Re: [Qemu-block] bdrv_reopen() and backing images

2016-09-12 Thread Kevin Wolf
Am 12.09.2016 um 15:48 hat Alberto Garcia geschrieben: > On Fri 09 Sep 2016 07:17:22 PM CEST, Kevin Wolf wrote: > > >> So the other alternative is to add "read-only" as an option and keep > >> it in sync with the BDRV_O_RDWR flag. This is similar to what commit > >> 91a097e7478940483 did with the

Re: [Qemu-block] [PATCH v24 00/12] Block replication for continuous checkpoints

2016-09-12 Thread Stefan Hajnoczi
On Mon, Aug 08, 2016 at 03:50:27PM +0100, Stefan Hajnoczi wrote: > On Wed, Jul 27, 2016 at 03:01:41PM +0800, Changlong Xie wrote: > > Block replication is a very important feature which is used for > > continuous checkpoints(for example: COLO). > > > > You can get the detailed information about