[libvirt] [PATCH 5/6] storage: fs: Properly parse backing store info
Eric Blake
eblake at redhat.com
Tue Jul 15 20:11:24 UTC 2014
On 07/15/2014 07:25 AM, Peter Krempa wrote:
> Use the backing store parser to properly create the information about a
> volume's backing store. Unfortunately as the storage driver isn't
> perpared to allow volumes backed by networked filesystems add a
s/perpared/prepared/
> workaroud that will avoid changing the XML output.
s/workaroud/workaround/
> ---
> src/storage/storage_backend_fs.c | 21 ++++++++++++++++-----
> 1 file changed, 16 insertions(+), 5 deletions(-)
>
>
> + /* XXX: Remote storage doesn't play nicely with volumes backed by
> + * remote storage. To avoid trouble, just fake the ou */
Incomplete thought?
> +
> if (target->backingStore->format == VIR_STORAGE_FILE_AUTO) {
> - if (!virStorageIsFile(target->backingStore->path) ||
> - (rc = virStorageFileProbeFormat(target->backingStore->path,
> + if ((rc = virStorageFileProbeFormat(target->backingStore->path,
> -1, -1)) < 0) {
Indentation is off.
ACK with those fixes.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 604 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20140715/44cc32b1/attachment-0001.sig>
More information about the libvir-list
mailing list