Mailing List Archive: MythTV: Commits

Ticket #6719: Add channel change monitoring into the signal monitor

Index | Next | Previous | View Threaded

mythtv at cvs

Jul 12, 2009, 4:14 PM

Post #1 of 17 (658 views)
Permalink

Ticket #6719: Add channel change monitoring into the signal monitor

#6719: Add channel change monitoring into the signal monitor
------------------------------+---------------------------------------------
Reporter: jppoet[at]gmail.com | Owner: ijr
Type: patch | Status: new
Priority: minor | Milestone: unknown
Component: MythTV - General | Version: unknown
Severity: medium | Mlocked: 0
------------------------------+---------------------------------------------
The signal monitor reports the status of tuning events to the
mythfrontend. By adding the channel change process to this list of
events, the user gets immediate feedback during a channel change. If the
channel change fails or takes a long time, the user can see this, and
choose to select a different channel or abort LiveTV.

Mythfrontend has a 15 second timeout before it gives up waiting for the
backend to send it data. Some external channel change scripts can take
over 15 seconds, which means they must be run in the background to prevent
a timeout. However, if a signal monitor is desired, the channel change
could not happen in the background because the monitor needs to know when
the channel change is complete. By moving channel change monitoring into
the "signal monitor", this problem is solved.

Note: With this change, any pre-existing external channel change scripts
which ran in the background, must be modified to run in the foreground.

--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6719>
MythTV <http://www.mythtv.org/>
MythTV
_______________________________________________
mythtv-commits mailing list
mythtv-commits[at]mythtv.org
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits

mythtv at cvs

Jul 13, 2009, 10:36 AM

Post #2 of 17 (618 views)
Permalink

Re: Ticket #6719: Add channel change monitoring into the signal monitor [In reply to]

#6719: Add channel change monitoring into the signal monitor
------------------------------+---------------------------------------------
Reporter: 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):

I attempted to use this patch to test out, as I have the exact issue this
is to fix... I receive compilation error:

channelscan/channelscan_sm.cpp: In member function ā:
channelscan/channelscan_sm.cpp:1333: error: no matching function for call
to ā
./signalmonitor.h:45: note: candidates are: virtual void
SignalMonitor::Start(bool)
make[2]: *** [channelscan_sm.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[2]: Leaving directory `/var/tmp/portage/media-
tv/mythtv-0.22_alpha20878/work/mythtv-0.22/libs/libmythtv'
make[1]: *** [sub-libmythtv-make_default] Error 2
make[1]: Leaving directory `/var/tmp/portage/media-
tv/mythtv-0.22_alpha20878/work/mythtv-0.22/libs'

--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6719#comment:1>
MythTV <http://www.mythtv.org/>
MythTV
_______________________________________________
mythtv-commits mailing list
mythtv-commits[at]mythtv.org
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits

mythtv at cvs

Jul 13, 2009, 1:02 PM

Post #3 of 17 (619 views)
Permalink

Re: Ticket #6719: Add channel change monitoring into the signal monitor [In reply to]

mythtv at cvs

Jul 13, 2009, 10:09 PM

Post #4 of 17 (616 views)
Permalink

Re: Ticket #6719: Add channel change monitoring into the signal monitor [In reply to]

mythtv at cvs

Jul 16, 2009, 5:22 PM

Post #5 of 17 (587 views)
Permalink

Re: Ticket #6719: Add channel change monitoring into the signal monitor [In reply to]

mythtv at cvs

Jul 17, 2009, 4:46 PM

Post #6 of 17 (581 views)
Permalink

Re: Ticket #6719: Add channel change monitoring into the signal monitor [In reply to]

mythtv at cvs

Aug 30, 2009, 11:16 AM

Post #7 of 17 (481 views)
Permalink

Re: Ticket #6719: Add channel change monitoring into the signal monitor [In reply to]

mythtv at cvs

Aug 31, 2009, 11:40 PM

Post #8 of 17 (466 views)
Permalink

Re: Ticket #6719: Add channel change monitoring into the signal monitor [In reply to]

mythtv at cvs

Sep 1, 2009, 9:54 AM

Post #9 of 17 (454 views)
Permalink

Re: Ticket #6719: Add channel change monitoring into the signal monitor [In reply to]

mythtv at cvs

Sep 22, 2009, 9:42 PM

Post #10 of 17 (296 views)
Permalink

Re: Ticket #6719: Add channel change monitoring into the signal monitor [In reply to]

mythtv at cvs

Sep 25, 2009, 7:01 AM

Post #11 of 17 (276 views)
Permalink

Re: Ticket #6719: Add channel change monitoring into the signal monitor [In reply to]

mythtv at cvs

Sep 25, 2009, 9:47 AM

Post #12 of 17 (276 views)
Permalink

Re: Ticket #6719: Add channel change monitoring into the signal monitor [In reply to]

mythtv at cvs

Oct 3, 2009, 1:09 PM

Post #13 of 17 (245 views)
Permalink

Re: Ticket #6719: Add channel change monitoring into the signal monitor [In reply to]

mythtv at cvs

Nov 7, 2009, 8:43 AM

Post #14 of 17 (162 views)
Permalink

Re: Ticket #6719: Add channel change monitoring into the signal monitor [In reply to]

mythtv at cvs

Nov 10, 2009, 9:59 AM

Post #15 of 17 (126 views)
Permalink

Re: Ticket #6719: Add channel change monitoring into the signal monitor [In reply to]

#6719: Add channel change monitoring into the signal monitor
------------------------------+---------------------------------------------
Reporter: jppoet@ā€¦ | Owner: danielk
Type: patch | Status: assigned
Priority: minor | Milestone: unknown
Component: MythTV - General | Version: unknown
Severity: medium | Mlocked: 0
------------------------------+---------------------------------------------

Comment(by Johnny Stenback <mythtv@ā€¦>):

For the record, the patch I just attached fixes a reproducible deadlock in
channel-thread-v1.4a.patch. The deadlock comes from a thread deadlocking
on attempting to re-lock pendingRecLock, which is non-recursive. This is
the stack for the deadlock:

#3 0x0683e982 in QMutex::lock() () from /usr/lib/libQtCore.so.4
#4 0x08075ffc in QMutexLocker::relock (this=0xad9f3828)
at /usr/include/QtCore/qmutex.h:120
#5 0x08075f7d in QMutexLocker::QMutexLocker (this=0xad9f3828,
m=0x8358904)
at /usr/include/QtCore/qmutex.h:102
#6 0x00fe0352 in TVRec::RecordPending (this=0x8358870, rcinfo=0xa85bd590,
secsleft=-1, hasLater=false) at tv_rec.cpp:383
#7 0x00fe1fa1 in TVRec::CancelNextRecording (this=0x8358870, cancel=true)
at tv_rec.cpp:480
#8 0x0807de73 in EncoderLink::CancelNextRecording (this=0x8387150,
cancel=true) at encoderlink.cpp:587
#9 0x080cb925 in MainServer::HandleRecorderQuery (this=0x8389180,
slist=@0xad9f40a0, commands=@0xad9f4098, pbs=0x8384428)
at mainserver.cpp:3468
...

