[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v6 14/15] block: Rewrite bdrv_close_all()
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH v6 14/15] block: Rewrite bdrv_close_all() |
Date: |
Thu, 5 Nov 2015 18:40:53 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
On 05/11/2015 18:37, Max Reitz wrote:
> $ x86_64-softmmu/qemu-system-x86_64 -qmp stdio \ -drive
> if=none,file=test.qcow2,id=drive0,node-name=node0 {"QMP":
> {"version": {"qemu": {"micro": 50, "minor": 4, "major": 2},
> "package": ""}, "capabilities": []}}
> {'execute':'qmp_capabilities'} {"return": {}}
> {'execute':'drive-backup','arguments':{'device':'drive0','target':'out.qcow2','format':'qcow2','sync':'none'}}
>
>
Formatting 'out.qcow2', fmt=qcow2 size=67108864 backing_file=test.qcow2
> backing_fmt=qcow2 encryption=off cluster_size=65536
> lazy_refcounts=off refcount_bits=16 {"return": {}}
> {'execute':'eject','arguments':{'device':'drive0'}} {"error":
> {"class": "GenericError", "desc": "Node 'drive0' is busy: block
> device is in use by block job: backup"}}
>
> So... Nobody does it, but apparently nobody has to because you
> cannot eject (to be more specific: use blockdev-remove-medium) a
> BDS from a BB while a block job is running on the BDS.
If you test it with all jobs, then it's okay. It's a regression, but
not introduced by your patch and apparently nobody noticed.
Even if nobody noticed, I wonder if this "Node 'foo' is busy" kind of
error deserves its own ErrorClass. Eric, what do you think?
Paolo
[Qemu-devel] [PATCH v6 15/15] iotests: Add test for multiple BB on BDS tree, Max Reitz, 2015/11/04
Re: [Qemu-devel] [PATCH v6 for-2.6 00/15] block: Rework bdrv_close_all(), Kevin Wolf, 2015/11/09