Re: [Qemu-block] [PATCH 1/1] block/raw-posix: avoid bogus fixup for cylinders on DASD disks

2016-01-07 Thread Christian Borntraeger
On 12/14/2015 06:49 PM, Cornelia Huck wrote: > On Mon, 14 Dec 2015 16:41:19 +0100 > Christian Borntraeger wrote: > >> large volume DASD that have > 64k cylinders do claim to have >> 0xFFFE cylinders as special value in the old 16 bit field. We >> want to pass this "token" along to the guest, inst

Re: [Qemu-block] [Qemu-devel] [PATCH] send readcapacity10 when readcapacity16 failed

2016-01-07 Thread Paolo Bonzini
On 06/01/2016 18:57, John Snow wrote: > Ronnie: Thanks for the explanation! > > Zhu: In light of this, can the patch be reworked slightly to explicitly > check *why* READCAPACITY16 failed and only attempt the READCAPACITY10 as > a fallback if it receives INVALID_OPCODE? > > If it fails for any

Re: [Qemu-block] [PATCH] send readcapacity10 when readcapacity16 failed

2016-01-07 Thread Paolo Bonzini
On 29/12/2015 04:32, Zhu Lingshan wrote: > When play with Dell MD3000 target, for sure it > is a TYPE_DISK, but readcapacity16 would fail. > Then we find that readcapacity10 succeeded. It > looks like the target just support readcapacity10 > even through it is a TYPE_DISK or have some > TYPE_ROM

Re: [Qemu-block] [PATCH v5 0/6] i386: expose floppy-related objects in SSDT

2016-01-07 Thread Michael S. Tsirkin
On Wed, Jan 06, 2016 at 03:04:40PM +0100, Igor Mammedov wrote: > On Wed, 30 Dec 2015 23:11:50 +0300 > Roman Kagan wrote: > > > Windows on UEFI systems is only capable of detecting the presence and > > the type of floppy drives via corresponding ACPI objects. > > > > Those objects are added in pa

Re: [Qemu-block] [PATCH v5 0/6] i386: expose floppy-related objects in SSDT

2016-01-07 Thread Michael S. Tsirkin
On Thu, Jan 07, 2016 at 12:56:09PM +0200, Michael S. Tsirkin wrote: > On Wed, Jan 06, 2016 at 03:04:40PM +0100, Igor Mammedov wrote: > > On Wed, 30 Dec 2015 23:11:50 +0300 > > Roman Kagan wrote: > > > > > Windows on UEFI systems is only capable of detecting the presence and > > > the type of flop

[Qemu-block] [PATCH 0/2] nbd: changes around allocating buffers for requests

2016-01-07 Thread Paolo Bonzini
Notably, patch 2 introduces blk_try_blockalign. If it is acked by the block layer maintainers I can submit the patch myself for inclusion. Paolo Paolo Bonzini (2): nbd: do not check request length except for reads and writes nbd: do not exit on failed memory allocation block/block-backend.

[Qemu-block] [PATCH 2/2] nbd: do not exit on failed memory allocation

2016-01-07 Thread Paolo Bonzini
The amount of memory allocated in nbd_co_receive_request is driven by the NBD client (possibly a virtual machine). Parallel I/O can cause the server to allocate a large amount of memory; check for failures and return ENOMEM in that case. Signed-off-by: Paolo Bonzini --- block/block-backend.c

Re: [Qemu-block] [PATCH v5 0/6] i386: expose floppy-related objects in SSDT

2016-01-07 Thread Igor Mammedov
On Thu, 7 Jan 2016 12:56:58 +0200 "Michael S. Tsirkin" wrote: > On Thu, Jan 07, 2016 at 12:56:09PM +0200, Michael S. Tsirkin wrote: > > On Wed, Jan 06, 2016 at 03:04:40PM +0100, Igor Mammedov wrote: > > > On Wed, 30 Dec 2015 23:11:50 +0300 > > > Roman Kagan wrote: > > > > > > > Windows on U

Re: [Qemu-block] [Qemu-devel] [PULL v2 10/40] blockdev: Implement change with basic operations

