pulseaudio init-script fails to restart

Bug #244414 reported by Aron Griffis
0
Affects Status Importance Assigned to Milestone
pulseaudio (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

Binary package hint: pulseaudio

package: pulseaudio-0.9.10-2ubuntu1
on Ubuntu intrepid

Sometimes pulseaudio takes a second to stop. This can cause
restart to fail:

$ sudo /etc/init.d/pulseaudio restart
Stopping PulseAudio Daemon.
Starting PulseAudio Daemon/usr/bin/pulseaudio already running.

This can be fixed by using the --retry option to
start-stop-daemon. With the attached patch, the initscript will
wait 3 seconds for pulseaudio to stop, and report the final status
appropriately. Note this is *up to* 3 seconds; start-stop-daemon
will poll regularly and exit early if possible.

Revision history for this message
Aron Griffis (agriffis) wrote :
Changed in pulseaudio:
status: New → Confirmed
Daniel T Chen (crimsun)
Changed in pulseaudio:
importance: Undecided → Low
status: Confirmed → In Progress
Revision history for this message
Aron Griffis (agriffis) wrote :

Because of sh -e, the script should abort immediately if start-stop-daemon returns non-zero status. To avoid that for the sake of error reporting, the provided patch could be modified as:

start-stop-daemon -p $PIDFILE --stop --retry TERM/3 || status=$?

Daniel T Chen (crimsun)
Changed in pulseaudio:
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package pulseaudio - 0.9.13-2ubuntu5

---------------
pulseaudio (0.9.13-2ubuntu5) jaunty; urgency=low

  [ Daniel T Chen ]
  * Add fixes from git:
    - 0015_use_fionread.patch,
    - 0016_add_pa-source-message-get-latency.patch,
    - 0017_allow_dev_dsp_w-ok.patch,
    - 0018_fix_return_val_pa-frame-aligned.patch,
    - 0019_fix_macro_pa-unlikely.patch,
    - 0020_catch_driver_errors.patch (LP: #312373),
    - 0021_convert_to_pa-bool-t.patch,
    - 0022_fix_return_val_dump-resample-methods.patch,
    - 0023_use_gdbm-nolock.patch,
    - 0024_dont_mix_front-center_into_rear.patch,
    - 0025_drop_cap-nice_properly.patch,
    - 0027_handle_multicast_sdp_with_same_ip_ttl_as_rtp.patch,
    - 0028_prefer_mixer_controls_with_vols.patch (LP: #281605),
    - 0029_add_minor_fixes.patch
  * Forward-port workaround from hardy branch for source/sink and
    suspend-on-idle race (LP: #203654):
    - 0026_work_around_suspend-on-idle_source_sink_race.patch
  * debian/control: Drop padevchooser(Recommends) and pavucontrol
    (Suggests) completely - functionality has been subsumed by
    jaunty's gnome-volume-control.
  * debian/pulseaudio.init:
    - Make initscript more informative in the default case of per-user
      sessions (LP: #259522),
    - Make initscript retry (until three-second timeout) if stopping
      fails when restarting (LP: #244414),
    - LSB {Required-*,Should-*} should specify hal instead of dbus,
      since hal is required (and already requires dbus) (LP: #244679)
  * Previous upload contains patches to close:
    LP: #193491, #268891, #295164, #298301

  [ Luke Yelavich ]
  * 0008_regen_autotools.patch: Pull some libtool 2.2 updates from git,
    and regenerate autotools glue.
  * debian/rules: Add --enable-static to force creation of libpulse.a,
    a side-effect of messing with the autotools files. This shouldn't
    be needed for the 0.9.14 release.

 -- Luke Yelavich <email address hidden> Fri, 09 Jan 2009 17:05:38 +1100

Changed in pulseaudio:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.