[mythtv] Mythfrontend hang with CVS

[Tako Schotanus]

Tako Schotanus wrote:

> Leo Weppelman wrote:
>
>> On Wed, Jun 09, 2004 at 04:01:09PM -0500, Stacey Son wrote:
>>  
>>
>>> Leo Weppelman wrote:
>>>
>>>   
>>>
>>>>> Thx but I have modify my patch too. Apply a patch on mediamonitor 
>>>>> cvs which is broken, have no sense.
>>>>> I should have prefer that you find what is wrong in the current 
>>>>> cvs ;-)
>>>>>       
>>>>
>>
>> OK, I left your stuff out except the remove of the close in
>> MythMediaDevice::unlock(). This close killed the media-change sensing.
>>
>>  
>>
>>>> Shall I combine the getfsent() nesting patch from Stacey, the patches
>>>>     
>>>> from Xavier - modified by Stacey and the stack-overflow fix by me into
>>>
>>>   
>>>
>>>> one patch and post it? This in an effort not to do it all twice or 
>>>> trice?
>>>>
>>>>     
>>>
>>> Leo,  I am wondering if the check you added in 
>>> MythCDROMLinux::unlock() is necessary any more given Xavier's 
>>> patches: Note that closeDevice() is no longer called in 
>>> MediaDevice::unlock().
>>>   
>>
>>
>> Yes, the check is still needed because the endless recursion is still
>> there.
>>
>> Attached is the combined patch that brings the mediamonitor 
>> functionality
>> back to the point before it broke by the ioctl-lock en FreeBSD patches.
>>
>> Please have a look so we can submit it.
>>
>>  
>>
> Well it fixes the hang at least, but I have no idea what more I should 
> test for.
>
I was a bit too quick with my last message:
because if I put in a CD and remove it again the frontend hangs and 
starts spewing an endless list of "Unlocking CDROM door" messages.

Cheers,
-Tako