https://bugs.freedesktop.org/show_bug.cgi?id=43503
--- Comment #5 from Colin Guthrie <[email protected]> 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. _______________________________________________ pulseaudio-bugs mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/pulseaudio-bugs