--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6719#comment:14>
MythTV <http://www.mythtv.org/>
MythTV
_______________________________________________
mythtv-commits mailing list
mythtv-commits[at]mythtv.org
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits

mythtv at cvs

Nov 18, 2009, 1:35 PM

Post #16 of 17 (74 views)
Permalink

Re: Ticket #6719: Add channel change monitoring into the signal monitor [In reply to]

mythtv at cvs

Nov 24, 2009, 2:14 PM

Post #17 of 17 (33 views)
Permalink

Re: Ticket #6719: Add channel change monitoring into the signal monitor [In reply to]

#6719: Add channel change monitoring into the signal monitor
------------------------------+---------------------------------------------
Reporter: jppoet@ā€¦ | Owner: danielk
Type: patch | Status: assigned
Priority: minor | Milestone: unknown
Component: MythTV - General | Version: unknown
Severity: medium | Mlocked: 0
------------------------------+---------------------------------------------

Comment(by hansonorders@ā€¦):

I re-worked the channel-thread-v1.4a.patch for 22890 (attached as v1.5)
and was able to build my package and install. Things were looking good
until I had back-to-back recordings on the same channel/mulltiplex using
the multirecord feature. Using a HD-3000 ATSC card, the second recording
of the back-to-back fails. (file size is "B" in mythweb and when searched
for, does not exist.)

{{{
[mythtv[at]mythbox-mbe ~]$ mythbackend --version
Please include all output in bug reports.
MythTV Version : 22890M
MythTV Branch : branches/release-0-22-fixes
Network Protocol : 50
Library API : 0.22.20091023-1
QT Version : 4.5.2
Options compiled in:
linux debug using_oss using_alsa using_backend using_directfb using_dvb
using_firewire using_frontend using_hdhomerun using_hdpvr using_iptv
using_ivtv using_joystick_menu using_lirc using_mheg using_opengl_video
using_opengl_vsync using_qtwebkit using_v4l using_x11 using_xrandr
using_xv using_xvmc using_xvmc_vld using_xvmcw using_bindings_perl
using_bindings_python using_opengl using_ffmpeg_threads using_libavc_5_3
using_live using_mheg

config.log:
Tue Nov 24 13:19:57 PST 2009
./configure --prefix=/usr --arch=i686 --disable-vdpau --enable-xvmc
--enable-xvmc-pro --enable-opengl-vsync --enable-libfaad --enable-dvb
--enable-firewire --compile-type=debug --with-bindings=perl,python
--enable-audio-alsa --disable-audio-jack --disable-audio-arts
}}}

--
Ticket URL: <http://svn.mythtv.org/trac/ticket/6719#comment:16>
MythTV <http://www.mythtv.org/>
MythTV
_______________________________________________
mythtv-commits mailing list
mythtv-commits[at]mythtv.org
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-commits

Index | Next | Previous | View Threaded

Interested in having your list archived? Contact lists@gossamer-threads.com