[Qemu-block] [PULL 2/7] block/nfs: cache allocated filesize for read-only files

2015-09-25 Thread Jeff Cody
From: Peter Lieven If the file is readonly its not expected to grow so save the blocking call to nfs_fstat_async and use the value saved at connection time. Also important the monitor (and thus the main loop) will not hang if block device info is queried and the NFS share is

Re: [Qemu-block] [Qemu-devel] [PATCH] block: disable I/O limits at the beginning of bdrv_close()

2015-09-25 Thread Alberto Garcia
On Fri 25 Sep 2015 04:22:26 PM CEST, Eric Blake wrote: >> Disabling I/O limits from a BDS also drains all pending throttled >> requests, so it should be done at the beginning of bdrv_close() with >> the rest of the bdrv_drain() calls before the BlockDriver is closed. > > Can this be abused? If I

[Qemu-block] [PULL 3/7] sheepdog: add reopen support

2015-09-25 Thread Jeff Cody
From: Liu Yuan With reopen supported, block-commit (and offline commit) is now supported for image files whose base image uses the Sheepdog protocol driver. Cc: qemu-de...@nongnu.org Cc: Jeff Cody Cc: Kevin Wolf Cc: Stefan

[Qemu-block] [PULL 6/7] sheepdog: use per AIOCB dirty indexes for non overlapping requests

2015-09-25 Thread Jeff Cody
From: Hitoshi Mitake In the commit 96b14ff85acf, requests for overlapping areas are serialized. However, it cannot handle a case of non overlapping requests. In such a case, min_dirty_data_idx and max_dirty_data_idx can be overwritten by the requests and invalid

[Qemu-block] [PULL 1/7] block/nfs: fix calculation of allocated file size

2015-09-25 Thread Jeff Cody
From: Peter Lieven st.st_blocks is always counted in 512 byte units. Do not use st.st_blksize as multiplicator which may be larger. Cc: qemu-sta...@nongnu.org Signed-off-by: Peter Lieven Reviewed-by: Max Reitz Reviewed-by: Jeff Cody

Re: [Qemu-block] [Qemu-devel] [PATCH] block: disable I/O limits at the beginning of bdrv_close()

2015-09-25 Thread Eric Blake
On 09/25/2015 07:41 AM, Alberto Garcia wrote: > Disabling I/O limits from a BDS also drains all pending throttled > requests, so it should be done at the beginning of bdrv_close() with > the rest of the bdrv_drain() calls before the BlockDriver is closed. Can this be abused? If I have a guest

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

2015-09-25 Thread Jeff Cody
The following changes since commit eb9d0ea063fc7bdfab76b84085602a9e48d13ec7: Merge remote-tracking branch 'remotes/pmaydell/tags/pull-target-arm-20150924' into staging (2015-09-24 01:32:11 +0100) are available in the git repository at: g...@github.com:codyprime/qemu-kvm-jtc.git

[Qemu-block] [PULL 5/7] Backup: don't do copy-on-read in before_write_notifier

2015-09-25 Thread Jeff Cody
From: Wen Congyang We will copy data in before_write_notifier to do backup. It is a nested I/O request, so we cannot do copy-on-read. The steps to reproduce it: 1. -drive copy-on-read=on,... // qemu option 2. drive_backup -f disk0 /path_to_backup.img // monitor command

[Qemu-block] [PULL 7/7] sheepdog: refine discard support

2015-09-25 Thread Jeff Cody
From: Hitoshi Mitake This patch refines discard support of the sheepdog driver. The existing discard mechanism was implemented on SD_OP_DISCARD_OBJ, which was introduced before fine grained reference counting on newer sheepdog. It doesn't care about relations of

Re: [Qemu-block] ide-test fails on PPC64 big-endian host

