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@xxxxxxxxxx> > > --- > > 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@xxxxxxxxxx> > > John > > [...] > > -- > libvir-list mailing list > libvir-list@xxxxxxxxxx > https://www.redhat.com/mailman/listinfo/libvir-list
Attachment:
signature.asc
Description: PGP signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
