[mythtv] [mythtv-commits] Ticket #10765: HD-PVR: Rework SignalMonitor to avoid reading from device

[John P Poet]

On Thu, May 31, 2012 at 8:16 AM, Steven Adeff <adeffs.mythtv at gmail.com>wrote:

> On Thu, May 31, 2012 at 10:05 AM, John P Poet <jppoet at gmail.com> wrote:
> > On Thu, May 31, 2012 at 7:19 AM, Steven Adeff <adeffs.mythtv at gmail.com>
> > wrote:
> >> On Wed, May 30, 2012 at 11:02 PM, John P Poet <jppoet at gmail.com> wrote:
> >> > On Wed, May 30, 2012 at 8:43 PM, Steven Adeff <
> adeffs.mythtv at gmail.com>
> >> > wrote:
> >> >> On Mon, May 28, 2012 at 12:10 PM,  <noreply at mythtv.org> wrote:
> >> >> > #10765: HD-PVR: Rework SignalMonitor to avoid reading from device
> >> >> > ------------------------------+------------------------
> >> >> >  Reporter:  jpoet             |          Owner:  jpoet
> >> >> >     Type:  Patch - Bug Fix   |         Status:  closed
> >> >> >  Priority:  minor             |      Milestone:  0.25.1
> >> >> > Component:  MythTV - General  |        Version:  0.25
> >> >> >  Severity:  medium            |     Resolution:  fixed
> >> >> >  Keywords:  HDPVR LiveTV      |  Ticket locked:  0
> >> >> > ------------------------------+------------------------
> >> >> > Changes (by wagnerrp):
> >> >> >
> >> >> >  * version:  Unspecified => 0.25
> >> >> >
> >> >> >
> >> >> > --
> >> >> > Ticket URL: <http://code.mythtv.org/trac/ticket/10765#comment:5>
> >> >> > MythTV <http://code.mythtv.org/trac>
> >> >> > MythTV Media Center
> >> >>
> >> >> since this was applied I've noticed this in my backend log:
> >> >>  (StartEncoding) MPEGRec(/dev/hdpvr2): StartEncoding
> >> >> May 30 19:36:28 MythCenter mythbackend[17345]: I RecThread
> >> >> mpegrecorder.cpp:1304 (StartEncoding) MPEGRec(/dev/hdpvr2): Encoding
> >> >> started
> >> >> May 30 19:36:28 MythCenter mythbackend[17345]: I RecThread
> >> >> DeviceReadBuffer.cpp:128 (Start) DevRdB(/dev/hdpvr2): Start() --
> begin
> >> >> May 30 19:36:28 MythCenter mythbackend[17345]: I RecThread
> >> >> DeviceReadBuffer.cpp:146 (Start) DevRdB(/dev/hdpvr2): Start() --
> >> >> middle
> >> >> May 30 19:36:28 MythCenter mythbackend[17345]: I RecThread
> >> >> DeviceReadBuffer.cpp:151 (Start) DevRdB(/dev/hdpvr2): Start() -- end
> >> >> May 30 19:36:31 MythCenter mythbackend[17345]: E DeviceReadBuffer
> >> >> DeviceReadBuffer.cpp:513 (Poll) DevRdB(/dev/hdpvr2): Poll giving up 2
> >> >> May 30 19:36:31 MythCenter mythbackend[17345]: E DeviceReadBuffer
> >> >> DeviceReadBuffer.cpp:351 (run) DevRdB(/dev/hdpvr2): fill_ringbuffer:
> >> >> error state
> >> >> May 30 19:36:31 MythCenter mythbackend[17345]: E RecThread
> >> >> mpegrecorder.cpp:1010 (run) MPEGRec(/dev/hdpvr2): Device error
> >> >> detected
> >> >> May 30 19:36:31 MythCenter mythbackend[17345]: I RecThread
> >> >> mpegrecorder.cpp:1247 (RestartEncoding) MPEGRec(/dev/hdpvr2):
> >> >> RestartEncoding
> >> >
> >> > <snip>
> >> >
> >> >
> >> > Have you tried yanking the HD-PVR's power cord for a minute?  About
> once
> >> > every couple of months, my HD-PVR will get in a state like that.
> >> > Turning it
> >> > off/on does not fix it -- I actually have to yank the power for a
> while.
> >>
> >> yea, that gets done about once a week, if not more often. I'll try it
> >> for a longer period of time and see if that does anything.
> >
> >
> >
> > That can also be caused by a loose cable. If nothing else works, I can
> send
> > you a patch to revert that change, but I really expect this problem to be
> > coincidence instead of cause and effect.
> >
> > John
> > --
> > A: Because it messes up the order in which people normally read text.
> > Q: Why is top-posting such a bad thing?
>
> I agree, and it's not like there weren't issues before. I just wasn't
> sure if I was hitting a side effect of the new code or not, or if this
> is intended behavior.
>
> that said, if Myth "knows" it's not receiving proper info from an
> HDPVR in this manner, could it be used to trigger the HDPVR
> "powercycle" device to initiate a power cycle?
>


I proposed integrating the HD-PVR killer into the myth code, and the idea
was rejected -- too specialized.

It has been a while since I looked at the "recovery" path when there are
HD-PVR problems.  We probably should be "backing off" a bit, when the
StartEncoding is repeatedly failing.


John
-- 
A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.mythtv.org/pipermail/mythtv-dev/attachments/20120531/f1be2f0b/attachment.html>