[vdr] [PATCH] rcu NPTL fix for 1.3.34
Klaus.Schmidinger at cadsoft.de
Sat Nov 5 13:56:07 CET 2005
Andreas Share wrote:
>> Andreas Share wrote:
>>> attached patch should make the rcu code NPTL compatible. Without the
>>> patch using an RCU will fail (vdr hang during startup) on an NPTL
>>> enabled system.
>>> I have only move over L. Nussels lirc NPTL fix to the rcu code.
>> Have you actually tested this?
>> I wonder if simply removing all lock calls would work,
>> because there _is_ a thread and a foreground function that
>> both want to transfer data...
> Yes, i have tested the patch on my system. With the lock´s the rcu-code
> block vdr completely during startup with NPTL enabled libraries,
> especialy this ones in /lib/tls. LD_ASSUME_KERNEL doesn´t work for this
> problem, only moving the tls-folder away from /lib (and do a ldconfig
> afters this) have let the rcu work again on my suse 9.1.
> Removing the lock in the main loop only will resolve this, but IR
> learning will fail, so i have removed the other locks also.
> With the patch vdr/rcu (learning included) works without any sideeffect
> in daily use since 21.10. on my system (internal IR-Header, DBox IR Codes).
> Andreas Share
Well, I don't have a good feeling simply throwing out the
locks here just because NPTL can't handle them.
I guess this will have to wait then until I use NPTL myself,
although I don't see this happening any time soon... ;-)
Unless, of course, you can come up with a solution that
still does some locking and runs with NPTL. The file handle
_is_ accessed from both the foreground and the background thread,
so this just can't work reliably without locking.
More information about the vdr