[pulseaudio-discuss] [PATCH] HAL hotplug support

[Lennart Poettering]

On Fri, 04.08.06 13:50, Shahms E. King (shahms at shahms.com) wrote:

> Attached is the current patch for HAL/DBus integration.  It takes care of most
> of the memory management issues in the last one as well as delaying the actual
> loading of the modules so that it always actually succeeds.
> 
> Currently missing is OSS support, although it shouldn't be too difficult to add
> as much of the infrastructure is there, just unimplemented.
> 
> There is also a rather serious issue with hot-remove at the moment, due to the
> ALSA modules continuously trying to read/write to the now-non-existant device
> which blocks further processing to respond to the device removed events and
> actually unload the modules.  I don't have enough experience with said modules
> (or ALSA in general) to properly debug and fix them at the moment.  My limited
> attempts to do so have, so far, resulted in no success ;-P

I will have a look into this

> Anyway, enjoy the extant patch, please give feedback so I can make it acceptable
> for merging.

Oh, actually I really liked your code, so I merged it already, in
r1187. Since it didn't touch any other files I saw no risk in doing
so. Thank you very much for your contribution!

I plan to make a few changes to your code (just some "const"s and
simple stuff like that), so if you make any changes, please track
current SVN! 

I would like to offer you SVN write access so that you can make
further changes directly in SVN. Is a user name of "shams" OK for you?

Thanks,

Lennart

-- 
Lennart Poettering; lennart [at] poettering [dot] net
ICQ# 11060553; GPG 0x1A015CC4; http://0pointer.net/lennart/