Re: [PATCH 3/6] qemu: Add option to treat missing USB devices as success

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

 



On Thu, Oct 11, 2012 at 03:35:27PM +0200, Jiri Denemark wrote:
> On Wed, Oct 10, 2012 at 16:35:05 +0100, Daniel P. Berrange wrote:
> > On Tue, Oct 09, 2012 at 02:13:24PM +0200, Jiri Denemark wrote:
> > > diff --git a/src/conf/domain_conf.h b/src/conf/domain_conf.h
> > > index fa5d1f4..3cb1193 100644
> > > --- a/src/conf/domain_conf.h
> > > +++ b/src/conf/domain_conf.h
> > > @@ -386,6 +386,7 @@ struct _virDomainHostdevDef {
> > >      int mode; /* enum virDomainHostdevMode */
> > >      int startupPolicy; /* enum virDomainStartupPolicy */
> > >      unsigned int managed : 1;
> > > +    unsigned int missing : 1;
> > >      union {
> > >          virDomainHostdevSubsys subsys;
> > >          struct {
> > 
> > I was wondering if we should actually expose this in the
> > XML, as a read-only attribute, only shown when requesting
> > the live XML. Otherwise there's no way for an application
> > to determine whether the USB device they requested was
> > actually given to the guest, without looking inside the
> > guest.
> 
> Yeah, this sounds like a useful thing. Could that be a follow up patch to
> avoid holding up the whole series just for this? :-)

As long as we do the followup now, I'm fine with it being separate.
Just don't want us to release anything without it.

Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|

--
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]