Re: [Qemu-devel] [PATCH for-2.10 0/4] block: Fix non-shared storage migration

2017-08-23 Thread Eric Blake
On 08/23/2017 08:42 AM, Fam Zheng wrote: > This fixes the issue reported as https://bugs.launchpad.net/bugs/1711602 > > Fam Zheng (3): > block-backend: Refactor inactivate check > block-backend: Allow more "can inactivate" cases > mirror: Mark target BB as "force allow inactivate" > > Stefa

Re: [Qemu-devel] [PATCH for-2.10 0/4] block: Fix non-shared storage migration

2017-08-23 Thread Eric Blake
On 08/23/2017 09:29 AM, Dr. David Alan Gilbert wrote: > * Fam Zheng (f...@redhat.com) wrote: >> This fixes the issue reported as https://bugs.launchpad.net/bugs/1711602 > > Yep, that fixes it in my test setup (1 of 1 passes!) As mentioned on IRC, I'll treat this as a Tested-by: tag on 4/4 in my p

Re: [Qemu-devel] [PATCH for-2.10 0/4] block: Fix non-shared storage migration

2017-08-23 Thread Dr. David Alan Gilbert
* Fam Zheng (f...@redhat.com) wrote: > This fixes the issue reported as https://bugs.launchpad.net/bugs/1711602 Yep, that fixes it in my test setup (1 of 1 passes!) Dave > Fam Zheng (3): > block-backend: Refactor inactivate check > block-backend: Allow more "can inactivate" cases > mirror:

Re: [Qemu-devel] [PATCH for-2.10 0/4] block: Fix non-shared storage migration

2017-08-23 Thread Stefan Hajnoczi
On Wed, Aug 23, 2017 at 09:42:38PM +0800, Fam Zheng wrote: > This fixes the issue reported as https://bugs.launchpad.net/bugs/1711602 > > Fam Zheng (3): > block-backend: Refactor inactivate check > block-backend: Allow more "can inactivate" cases > mirror: Mark target BB as "force allow inac

[Qemu-devel] [PATCH for-2.10 0/4] block: Fix non-shared storage migration

2017-08-23 Thread Fam Zheng
This fixes the issue reported as https://bugs.launchpad.net/bugs/1711602 Fam Zheng (3): block-backend: Refactor inactivate check block-backend: Allow more "can inactivate" cases mirror: Mark target BB as "force allow inactivate" Stefan Hajnoczi (1): block: Update open_flags after ->inacti

[Qemu-devel] [PATCH for-2.10 0/4] block: Fix non-shared storage migration

2017-08-14 Thread Fam Zheng
"nbd-server-add -w" doesn't work when we are in "-incoming defer" state: (qemu) nbd_server_add -w drive-virtio-disk0 Block node is read-only Two problems are faced: - nbd_export_new() calls bdrv_invalidate_cache() too late. - bdrv_invalidate_cache() restores qdev permission (which ar