After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 141793 - gnome-settings-daemon keeps occupying alsa (GST backend)
gnome-settings-daemon keeps occupying alsa (GST backend)
Status: RESOLVED FIXED
Product: gnome-control-center
Classification: Core
Component: [obsolete] settings-daemon
2.8.x
Other Linux
: Normal normal
: ---
Assigned To: Bastien Nocera
Control-Center Maintainers
: 155678 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2004-05-04 05:31 UTC by Jakub Kulesza
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: 2.5/2.6


Attachments
patch to free alsa with gstreamer (5.01 KB, patch)
2004-08-13 11:13 UTC, Arnaud Patard
none Details | Review
updated patch (5.23 KB, patch)
2004-11-30 16:38 UTC, Sebastien Bacher
none Details | Review
patch which fixes the problem (1.06 KB, patch)
2004-12-09 16:37 UTC, Christophe Fergeau
none Details | Review
should be more correct (1.14 KB, patch)
2004-12-09 16:48 UTC, Christophe Fergeau
none Details | Review

Description Jakub Kulesza 2004-05-04 05:31:50 UTC
gnome-settings-daemon keeps occupying /dev/snd/controlC0 so restarting alsa
after a laptop suspend is not possible. 

i expected it would act as gnome-2.4 where that file was not occupied all the time.

i used the gentoo's gnome-2.6_p1.ebuild to build it using
CFLAGS="-march=pentium3 -O3 -pipe"
CHOST="i686-pc-linux-gnu"


Keep up the good work !!! (i hate myself for not being a programmer)
Comment 1 Arnaud Patard 2004-07-03 16:08:20 UTC
Hi,

The same problem has also been reported on Debian BTS :
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=255793&repeatmerged=no
Comment 2 Bastien Nocera 2004-07-15 22:42:51 UTC
2004-07-15  Bastien Nocera  <hadess@hadess.net>
                                                                                
        * actions/acme-volume-alsa.c: (acme_volume_alsa_finalize),
        (acme_volume_alsa_set_mute), (acme_volume_alsa_get_mute),
        (acme_volume_alsa_get_volume), (acme_volume_alsa_set_volume),
        (acme_volume_alsa_close_real), (acme_volume_alsa_open),
        (acme_volume_alsa_close), (acme_volume_alsa_init):
        rework the ALSA mixer so that:
        - the ALSA device isn't kept open at all times, only for 4 seconds
          after the last use (Closes: #141793)
        - the hardware mute is used if it is available
          (Closes: #141404, #144489, #140937)
        - lowering the volume doesn't move the balance to the right
        - it doesn't leak alsa-lib mixers all around
Comment 3 Arnaud Patard 2004-08-12 17:45:46 UTC
This bug is still there as you fixed the code that handle directly alsa. The
gstreamer code needs also to be fixed (as it's possible to use alsa with gstreamer).
Comment 4 Arnaud Patard 2004-08-13 11:13:37 UTC
Created attachment 30511 [details] [review]
patch to free alsa with gstreamer
Comment 5 Kasper Ronning 2004-10-18 16:46:26 UTC
*** Bug 155678 has been marked as a duplicate of this bug. ***
Comment 6 Sebastien Bacher 2004-11-21 22:16:25 UTC
the gstreamer part of the patch is used in the debian package and seems to work
fine, permission to commit to HEAD ?
Comment 7 Bastien Nocera 2004-11-30 15:44:06 UTC
The patch doesn't apply to HEAD any more. Arnaud, could you update the patch to
fit on either HEAD or gnome-2-8?
Comment 8 Sebastien Bacher 2004-11-30 16:38:54 UTC
Created attachment 34328 [details] [review]
updated patch

oups, I was supposed to do that. This patch is the version currently used in
the Debian package, I'll commit it now.
Comment 9 Sebastien Bacher 2004-11-30 16:49:24 UTC
commit to gnome-2-8 and HEAD
Comment 10 Christophe Fergeau 2004-12-09 16:33:05 UTC
This breaks acme for me
Comment 11 Christophe Fergeau 2004-12-09 16:37:44 UTC
Created attachment 34666 [details] [review]
patch which fixes the problem

The attached patch fixes the problem.
-	return FALSE;
+	return TRUE;
is what fixes it, I dunno if it's correct though. The
acme_volume_gstreamer_open function clearly  has an issue though since it does
a lot of stuff, but unconditionnally return FALSE as if it had failed.

The rest of the patch cleans up some warnings when gnome-settings-daemon is run
from a terminal.
Comment 12 Christophe Fergeau 2004-12-09 16:48:50 UTC
Created attachment 34667 [details] [review]
should be more correct
Comment 13 Christophe Fergeau 2004-12-09 16:57:26 UTC
Fixed in HEAD and gnome-2-8