[mythtv] [mythtv-commits] Ticket #6719: Add channel change monitoring into the signal monitor

[Mark Buechler]

Hi

On Thu, Sep 3, 2009 at 12:13 PM, Eric Bosch <eric.bosch at comcast.net> wrote:

> Mark Buechler wrote:
> > Hi
> >
> > On Tue, Sep 1, 2009 at 12:54 PM, MythTV <mythtv at cvs.mythtv.org
> > <mailto:mythtv at cvs.mythtv.org>> wrote:
> >
> >     #6719: Add channel change monitoring into the signal monitor
> >
> ------------------------------+---------------------------------------------
> >      Reporter:  jppoet at gmail.com <mailto:jppoet at gmail.com>  |
> >      Owner:  ijr
> >         Type:  patch             |       Status:  new
> >      Priority:  minor             |    Milestone:  unknown
> >     Component:  MythTV - General  |      Version:  unknown
> >      Severity:  medium            |   Resolution:
> >      Mlocked:  0                 |
> >
> ------------------------------+---------------------------------------------
> >
> >     Comment(by eric.bosch at comcast.net <mailto:eric.bosch at comcast.net>):
> >
> >      I will test this again.  It was working well before, but I dropped
> >     using
> >      it at one point due to a different problem.  Now, after an upgrade
> to
> >      current CVS today (21623) my firewire capture has ceased to
> function,
> >      failing to lock on signal, even though firewire tester shows it as
> fine
> >
> >     --
> >
> > I suspect there may be a race condition here between
> > SetupSignalMonitor() and SetupDTVSignalMonitor(). SetupSignalMonitor()
> > may be attempting to
> > GetDTVSignalMonitor()->GetStreamData()->SetVideoStreamsRequired(0) while
> > GetStreamData() is NULL before SetupDTVSignalMonitor() has a chance to
> > initialize it.. but I could be wrong.
> >
> > - Mark.
> >
> >
> > ------------------------------------------------------------------------
> >
> > _______________________________________________
> > mythtv-dev mailing list
> > mythtv-dev at mythtv.org
> > http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-dev
> I'm seeing some deadlocks on channel changes with this using a Firewire
> and pvr 150 tuners
>

The race I'm was referring to wouldn't cause a deadlock but instead a
segfault.

- Mark.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mythtv.org/pipermail/mythtv-dev/attachments/20090903/b23d9fa6/attachment.htm>