2016-01-07 Thread Peter Maydell
On 10 November 2015 at 14:09, Kevin Wolf wrote: > From: Max Reitz > > Implement 'change' on block devices by calling blockdev-open-tray, > blockdev-remove-medium, blockdev-insert-medium (a variation of that > which does not need a node-name) and blockdev-close-tray. > > Signed-off-by: Max Reitz

Re: [Qemu-block] [Qemu-devel] [PATCH 10/13] block: Assert that bdrv_release_dirty_bitmap succeeded

2016-01-07 Thread John Snow
On 01/04/2016 05:27 AM, Fam Zheng wrote: > This makes sure we don't leak a dirty bitmap in any case. > > Signed-off-by: Fam Zheng > --- > block/dirty-bitmap.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/block/dirty-bitmap.c b/block/dirty-bitmap.c > index 9cac794..60ee965 100644 >

Re: [Qemu-block] [Qemu-devel] [PULL v2 10/40] blockdev: Implement change with basic operations

2016-01-07 Thread Peter Maydell
On 7 January 2016 at 19:37, Max Reitz wrote: > Compare floppy disks, for which we now have a "virtual" tray status: > Whenever a medium is inserted, the "tray" is considered closed. > Otherwise, it is open. This works pretty much like a physical tray would > work; whenever the tray is closed, you

Re: [Qemu-block] [Qemu-devel] [PATCH 11/13] hbitmap: serialization

2016-01-07 Thread John Snow
On 01/04/2016 05:27 AM, Fam Zheng wrote: > From: Vladimir Sementsov-Ogievskiy > > Functions to serialize / deserialize(restore) HBitmap. HBitmap should be > saved to linear sequence of bits independently of endianness and bitmap > array element (unsigned long) size. Therefore Little Endian is c

Re: [Qemu-block] [PATCH 1/2] nbd: do not check request length except for reads and writes

2016-01-07 Thread Max Reitz
On 07.01.2016 14:44, Paolo Bonzini wrote: > Only reads and writes need to allocate memory correspondent to the > request length. Other requests can be sent to the storage without > allocating any memory, and thus any request length is acceptable. > > Reported-by: Sitsofe Wheeler > Cc: qemu-block

Re: [Qemu-block] [PATCH 2/2] nbd: do not exit on failed memory allocation

2016-01-07 Thread Max Reitz
On 07.01.2016 14:44, Paolo Bonzini wrote: > The amount of memory allocated in nbd_co_receive_request is driven by the > NBD client (possibly a virtual machine). Parallel I/O can cause the > server to allocate a large amount of memory; check for failures and > return ENOMEM in that case. > > Signe

Re: [Qemu-block] [Qemu-devel] [PATCH 00/13] Dirty bitmap changes for migration/persistence work

2016-01-07 Thread John Snow
On 01/04/2016 05:27 AM, Fam Zheng wrote: > Two major features are added to block dirty bitmap (and underlying HBitmap) in > this series: meta bitmap and serialization, together with all other supportive > patches. > > Both operations are common in dirty bitmap migration and persistence: they >

Re: [Qemu-block] [Qemu-devel] [PULL v2 10/40] blockdev: Implement change with basic operations

2016-01-07 Thread Peter Maydell
On 7 January 2016 at 20:14, Max Reitz wrote: > On 07.01.2016 20:56, Peter Maydell wrote: >> It looks like sd.c is the only one which implements a change_media_cb >> but no is_tray_open, but it would be nice if we could implement this >> in the default blk_dev_is_tray_open() method rather than in t

Re: [Qemu-block] [Qemu-devel] [PULL v2 10/40] blockdev: Implement change with basic operations

2016-01-07 Thread Max Reitz
On 07.01.2016 22:42, Peter Maydell wrote: > On 7 January 2016 at 20:14, Max Reitz wrote: >> On 07.01.2016 20:56, Peter Maydell wrote: >>> It looks like sd.c is the only one which implements a change_media_cb >>> but no is_tray_open, but it would be nice if we could implement this >>> in the defaul

Re: [Qemu-block] [Qemu-devel] [PULL v2 10/40] blockdev: Implement change with basic operations

