[mythtv] Mythfrontend hang with CVS
Xavier Hervy
maxpower44 at tiscali.fr
Wed Jun 9 18:01:38 EDT 2004
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.
>
> Leo.
>
No time to try it this day. i will apply tomorow (i hope) and send feedback.
thx
More information about the mythtv-dev
mailing list