Opened 12 years ago

Closed 4 months ago

#8778 closed Developer Task (Trac EOL)

RecordingProfile speedups / refactor

Reported by: stuartm Owned by: stuartm
Priority: minor Milestone: needs_triage
Component: MythTV - Recording Version: Master Head
Severity: medium Keywords: speedup memory leak
Cc: Ticket locked: no

Description

Obtaining the profile via RecordingProfile? in TVRec is pretty slow when it doesn't need to be and this impacts channel change times. Populating the 'SelectSetting?' and it's backing ComboBox? alone can take 100ms and this is done twice, once for the signal monitor and once for the actual program.

Attached is an initial suggested change which eliminates a redundant query on the database when loadByType() and loadByGroup() are used (untested atm).

Further improvements can be made by not building the settings object (and therefore the widget) just to return the profile from the database.

Attachments (2)

recprofile.diff​ (5.7 KB) - added by anonymous 12 years ago.
valgrind-and-logs.tar.bz2​ (307.9 KB) - added by Stuart Auchterlonie 12 years ago.
Valgrind logs and backend log from 2 day long run showing memory leaks in RecordingProfile?

Download all attachments as: .zip

Change History (10)

Changed 12 years ago by anonymous

Attachment: recprofile.diff​ added

comment:1 Changed 12 years ago by stuartm

Owner: changed from danielk to stuartm
Status: new → accepted

comment:2 Changed 12 years ago by stuartm

Reporter: changed from anonymous to stuartm

Changed 12 years ago by Stuart Auchterlonie

Attachment: valgrind-and-logs.tar.bz2​ added

Valgrind logs and backend log from 2 day long run showing memory leaks in RecordingProfile?

comment:3 Changed 12 years ago by Stuart Auchterlonie

Keywords: speedup memory leak added
Summary: RecordingProfile speedups → RecordingProfile speedups / refactor

Added valgrind logs which show the leaks that the RecordingProfile? class causes.

comment:4 Changed 12 years ago by stuartm

(In [6dd9b7892]) Recording profile optimisations (part one).

comment:5 Changed 12 years ago by stuartm

(In [ad967599]) Reapply, "Recording profile optimisations (part one)".

comment:6 Changed 11 years ago by stuartm

Type: patch → Developer Task

comment:7 Changed 2 years ago by Stuart Auchterlonie

Milestone: unknown → needs_triage

comment:8 Changed 4 months ago by Stuart Auchterlonie

Resolution: → Trac EOL
Status: accepted → closed

We have moved all bug tracking to github [1]

If you continue to have this issue, please open a new issue at github, referencing this ticket.

[1] - ​https://github.com/MythTV/mythtv/issues

Note: See TracTickets for help on using tickets.