[mythtv-commits] Ticket #4341: Internal video player lockup

[MythTV]

#4341: Internal video player lockup
-----------------------------------------+----------------------------------
 Reporter:  marcus.t.andersson at spray.se  |        Owner:  ijr           
     Type:  defect                       |       Status:  infoneeded_new
 Priority:  major                        |    Milestone:  unknown       
Component:  mythtv                       |      Version:  0.20.2        
 Severity:  medium                       |   Resolution:                
  Mlocked:  0                            |  
-----------------------------------------+----------------------------------

Comment(by mythtv at hbuus.com):

 I don't know if it can be reproduced in trunk, but I can tell you that the
 code I changed to fix the deadlock has not changed in trunk. Which leads
 me to believe it might be possible to reproduce it in trunk.

 As a developer with 20+ years of experience, I can certainly understand
 your hesitation. However, 0.20.2 is in wide use and I suspect I'm not the
 ony one who has encountered this deadlock.

 FWIW, I don't have a second machine to set up a trunk development
 environment to confirm this. If that's a showstopper for considering my
 patch, then perhaps I should try to share it with some downstream group,
 like debian-multimedia.org or mythbuntu? Or see if I can recruit some
 testers in the various user forums to validate the problem and the fix?

 I suppose I could set up a separate root partition, or learn how chroot
 works so that I don't clobber my current mythtv setup. I am dependent on
 mythtv now so booting into a separate root partition means either choosing
 when I test so I can still record with 0.20.2 or losing out on some
 favorite shows. I don't know how deep chrooting goes, so I would want to
 be very careful about mysql, and perhaps some other gotchas I don't know
 about. Your thoughts?

 After summitting this reply, I will attach my patch. It's pretty small,
 mostly comments in videoout_xv.cpp. The substance involves swapping two
 lines in that source file. It also includes a patch to RingBuffer.cpp,
 which I know is already in trunk. Perhaps someone would be willing to take
 a few minutes and look at the comments in the videoout_xv.cpp portion of
 the patch?

-- 
Ticket URL: <http://svn.mythtv.org/trac/ticket/4341#comment:3>
MythTV <http://svn.mythtv.org/trac>
MythTV