2016-01-07 Thread Peter Maydell
On 7 January 2016 at 21:57, Max Reitz wrote: > On 07.01.2016 22:42, Peter Maydell wrote: >> Well, previously sd.c didn't need to have any state for this >> to all work right (or indeed care about implementing a fake >> tray status for a device that doesn't have a tray), so it seems >> odd that we

Re: [Qemu-block] [Qemu-devel] [PULL v2 10/40] blockdev: Implement change with basic operations

2016-01-07 Thread Max Reitz
On 07.01.2016 23:19, Peter Maydell wrote: > On 7 January 2016 at 21:57, Max Reitz wrote: >> On 07.01.2016 22:42, Peter Maydell wrote: >>> Well, previously sd.c didn't need to have any state for this >>> to all work right (or indeed care about implementing a fake >>> tray status for a device that d

[Qemu-block] [PULL 02/21] qemu-iotests: make check-block.sh work on out-of-tree builds

2016-01-07 Thread Max Reitz
From: Paolo Bonzini Since check-block.sh, the "check" script has learnt to find the source path. On the other hand, it expects common.env to be in the build tree (both changes made in commit 76c7560, "configure: Enable out-of-tree iotests", 2014-05-24). So, it is wrong to invoke "check" from th

[Qemu-block] [PULL 01/21] iotests: 086: Add raw format

2016-01-07 Thread Max Reitz
From: Fam Zheng Raw is as qualified as qcow2 for this test case, add it for more coverage. Signed-off-by: Fam Zheng Message-id: 1450851979-15580-1-git-send-email-f...@redhat.com Signed-off-by: Max Reitz --- tests/qemu-iotests/086 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --g

[Qemu-block] [PULL 00/21] Block layer patches

2016-01-07 Thread Max Reitz
Kevin is on PTO this week, so I am filling in for him. The following changes since commit a7e00e2536941a6e570b45b7ab4afec4505ff67e: petalogix-ml605: Set the MicroBlaze CPU version to 8.10.a (2016-01-07 14:57:26 +0100) are available in the git repository at: git://github.com/XanClic/qemu.g

[Qemu-block] [PULL 03/21] block: use drained section in bdrv_close

2016-01-07 Thread Max Reitz
From: Paolo Bonzini bdrv_close is used when ejecting a medium. Use a drained section to ensure that all I/O goes to either the old medium or the bitbucket. Signed-off-by: Paolo Bonzini Message-id: 1450867706-19860-2-git-send-email-pbonz...@redhat.com Signed-off-by: Max Reitz --- block.c | 4

[Qemu-block] [PULL 11/21] iotests: 037: Use TEST_IMG override instead of "mv"

2016-01-07 Thread Max Reitz
From: Fam Zheng Signed-off-by: Fam Zheng Reviewed-by: Max Reitz Message-id: 1450752561-9300-8-git-send-email-f...@redhat.com Signed-off-by: Max Reitz --- tests/qemu-iotests/037 | 5 - tests/qemu-iotests/037.out | 2 +- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/tes

[Qemu-block] [PULL 09/21] iotests: 028: Use TEST_IMG override instead of "mv"

2016-01-07 Thread Max Reitz
From: Fam Zheng Signed-off-by: Fam Zheng Reviewed-by: Max Reitz Message-id: 1450752561-9300-6-git-send-email-f...@redhat.com Signed-off-by: Max Reitz --- tests/qemu-iotests/028 | 5 - tests/qemu-iotests/028.out | 2 +- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/tes

[Qemu-block] [PULL 04/21] block/qapi: Clear err for further error

2016-01-07 Thread Max Reitz
From: Fam Zheng Since a5002d5 (block/qapi: allow best-effort query) we don't return at this error, however err must be cleared before passing to bdrv_query_snapshot_info_list below, as required by error API. Signed-off-by: Fam Zheng Message-id: 1450779107-26765-1-git-send-email-f...@redhat.com

[Qemu-block] [PULL 07/21] iotests: 020: Use TEST_IMG override instead of "mv"

