Bug 13704 - fix sound.agent to do at least some work
Summary: fix sound.agent to do at least some work
Status: CLOSED NOTABUG
Alias: None
Product: Sisyphus
Classification: Development
Component: udev (show other bugs)
Version: unstable
Hardware: all Linux
: P2 normal
Assignee: Alexey Shabalin
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks: 15333
  Show dependency tree
 
Reported: 2007-12-14 16:21 MSK by Michael Shigorin
Modified: 2012-03-16 13:58 MSK (History)
3 users (show)

See Also:


Attachments
patch against 108-alt2 (2.34 KB, patch)
2007-12-14 16:23 MSK, Michael Shigorin
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Shigorin 2007-12-14 16:21:39 MSK
/lib/udev/sound.agent is currently very lazy: if it doesn't find *any*
utility/directory, it bails out immediately; for LTSP clients, it would be quite
beneficial to do at least *some* job, like setting up mixer to some sane
defaults even if there's no /usr/sbin/alsactl and /var/lib/hotplug/

PS: this fix would help with sound out-of-box for terminal server distro, btw...
Comment 1 Michael Shigorin 2007-12-14 16:23:50 MSK
Created attachment 2329 [details]
patch against 108-alt2

прикладываю патч между 108-alt2 и 108-alt2.1, доступным здесь:
http://git.altlinux.org/people/mike/packages/?p=udev.git;a=commitdiff;h=411f022a80f395436326c5e7c8d88f9a55daf33c
Comment 2 Michael Shigorin 2007-12-14 16:24:42 MSK
вдогонку:

[15:20:10] <Led> sound.agent  не выставляет дефолтное значение Front. Значит на
интегрированных 5.1/7.1 адаптерах мы звука не получим
[15:20:51] <gvy> ну вешай тоже багу :)
[15:20:59] <gvy> или скажи, сразу пропатчу
[15:21:10] <gvy> что именно выставлять -- Front?  в 75?
[15:23:02] <Led> почини sound.agent на счё Front:
mixer PCM 80%
например.
Тогда я уберу выставление Volume'ов в %_initdir/ltsp-client вобще
[15:23:05] <Led> ой, не PCM, а Front
Comment 3 Michael Shigorin 2007-12-14 16:26:13 MSK
[15:23:49] <gvy> ok
[15:25:34] <Led> хотя... у меня Front в 100% выставляется...
Comment 4 Sergey Vlasov 2007-12-14 18:57:58 MSK
Проверку наличия исполняемых файлов действительно можно сделать в виде:

[ -x "$AMIXER" ] || [ -x "$ALSACTL" ] || exit 1

Т.е., если нет ни одного бинарника - это либо несмонтированный /usr при запуске
udevd из rc.sysinit, либо система, в которой нечем работать со звуком; если есть
хотя бы один бинарник, ждать другого уже бессмысленно, но можно сделать хотя бы
часть работы.

Проверку наличия /var/lib/hotplug не хотелось бы отрывать, поскольку она также
охватывает случай несмонтированного /var при запуске из rc.sysinit - тогда, даже
если вернуть ошибку, получится, что перед восстановлением сохранённого состояния
некоторое время будет действовать настройка по умолчанию, устанавливаемая
sound.agent, что в некоторых случаях может быть нежелательно. Думаю, что лучше
будет обеспечить наличие пустого /var/lib/hotplug в системе.
Comment 5 Michael Shigorin 2007-12-16 03:43:33 MSK
Кажется, /var/lib/hotplug для нашего случая обеспечивается ltsp5-5.0-alt0.65.
Comment 6 Michael Shigorin 2007-12-16 03:58:51 MSK
В смысле в ltsp5-client-full-5.0-alt0.11 :)
Comment 7 Valery Inozemtsev 2008-06-19 20:58:36 MSD
в udev больше нет sound.agent