2015-09-25 Thread John Snow
On 09/25/2015 03:20 PM, Peter Maydell wrote: > Hi. I was looking at adding ppc64be to the set of machines I do build > tests on before merging patches, but right now "make check" fails on > this host. > > ERROR:/home/pm215/qemu/tests/ide-test.c:721:cdrom_pio_impl: assertion > failed ((data) &

Re: [Qemu-block] [PULL 0/7] Block patches

2015-09-25 Thread Peter Maydell
On 25 September 2015 at 08:50, Jeff Cody wrote: > The following changes since commit eb9d0ea063fc7bdfab76b84085602a9e48d13ec7: > > Merge remote-tracking branch > 'remotes/pmaydell/tags/pull-target-arm-20150924' into staging (2015-09-24 > 01:32:11 +0100) > > are available in

[Qemu-block] [PATCH v10 04/10] block: make bdrv_put_ref_bh_schedule() as a public API

2015-09-25 Thread Wen Congyang
Signed-off-by: Wen Congyang --- block.c | 25 + blockdev.c| 37 ++--- include/block/block.h | 1 + 3 files changed, 32 insertions(+), 31 deletions(-) diff --git a/block.c b/block.c index

[Qemu-block] [PATCH v10 03/10] Allow creating backup jobs when opening BDS

2015-09-25 Thread Wen Congyang
When opening BDS, we need to create backup jobs for image-fleecing. Signed-off-by: Wen Congyang Signed-off-by: zhanghailiang Signed-off-by: Gonglei Reviewed-by: Stefan Hajnoczi Reviewed-by:

[Qemu-block] [PATCH v10 06/10] Add new block driver interfaces to control block replication

2015-09-25 Thread Wen Congyang
Signed-off-by: Wen Congyang Signed-off-by: zhanghailiang Signed-off-by: Gonglei Cc: Luiz Capitulino Cc: Michael Roth Reviewed-by: Paolo Bonzini

[Qemu-block] [PATCH v10 07/10] quorum: implement block driver interfaces for block replication

2015-09-25 Thread Wen Congyang
Signed-off-by: Wen Congyang Signed-off-by: zhanghailiang Signed-off-by: Gonglei Reviewed-by: Alberto Garcia --- block/quorum.c | 77 ++ 1

Re: [Qemu-block] [Qemu-devel] [PATCH 11/16] Add new block driver interfaces to control block replication

2015-09-25 Thread Wen Congyang
On 09/03/2015 12:33 AM, Eric Blake wrote: > On 09/02/2015 02:51 AM, Wen Congyang wrote: >> Signed-off-by: Wen Congyang >> Signed-off-by: zhanghailiang >> Signed-off-by: Gonglei >> Cc: Luiz Capitulino

[Qemu-block] [PATCH v10 02/10] Backup: clear all bitmap when doing block checkpoint

2015-09-25 Thread Wen Congyang
Signed-off-by: Wen Congyang Signed-off-by: zhanghailiang Signed-off-by: Gonglei Reviewed-by: Jeff Cody --- block/backup.c | 14 ++ blockjob.c | 11 +++

[Qemu-block] [PATCH v10 10/10] Add a new API to start/stop replication, do checkpoint to all BDSes

2015-09-25 Thread Wen Congyang
Signed-off-by: Wen Congyang Signed-off-by: zhanghailiang Signed-off-by: Gonglei --- block.c | 83 +++ include/block/block.h | 4 +++ 2 files changed, 87

[Qemu-block] [PATCH v10 09/10] support replication driver in blockdev-add

2015-09-25 Thread Wen Congyang
Signed-off-by: Wen Congyang --- qapi/block-core.json | 21 ++--- 1 file changed, 18 insertions(+), 3 deletions(-) diff --git a/qapi/block-core.json b/qapi/block-core.json index d5a177b..0907a72 100644 --- a/qapi/block-core.json +++ b/qapi/block-core.json @@

[Qemu-block] [PATCH v10 00/10] Block replication for continuous checkpoints

2015-09-25 Thread Wen Congyang
Block replication is a very important feature which is used for continuous checkpoints(for example: COLO). You can the detailed information about block replication from here: http://wiki.qemu.org/Features/BlockReplication Usage: Please refer to docs/block-replication.txt This patch series is

[Qemu-block] [PATCH v10 05/10] docs: block replication's description

2015-09-25 Thread Wen Congyang
Signed-off-by: Wen Congyang Signed-off-by: Yang Hongyang Signed-off-by: zhanghailiang Signed-off-by: Gonglei --- docs/block-replication.txt | 259 +

[Qemu-block] [PATCH v10 01/10] allow writing to the backing file

2015-09-25 Thread Wen Congyang
For block replication, we have such backing chain: secondary disk <-- hidden disk <-- active disk secondary disk is top BDS(use bacing reference), so it can be opened in read-write mode. But hidden disk is read only, and we need to write to hidden disk(backup job will write data to it). TODO:

[Qemu-block] [PATCH] block: disable I/O limits at the beginning of bdrv_close()

2015-09-25 Thread Alberto Garcia
Disabling I/O limits from a BDS also drains all pending throttled requests, so it should be done at the beginning of bdrv_close() with the rest of the bdrv_drain() calls before the BlockDriver is closed. Signed-off-by: Alberto Garcia --- block.c | 11 ++- 1 file

Re: [Qemu-block] [PATCH 05/16] block: Convert bs->file to BdrvChild

2015-09-25 Thread Alberto Garcia
On Thu 17 Sep 2015 03:48:09 PM CEST, Kevin Wolf wrote: > @@ -1929,6 +1925,11 @@ void bdrv_close(BlockDriverState *bs) > bdrv_unref(backing_hd); > } > > +if (bs->file != NULL) { > +bdrv_unref(bs->file->bs); > +bs->file =