2016-01-07 Thread Max Reitz
From: Fam Zheng Signed-off-by: Fam Zheng Reviewed-by: Max Reitz Message-id: 1450752561-9300-4-git-send-email-f...@redhat.com Signed-off-by: Max Reitz --- tests/qemu-iotests/020 | 7 +-- tests/qemu-iotests/020.out | 2 +- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/t

[Qemu-block] [PULL 05/21] iotests: 018: Use TEST_IMG override instead of "mv"

2016-01-07 Thread Max Reitz
From: Fam Zheng Signed-off-by: Fam Zheng Reviewed-by: Max Reitz Message-id: 1450752561-9300-2-git-send-email-f...@redhat.com Signed-off-by: Max Reitz --- tests/qemu-iotests/018 | 8 tests/qemu-iotests/018.out | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/

[Qemu-block] [PULL 16/21] qemu-iotests: s390x: fix test 051

2016-01-07 Thread Max Reitz
From: Bo Tu Replace the remaining "-drive file..." by "-drive file=...,if=none,id=$device_id", then x86 and s390x can get the common output. "if=ide, if=floppy, if=scsi" are not supported by s390x, so these test cases are not executed for s390x platform. Signed-off-by: Bo Tu Message-id: 1451885

[Qemu-block] [PULL 17/21] block: Rename BLOCK_OP_TYPE_MIRROR to BLOCK_OP_TYPE_MIRROR_SOURCE

2016-01-07 Thread Max Reitz
From: Fam Zheng It's necessary to distinguish source and target before we can add blockdev-mirror, because we would want a concrete type of operation to check on target bs before starting. Signed-off-by: Fam Zheng Reviewed-by: Max Reitz Message-id: 1450932306-13717-2-git-send-email-f...@redhat

[Qemu-block] [PULL 06/21] iotests: 019: Use TEST_IMG override instead of "mv"

2016-01-07 Thread Max Reitz
From: Fam Zheng Signed-off-by: Fam Zheng Reviewed-by: Max Reitz Message-id: 1450752561-9300-3-git-send-email-f...@redhat.com Signed-off-by: Max Reitz --- tests/qemu-iotests/019 | 13 +++-- tests/qemu-iotests/019.out | 4 ++-- 2 files changed, 9 insertions(+), 8 deletions(-) diff

[Qemu-block] [PULL 12/21] iotests: 038: Use TEST_IMG override instead of "mv"

2016-01-07 Thread Max Reitz
From: Fam Zheng Signed-off-by: Fam Zheng Reviewed-by: Max Reitz Message-id: 1450752561-9300-9-git-send-email-f...@redhat.com Signed-off-by: Max Reitz --- tests/qemu-iotests/038 | 5 - tests/qemu-iotests/038.out | 2 +- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/tes

[Qemu-block] [PULL 18/21] block: Extract blockdev part of qmp_drive_mirror

2016-01-07 Thread Max Reitz
From: Fam Zheng This is the part that will be reused by blockdev-mirror. Signed-off-by: Fam Zheng Reviewed-by: Max Reitz Message-id: 1450932306-13717-3-git-send-email-f...@redhat.com Signed-off-by: Max Reitz --- blockdev.c | 114 ++---

[Qemu-block] [PULL 10/21] iotests: 034: Use TEST_IMG override instead of "mv"

2016-01-07 Thread Max Reitz
From: Fam Zheng Signed-off-by: Fam Zheng Reviewed-by: Max Reitz Message-id: 1450752561-9300-7-git-send-email-f...@redhat.com Signed-off-by: Max Reitz --- tests/qemu-iotests/034 | 6 +- tests/qemu-iotests/034.out | 2 +- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/te

[Qemu-block] [PULL 08/21] iotests: 024: Use TEST_IMG override instead of "mv"

2016-01-07 Thread Max Reitz
From: Fam Zheng Signed-off-by: Fam Zheng Reviewed-by: Max Reitz Message-id: 1450752561-9300-5-git-send-email-f...@redhat.com Signed-off-by: Max Reitz --- tests/qemu-iotests/024 | 10 -- tests/qemu-iotests/024.out | 4 ++-- 2 files changed, 10 insertions(+), 4 deletions(-) diff -

