<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Wed, Jun 27, 2018 at 2:19 PM Tanu Kaskinen <<a href="mailto:tanuk@iki.fi">tanuk@iki.fi</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Having a warning in the syslog should make it a bit easier to debug<br>
situations where some other program is hogging the sound card.<br></blockquote><div><br></div><div>+1. Just the other day someone on IRC had this problem and debug logs were needed.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
---<br>
 src/modules/module-udev-detect.c | 4 +++-<br>
 1 file changed, 3 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/src/modules/module-udev-detect.c b/src/modules/module-udev-detect.c<br>
index d119f96ae..4598a3d13 100644<br>
--- a/src/modules/module-udev-detect.c<br>
+++ b/src/modules/module-udev-detect.c<br>
@@ -308,7 +308,9 @@ static void verify_access(struct userdata *u, struct device *d) {<br>
              * inotify and can then recheck. */<br>
<br>
             busy = is_card_busy(path_get_card_id(d->path));<br>
-            pa_log_debug("%s is busy: %s", d->path, pa_yes_no(busy));<br>
+<br>
+            if (busy)<br>
+                pa_log_warn(_("Can't use sound card %s, because some other program is using it."), d->card_name);<br></blockquote><div><br></div><div>I think the path info is important because it is something that one can then pass on to lsof.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
             if (!busy) {<br></blockquote><div><br></div><div>This should be an else, otherwise it's weird to have if (foo) {} if (!foo) {} blocks.</div><div><br></div></div></div>