Opened 16 years ago
Closed 16 years ago
#5369 closed patch (wontfix)
reduce dirty region check interval of mythtv ui
| Reported by: | Owned by: | stuartm | |
|---|---|---|---|
| Priority: | minor | Milestone: | unknown |
| Component: | mythtv | Version: | unknown |
| Severity: | low | Keywords: | repaint ui |
| Cc: | Ticket locked: | no |
Description
Running powertop on my idle AppleTV frontend reveals that 77% of the wakeups from idle are caused by MythTV repainting its UI 70 times per second. The attached patch reduces this to 5 times per second, and doesn't have any side effects, as far as I can tell.
Attachments (1)
Change History (8)
Changed 16 years ago by
| Attachment: | myth_repaint_ui.diff added |
|---|
comment:1 Changed 16 years ago by
| Resolution: | → wontfix |
|---|---|
| Status: | new → closed |
comment:2 Changed 16 years ago by
| Owner: | changed from Isaac Richards to stuartm |
|---|
Which theme are you using?
comment:3 Changed 16 years ago by
I'm using Retro, and I don't think it has any animations, does it? The same goes for the other (conventional) themes I've used.
The point is, even if 5Hz is insufficient for animations, repainting the UI 70 times per second all the time (as even mythwelcome does) is unnecessary, and interferes with the power saving features of the kernel and the hardware.
comment:4 Changed 16 years ago by
| Resolution: | wontfix |
|---|---|
| Status: | closed → new |
We shouldn't be repainting all the time, when it was pointed it out to me that we might be even with no animations I thought I better check which theme you were using.
Which painter are you using?
comment:5 Changed 16 years ago by
I use Qt, although OpenGL produces the same results wrt the number of wakeups.
comment:6 Changed 16 years ago by
| Summary: | reduce repaint interval of mythtv ui → reduce dirty region check interval of mythtv ui |
|---|
Ok, so it's not repaints which are causing the wakeups it's the draw timer, which only triggers a repaint if something has changed.
comment:7 Changed 16 years ago by
| Resolution: | → wontfix |
|---|---|
| Status: | new → closed |
The patch is wrong, but if someone wants to implement the correct solution by replacing polling entirely then I'll look at this again.
Aside from making animation impossible.