[Qemu-block] [PULL 20/21] qmp: Add blockdev-mirror command

2016-01-07 Thread Max Reitz
From: Fam Zheng This will start a mirror job from a named device to another named device, its relation with drive-mirror is similar with blockdev-backup to drive-backup. In blockdev-mirror, the target node should be prepared by blockdev-add, which will be responsible for assigning a name to the

[Qemu-block] [PULL 13/21] iotests: 050: Use TEST_IMG override instead of "mv"

2016-01-07 Thread Max Reitz
From: Fam Zheng Signed-off-by: Fam Zheng Reviewed-by: Max Reitz Message-id: 1450752561-9300-10-git-send-email-f...@redhat.com Signed-off-by: Max Reitz --- tests/qemu-iotests/050 | 9 +++-- tests/qemu-iotests/050.out | 4 ++-- 2 files changed, 9 insertions(+), 4 deletions(-) diff --gi

[Qemu-block] [PULL 19/21] block: Add check on mirror target

2016-01-07 Thread Max Reitz
From: Fam Zheng Signed-off-by: Fam Zheng Reviewed-by: Max Reitz Message-id: 1450932306-13717-4-git-send-email-f...@redhat.com Signed-off-by: Max Reitz --- blockdev.c| 3 +++ include/block/block.h | 1 + 2 files changed, 4 insertions(+) diff --git a/blockdev.c b/blockdev.c index 2

[Qemu-block] [PULL 14/21] iotests: 095: Use TEST_IMG override instead of "mv"

2016-01-07 Thread Max Reitz
From: Fam Zheng Signed-off-by: Fam Zheng Message-id: 1450752561-9300-11-git-send-email-f...@redhat.com Signed-off-by: Max Reitz --- tests/qemu-iotests/095 | 6 ++ tests/qemu-iotests/095.out | 4 ++-- 2 files changed, 4 insertions(+), 6 deletions(-) diff --git a/tests/qemu-iotests/095

[Qemu-block] [PULL 15/21] iotests: 095: Filter _img_info output

2016-01-07 Thread Max Reitz
From: Fam Zheng Signed-off-by: Fam Zheng Message-id: 1450752561-9300-12-git-send-email-f...@redhat.com Signed-off-by: Max Reitz --- tests/qemu-iotests/095 | 4 ++-- tests/qemu-iotests/095.out | 2 -- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/tests/qemu-iotests/095 b/te

[Qemu-block] [PULL 21/21] iotests: Add test cases for blockdev-mirror

2016-01-07 Thread Max Reitz
From: Fam Zheng Signed-off-by: Fam Zheng Reviewed-by: Max Reitz Message-id: 1450932306-13717-6-git-send-email-f...@redhat.com Signed-off-by: Max Reitz --- tests/qemu-iotests/041 | 100 ++--- tests/qemu-iotests/041.out | 4 +- 2 files changed, 79 i

Re: [Qemu-block] [Qemu-devel] [PATCH 00/13] Dirty bitmap changes for migration/persistence work

2016-01-07 Thread Fam Zheng
On Thu, 01/07 16:32, John Snow wrote: > > > On 01/04/2016 05:27 AM, Fam Zheng wrote: > > Two major features are added to block dirty bitmap (and underlying HBitmap) > > in > > this series: meta bitmap and serialization, together with all other > > supportive > > patches. > > > > Both operation

[Qemu-block] [PATCH v6 01/15] block: Add "file" output parameter to block status query functions

2016-01-07 Thread Fam Zheng
The added parameter can be used to return the BDS pointer which the valid offset is referring to. Its value should be ignored unless BDRV_BLOCK_OFFSET_VALID in ret is set. Until block drivers fill in the right value, let's clear it explicitly right before calling .bdrv_get_block_status. The "bs->

[Qemu-block] [PATCH v6 03/15] qcow2: Assign bs->file->bs to file in qcow2_co_get_block_status

