[vdr] [ANNOUNCE] VDR developer version 1.7.20

Udo Richter udo_richter at gmx.de
Fri Aug 19 20:48:13 CEST 2011


Am 16.08.2011 23:13, schrieb Klaus Schmidinger:
> On 16.08.2011 19:56, Udo Richter wrote:
>> Am 16.08.2011 18:57, schrieb Klaus Schmidinger:
>>> - cSkins::Message() now blocks calls from background threads (thanks to
>>>    Michael Eiler for reporting a crash in such a scenario).
>>
>> Unfortunately, this will break the osdserver-plugin, that does call
>> these directly from the network interface thread - though not without
>> first locking the main thread in a safe state.
>>
>> I'll see if I can work around this, or if I can come up with some other
>> solution.
> 
> Actually cSkins::QueueMessage() is supposed to be used for issuing
> messages from a background thread.

Sure, however QueueMessage does not wait and does not return an user key
press. Osdserver exports all of the message functions, including
QueueMessage.

I'll probably solve this by a main thread callback, some other parts of
OsdServer do this already.


A solution / extension on VDR side would be to replace the 'main thread'
concept with a 'big VDR lock': A global lock that the main thread
releases just before sleeping, and re-locks after waking up. That way,
any background thread could use that lock to trap the main thread
safely, and use all of the not thread safe parts of VDR that previously
were only safely available to the main thread.

I think the kernel guys have a big lock they don't use any more, maybe
we can get that one. ;)


Cheers,

Udo



More information about the vdr mailing list