Opened 11 years ago

Closed 4 years ago

Last modified 3 years ago

#11475 closed Patch - Bug Fix (Unverified)

HDHomeRun recorder repeatedly setting filter tracking ticket

Reported by: Gary Buhrmaster <gary.buhrmaster@…> Owned by: Klaas de Waal
Priority: minor Milestone: 32.0
Component: MythTV - DVB Version: Master Head
Severity: medium Keywords: hdhomerun
Cc: Ticket locked: no

Description

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 (2)

filter-test.patch (2.1 KB) - added by Gary Buhrmaster <gary.buhrmaster@…> 11 years ago.
Proof of concept filter patch
20210807-trac-11475-hdhr-pid.patch (2.2 KB) - added by Klaas de Waal 3 years ago.
Gary's patch updated for today's master.

Download all attachments as: .zip

Change History (10)

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

Attachment: filter-test.patch added

Proof of concept filter patch

comment:1 Changed 11 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.

Nick

comment:2 Changed 9 years ago by stuartm

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

comment:3 Changed 9 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 4 years 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 4 years 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 4 years ago by Stuart Auchterlonie

Resolution: Unverified
Status: newclosed

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

comment:7 Changed 3 years ago by Klaas de Waal

Milestone: unknown32.0
Owner: changed from danielk to Klaas de Waal
Type: Bug Report - GeneralPatch - Bug Fix
Version: 0.26-fixesMaster Head

Patch updated for today's master. Under test.

Changed 3 years ago by Klaas de Waal

Gary's patch updated for today's master.

comment:8 Changed 3 years ago by Klaas de Waal

Note: See TracTickets for help on using tickets.