2016-01-07 Thread Fam Zheng
Reviewed-by: Stefan Hajnoczi Signed-off-by: Fam Zheng --- block/qcow2.c | 1 + 1 file changed, 1 insertion(+) diff --git a/block/qcow2.c b/block/qcow2.c index 7096a29..da74eb7 100644 --- a/block/qcow2.c +++ b/block/qcow2.c @@ -1348,6 +1348,7 @@ static int64_t coroutine_fn qcow2_co_get_block_st

[Qemu-block] [PATCH v6 02/15] qcow: Assign bs->file->bs to file in qcow_co_get_block_status

2016-01-07 Thread Fam Zheng
Reviewed-by: Eric Blake Reviewed-by: Stefan Hajnoczi Signed-off-by: Fam Zheng --- block/qcow.c | 1 + 1 file changed, 1 insertion(+) diff --git a/block/qcow.c b/block/qcow.c index 558f443..b59383f 100644 --- a/block/qcow.c +++ b/block/qcow.c @@ -509,6 +509,7 @@ static int64_t coroutine_fn qco

[Qemu-block] [PATCH v6 04/15] raw: Assign bs to file in raw_co_get_block_status

2016-01-07 Thread Fam Zheng
Reviewed-by: Eric Blake Reviewed-by: Stefan Hajnoczi Signed-off-by: Fam Zheng --- block/raw-posix.c | 1 + block/raw_bsd.c | 1 + 2 files changed, 2 insertions(+) diff --git a/block/raw-posix.c b/block/raw-posix.c index 6fc0b71..344272f 100644 --- a/block/raw-posix.c +++ b/block/raw-posix.c

[Qemu-block] [PATCH v6 00/15] qemu-img map: Allow driver to return file of the allocated block

2016-01-07 Thread Fam Zheng
v6: Two more changes as suggested by Max: - Fix comments of BDRV_BLOCK_OFFSET_VALID and BDRV_BLOCK_DATA; - Really keep 102 working; - Add rev-by in patch 11. v5: Address Max's review comments: - "It's" -> "Its" in commit message in patch 1; - Retain the "bs->file" condition to

[Qemu-block] [PATCH v6 11/15] vmdk: Return extent's file in bdrv_get_block_status

2016-01-07 Thread Fam Zheng
Signed-off-by: Fam Zheng Reviewed-by: Max Reitz --- block/vmdk.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/block/vmdk.c b/block/vmdk.c index f5a56fd..59e1ffe 100644 --- a/block/vmdk.c +++ b/block/vmdk.c @@ -1265,6 +1265,7 @@ static int64_t coroutine_fn vmdk_

[Qemu-block] [PATCH v6 14/15] qemu-img: Make MapEntry a QAPI struct

2016-01-07 Thread Fam Zheng
The "flags" bit mask is expanded to two booleans, "data" and "zero"; "bs" is replaced with "filename" string. Refactor the merge conditions in img_map() into entry_mergeable(). Reviewed-by: Eric Blake Reviewed-by: Stefan Hajnoczi Signed-off-by: Fam Zheng --- qapi/block-core.json | 27

[Qemu-block] [PATCH v6 06/15] parallels: Assign bs->file->bs to file in parallels_co_get_block_status

2016-01-07 Thread Fam Zheng
Reviewed-by: Eric Blake Reviewed-by: Stefan Hajnoczi Signed-off-by: Fam Zheng --- block/parallels.c | 1 + 1 file changed, 1 insertion(+) diff --git a/block/parallels.c b/block/parallels.c index d83246b..129668b 100644 --- a/block/parallels.c +++ b/block/parallels.c @@ -273,6 +273,7 @@ static

[Qemu-block] [PATCH v6 05/15] iscsi: Assign bs to file in iscsi_co_get_block_status

