On 10/05/2012 12:25 PM, Blue Swirl wrote: > On Fri, Oct 5, 2012 at 6:07 PM, Corey Bryant <coreyb@xxxxxxxxxxxxxxxxxx> wrote: >> These new options can be used for passing drive file descriptors >> on the command line, instead of using the file option to specify >> a file name. >> >> These new command line options mirror the existing add-fd QMP >> command which allows an fd to be passed to QEMU via SCM_RIGHTS and >> added to an fd set. The opaque option is also available with >> add-fd, and allows a free-form string to be stored in the fd set >> along with the fd. >> >> + .name = "opaque", > > 'opaque' is not very descriptive and it's also not obvious (except > from the help text) that it's only interesting for file descriptors. > How about fd_name, fd_tag or fd_descr? Hmm, since opaque is per-fd in the existing monitor command, that means my proposal needs a slight modification to: -fdset set=1,fd=24,opaque="rdonly",fd=25,opaque="rdwr" or some other way where we can specify multiple fds and multiple opaque strings per set. At any rate, this just proves that we need to nail down the command line implementation to something that is easy enough to use, before coding up something that locks us in to bad design. -- Eric Blake eblake@xxxxxxxxxx +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