[libvirt] [PATCH 4/6] conf: introduce <output> element for <sound> devices

[Pavel Hrdina]

On Mon, Nov 20, 2017 at 06:04:27PM -0500, John Ferlan wrote:
> 
> 
> On 11/14/2017 08:45 AM, Pavel Hrdina wrote:
> > So far it was not possible to specify how the audio output from guest
> 
> s/So far it was/It is/
> 
> > should be presented to host/users.  Now it will be possible to do so
> > via <output> element for <sound> device where you specify the output
> > "type".
> > 
> > Signed-off-by: Pavel Hrdina <phrdina at redhat.com>
> > ---
> >  docs/formatdomain.html.in     |  9 +++++++
> >  docs/schemas/domaincommon.rng | 14 ++++++++++
> >  src/conf/domain_conf.c        | 61 +++++++++++++++++++++++++++++++++++++++++++
> >  src/conf/domain_conf.h        | 14 ++++++++++
> >  src/libvirt_private.syms      |  2 ++
> >  5 files changed, 100 insertions(+)
> > 
> > diff --git a/docs/formatdomain.html.in b/docs/formatdomain.html.in
> > index 47c43d0666..3b7c367fc7 100644
> > --- a/docs/formatdomain.html.in
> > +++ b/docs/formatdomain.html.in
> > @@ -7058,6 +7058,15 @@ qemu-kvm -net nic,model=? /dev/null
> >        slot, <a href="#elementsAddress">documented above</a>.
> >      </p>
> >  
> > +    <p>
> > +      <span class="since">Since 3.10.0</span> sound device can have
> 
> s/sound/, a sound/
> 
> > +      an optional <code>output</code> element which configures where
> > +      the audio output is connected within host. There is mandatory
> > +      <code>type</code> attribute where valid values are 'none' to
> > +      disable the audio output, 'spice', 'pa', 'sdl', 'alsa', 'oss'.
> 
> I've become preferential to seeing these in a list. I have no idea based
> on the text here what 'spice', ... 'oss' really means.  IOW: Why would I
> want to set to something else - what does it do/provide?

I'm not sure how to describe it more than providing links to all the
projects.  SPICE is the only unique output since the audio stream is
sent over SPICE protocol, but the remaining outputs are IMHO
self-explanatory because the are standard linux/unix audio
layers/libraries.

Describing all the differences is out of scope of our documentation.

> 
> > +      This might not be supported by all hypervisors.
> 
> True, but perhaps also true of many other things, too.

I can change it to "Currently supported only by QEMU driver."

> > +    </p>
> > +
> >      <h4><a id="elementsWatchdog">Watchdog device</a></h4>
> >  
> >      <p>
> 
> 
> Reviewed-by: John Ferlan <jferlan at redhat.com>
> 
> John
> 
> [...]
> 
> --
> libvir-list mailing list
> libvir-list at redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20171121/3c22529a/attachment-0001.sig>