fails to start on boot due to hal not running

Bug #244679 reported by Rudd-O
4
Affects Status Importance Assigned to Milestone
pulseaudio (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

PA fails to start in /etc/rc*.d/S20pulseaudio because /etc/rc*.d/S24hal is started afterwards, and hal is a requirement for pulseaudio.

Fix is to make pulseaudio start at S25 in the sysvinit system, and add "Should-start: dbus hal" "Should-stop: dbus hal" in the header of the initscript.

Can we fix this in the current release? We need to, otherwise PA is useless in a default boot system that is using PA as systemwide daemon.

Revision history for this message
Luke Yelavich (themuso) wrote : Re: [Bug 244679] [NEW] fails to start on boot due to hal not running

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

By default, pulseaudio only runs as the user, and not system wide. Unless you have a specific reason, pulseaudio should not be run system wide as it is a possible security risk. Added to that, pulseaudio will only use the hal module if its available and hal is running. Perhaps the hal module loading needs to prevent pulseaudio from failing etc, but people who set up pulseaudio system wide, do so for a good reason, and I'd suspect they would know what they are doing.

 affects ubuntu/pulseaudio
 status wontfix
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)

iD8DBQFIaszbjVefwtBjIM4RApvVAJ9pE3bGVtyJFDJ77bOHuA35mbQNhACcD8hw
ihdIhMcek+qbM93Fzq8zEnI=
=LmjE
-----END PGP SIGNATURE-----

Changed in pulseaudio:
status: New → Won't Fix
Revision history for this message
Rudd-O (rudd-o) wrote :

> By default, pulseaudio only runs as the user, and not system wide.

Yes. What does that have to do with my bug report?

> Unless you have a specific reason, pulseaudio should not be run system wide as it is a possible security risk.

I have a specific reason (the reason being I am not a moron -- I know what I am doing) so that's why I enabled it in /etc/default/pulseaudio. Again, what does that have to do with my bug report?

> Added to that, pulseaudio will only use the hal module if its available and hal is running.

Not true, if HAL is not available, PA refuses to start. This is FACT.

> Perhaps the hal module loading needs to prevent pulseaudio from failing etc

No, what needs to be fixed is the initscript's ordering and dependencies. If you "fix" the hal module loading by loading the old "detect" module, you get only one sound output in stereo, not several outputs.

> but people who set up pulseaudio system wide, do so for a good reason, and I'd suspect they would know what they are doing.

I know what I am doing and I know why I submitted the bug.

In summary:

- there is a bug in the Ubuntu initscript that triggers in a certain scenario which is not the default but it is a supported scenario by the original developers.
- fixing this bug with the fix that I have researched and tested introduces no regressions or changes in behavior to default-configured systems
- fixing this bug with the fix that I have researched and tested fixes the needs of us who use PA system-wide

On the other hand, if you would like to cut your nose to spite your face, you could theoretically remove the initscript altogether in the next update, right? I mean, it's not like PA defaults to systemwide, right? You even said it yourself: "people who set up pulseaudio system wide, do so for a good reason, and I'd suspect they would know what they are doing". So that means *I* know what I am doing and I could very well write my own initscript, right?

News for you: I did. I fixed the initscript and generously contributed that fix back to the community. It took me quite a while to figure the bug out and fix it. Did you even give this bug a longer-than-cursory look?

This excuse of a triage is shameful. So, tell me again: why did you triage this bug as WONTFIX?

Changed in pulseaudio:
status: Won't Fix → Confirmed
Revision history for this message
Daniel T Chen (crimsun) wrote :

rc2.d order fixed in jaunty; remaining changes forthcoming

Changed in pulseaudio:
importance: Undecided → Low
status: Confirmed → 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.