Re: [PATCH] Ignore char devices in storage pools by default

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 03/20/2014 10:08 AM, Ján Tomko wrote:
> Without this, using /dev/mapper as a directory pool
> fails in virStorageBackendUpdateVolTargetInfoFD:
> cannot seek to end of file '/dev/mapper/control': Illegal seek
> 
> Skip over character devices by default.

I agree that character devices can't serve as a storage volume.

It may still be nice to have storage pools express all file names,
including those files that can't be volumes (dangling symlinks,
character devices, named fifos or sockets) rather than its current
filtering (regular files, block devices, directories, symlinks to one of
the above).  Doing so would make storage pools become a way to do remote
directory listings, for all types of files and not just storage volumes,
which becomes handy for choosing new volume names (which should not
conflict with existing files, even where the existing files cannot be a
storage volume).  But that's a bigger change for a different day.

> 
> https://bugzilla.redhat.com/show_bug.cgi?id=710866
> ---
>  src/storage/storage_backend.h | 1 -
>  1 file changed, 1 deletion(-)

ACK.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]