2016-01-07 Thread Fam Zheng
Reviewed-by: Eric Blake Reviewed-by: Stefan Hajnoczi Signed-off-by: Fam Zheng --- block/iscsi.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/block/iscsi.c b/block/iscsi.c index 2d1e230..8c7f1b3 100644 --- a/block/iscsi.c +++ b/block/iscsi.c @@ -625,6 +625,9 @@ out: if (iTask.task

[Qemu-block] [PATCH v6 07/15] qed: Assign bs->file->bs to file in bdrv_qed_co_get_block_status

2016-01-07 Thread Fam Zheng
Reviewed-by: Eric Blake Reviewed-by: Stefan Hajnoczi Signed-off-by: Fam Zheng --- block/qed.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/block/qed.c b/block/qed.c index a6bbd8b..03af9c1 100644 --- a/block/qed.c +++ b/block/qed.c @@ -692,6 +692,7 @@ typedef struct { uint64_t pos

[Qemu-block] [PATCH v6 09/15] vdi: Assign bs->file->bs to file in vdi_co_get_block_status

2016-01-07 Thread Fam Zheng
Reviewed-by: Stefan Hajnoczi Signed-off-by: Fam Zheng --- block/vdi.c | 1 + 1 file changed, 1 insertion(+) diff --git a/block/vdi.c b/block/vdi.c index 2199fd3..6b1a57b 100644 --- a/block/vdi.c +++ b/block/vdi.c @@ -550,6 +550,7 @@ static int64_t coroutine_fn vdi_co_get_block_status(BlockDriv

[Qemu-block] [PATCH v6 13/15] qemu-img: In "map", use the returned "file" from bdrv_get_block_status

2016-01-07 Thread Fam Zheng
Now all drivers should return a correct "file", we can make use of it, even with the recursion into backing chain above. Reviewed-by: Eric Blake Reviewed-by: Stefan Hajnoczi Signed-off-by: Fam Zheng --- qemu-img.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/qemu-img.c b

[Qemu-block] [PATCH v6 15/15] iotests: Add "qemu-img map" test for VMDK extents

2016-01-07 Thread Fam Zheng
Reviewed-by: Eric Blake Reviewed-by: Stefan Hajnoczi Signed-off-by: Fam Zheng --- tests/qemu-iotests/059 | 10 ++ tests/qemu-iotests/059.out | 38 ++ 2 files changed, 48 insertions(+) diff --git a/tests/qemu-iotests/059 b/tests/qemu-iotests/059 i

[Qemu-block] [PATCH v6 10/15] vpc: Assign bs->file->bs to file in vpc_co_get_block_status

2016-01-07 Thread Fam Zheng
Reviewed-by: Stefan Hajnoczi Signed-off-by: Fam Zheng --- block/vpc.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/block/vpc.c b/block/vpc.c index 912f5d0..412ff41 100644 --- a/block/vpc.c +++ b/block/vpc.c @@ -588,6 +588,7 @@ static int64_t coroutine_fn vpc_co_get_block_status(BlockDr

[Qemu-block] [PATCH v6 12/15] block: Use returned *file in bdrv_co_get_block_status

2016-01-07 Thread Fam Zheng
Now that all drivers return the right "file" pointer, we can use it. Signed-off-by: Fam Zheng --- block/io.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/block/io.c b/block/io.c index c3bb7c5..1ca4e61 100644 --- a/block/io.c +++ b/block/io.c @@ -1550,13 +1550,13 @@ sta

[Qemu-block] [PATCH v6 08/15] sheepdog: Assign bs to file in sd_co_get_block_status

2016-01-07 Thread Fam Zheng
Reviewed-by: Stefan Hajnoczi Signed-off-by: Fam Zheng --- block/sheepdog.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/block/sheepdog.c b/block/sheepdog.c index 0f6789e..d5e7ff8 100644 --- a/block/sheepdog.c +++ b/block/sheepdog.c @@ -2740,6 +2740,9 @@ sd_co_get_block_status(BlockDriv

Re: [Qemu-block] [Qemu-devel] [PATCH 02/13] typedefs: Add BdrvDirtyBitmap and HBitmapIter

2016-01-07 Thread Fam Zheng
On Tue, 01/05 17:14, John Snow wrote: > > > On 01/04/2016 05:27 AM, Fam Zheng wrote: > > Following patches to refactor and move block dirty bitmap code could use > > this. > > > > Signed-off-by: Fam Zheng > > --- > > include/qemu/typedefs.h | 2 ++ > > 1 file changed, 2 insertions(+) > > > >