[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 01/10] block: Auto-generate node_names for ea
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v5 01/10] block: Auto-generate node_names for each BDS entry |
Date: |
Mon, 16 Jun 2014 08:15:54 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.6.0 |
On 06/16/2014 08:10 AM, Benoît Canet wrote:
>> The prefix is to aid in identifying it as a qemu-generated name, the
>> numeric portion is to guarantee uniqueness in a given qemu session, and
>> the random characters are to further avoid any accidental collisions
>> with user-specified node-names.
>
> if the __qemu## determined a different name space no collision resolution
> would be
> needed.
>
> Also Eric you must take care that these node name does not ends up in libvirt
> XML since a new qemu process can generate a new different set of node name.
Correct. Either libvirt will always generate its own node names (and the
presence of a qemu-generated name is a bug in libvirt), or libvirt will
query node-names as it runs, and clear them out when the qemu process
goes away. It will NOT expose qemu names to user-visible XML (although
it should be safe to save the qemu names to internal-only XML used to
cache results across libvirtd restarts while still communicating to the
same qemu process).
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH v5 00/10] Modify block jobs to use node-names, Jeff Cody, 2014/06/13
- [Qemu-devel] [PATCH v5 01/10] block: Auto-generate node_names for each BDS entry, Jeff Cody, 2014/06/13
- [Qemu-devel] [PATCH v5 02/10] block: add helper function to determine if a BDS is in a chain, Jeff Cody, 2014/06/13
- [Qemu-devel] [PATCH v5 03/10] block: simplify bdrv_find_base() and bdrv_find_overlay(), Jeff Cody, 2014/06/13
- [Qemu-devel] [PATCH v5 04/10] block: make 'top' argument to block-commit optional, Jeff Cody, 2014/06/13
- [Qemu-devel] [PATCH v5 06/10] block: extend block-commit to accept a string for the backing file, Jeff Cody, 2014/06/13
- [Qemu-devel] [PATCH v5 05/10] block: Accept node-name arguments for block-commit, Jeff Cody, 2014/06/13
- [Qemu-devel] [PATCH v5 07/10] block: add ability for block-stream to use node-name, Jeff Cody, 2014/06/13
- [Qemu-devel] [PATCH v5 08/10] block: add backing-file option to block-stream, Jeff Cody, 2014/06/13