Bug 13704

Summary: fix sound.agent to do at least some work
Product: Sisyphus Reporter: Michael Shigorin <mike>
Component: udevAssignee: Alexey Shabalin <shaba>
Status: CLOSED NOTABUG QA Contact: qa-sisyphus
Severity: normal    
Priority: P2 CC: arseny, led, shaba
Version: unstable   
Hardware: all   
OS: Linux   
Bug Depends on:    
Bug Blocks: 15333    
Attachments:
Description Flags
patch against 108-alt2 none

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