[pulseaudio-tickets] [Bug 43503] Xen paravirtualized driver frontend module

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=43503

--- Comment #5 from Colin Guthrie <fdo at colin.guthr.ie> 2011-12-07 02:46:49 PST ---
(In reply to comment #4)
> I'm not exactly a Xen expert either, but I asked one:)
> 
> Although without Xen-specific code you wouldn't be able to detect the audio
> backend itself, it is possible to detect through udev that you are running in a
> domU guest (i.e., you are not running in the dom0 'host'). In domU guests you
> can't access any hardware devices directly anyway, so your best chance is to
> just try the Xen module instead.
> 
> I think there shouldn't be a problem with including this in udev-detect.

Tanu, what do you think the best plan is here?

1) Have udev-detect notice that it is running as a guest and just unload itself
(cleanly). and have a totally separate module module-xenpv-detect configured in
our default.pa file which is similarly designed to exit gracefully if it does
not detect a Xen guest?

2) module-udev-detect itself actually loads module-xenpv-detect and unloads
itself when it detects Xen guest mode.


While the latter is less "pure" from a configuration aspect, it's nicer to not
try and load xen stuff when not needed if the support is compiled in (and doing
.nofail blocks in the default.pa is a bit crappy to work around "optional
modules" as it's arguably not an optional module when you are running under
Xen).


Or we could refactor some more how we structure our default.pa file. Some time
ago I added support for including directories containing *.pa files. We should
maybe just define a "/etc/pulse/default.pa.d/detect.d/" folder and put udev.pa
and xenpv.pa files in there... That way it's 100% a packaging issue at the
distro level. That might be nicer than hard coding and unnecessary loading of
modules that will not be needed.

I personally prefer this approach, so I would opt for code in the form of
suggest 1 above, but any further comments welcome :)

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA Contact for the bug.
You are the assignee for the bug.