Opened 8 years ago

Closed 11 months ago

#11475 closed Bug Report - General (Unverified)

HDHomeRun recorder repeatedly setting filter tracking ticket

Reported by: Gary Buhrmaster <gary.buhrmaster@…> Owned by: danielk
Priority: minor Milestone: unknown
Component: MythTV - DVB Version: 0.26-fixes
Severity: medium Keywords: hdhomerun
Cc: Ticket locked: no


After the reports on list, irc, and the SD forums indicated that there was an issue with the MythTV HDHomeRun recorder repeatedly issuing set filter, I have performed a git bisect of MythTV, and the result is that the first commit that is causing the problem appears to be 9b22460f53bf6313516f33987e4260897c6076a3 which is "Merge of various recorder changes from mythtv-rec2". This seems to generate new filter updates hundreds (or more) times per second. The second commit 0542a100237dd26bcbc60212c443f4619586f15b "Refs #10414. Lower HDHomeRun recorder CPU usage a tiny bit more." sets a timer to only update the filters about 10 times per second (the currently observed rate). For testing purposes, I have created a patch against fixes/0.26 which updates the filter on the hdhomerun only when the code indicates that the filter list has been changed (attached). It should not be considered complete, but simply as a proof of concept. In particular, I have not looked at EIT filter changes, nor other recorders.

Kenni Lund suggested that I submit this ticket.

Attachments (1)

filter-test.patch (2.1 KB) - added by Gary Buhrmaster <gary.buhrmaster@…> 8 years ago.
Proof of concept filter patch

Download all attachments as: .zip

Change History (7)

Changed 8 years ago by Gary Buhrmaster <gary.buhrmaster@…>

Attachment: filter-test.patch added

Proof of concept filter patch

comment:1 Changed 8 years ago by Silicondust

Related... the filter should not be set at all if QuickTune? is used.

QuickTune? tells the HDHomeRun to find the specified program and auto-set the PID filter.

Worked correctly in 0.24. In later releases MythTV sets the filter thus clearing the quicktune mode.


comment:2 Changed 6 years ago by stuartm

Gary, I don't suppose you have an updated patch which might be considered complete?

comment:3 Changed 6 years ago by Gary Buhrmaster <gary.buhrmaster@…>

Well, the patch, as submitted, worked for me, with no negative impacts. But I really (really (really)) wanted someone with expertice in the dvb code to look it over (perhaps danielk at the time, since he was the author of much of the recent code?) since my knowledge of the intricacies of all the various classes was (and is) limited, and I could have easily missed a code path that needed to be addressed. I will admit that usually when I try to get my head around the recorder code, I up being lost in a maze of classes all alike (but different). I no longer use a HDHR in non-OCUR mode (i.e. my clear QAM days have long since been encrypted out of existence, and OTA is not viable at the moment), so the patch no longer matters to me. It should be noted that SiliconDust? has updated their firmware so that the impact of continuously (10 times per second) updating the filters is no longer operational impacting for that device (which was the only device directly fixed by this patch). Perhaps the ticket should be closed as wontfix (or fixed upstream?) for ticket cleanliness.

comment:4 Changed 15 months ago by Klaas de Waal

The PID filter is set about 10 times per second. In my understanding it only needs to be set when the list of PIDs to listen to is changed.

comment:5 in reply to:  4 Changed 15 months ago by Gary Buhrmaster

Replying to Klaas de Waal:

The PID filter is set about 10 times per second. In my understanding it only needs to be set when the list of PIDs to listen to is changed.

Correct. And that was what the patch attempted to do (when submitted seven or so years ago), although not at all elegantly.

comment:6 Changed 11 months ago by Stuart Auchterlonie

Resolution: Unverified
Status: newclosed

Closing out old tickets. HDhomerun support has changed substantially since this was raised.

Note: See TracTickets for help on using tickets.