[mythtv] [mythtv-commits] Ticket #7525: Optimize Pulse handling in MythUI

[Jim Stichnoth]

On Sat, Aug 21, 2010 at 11:00 PM, Gavin Hurlbut <gjhurlbu at gmail.com> wrote:
> Jim (if you are listening):
> I have been trying this with current trunk, and it doesn't seem to be
> operational anymore.  In the program listing, the alphapulse "Recording..."
> in Arclight gets stuck on permanently, and doesn't ever seem to change
> alpha.
> On the good side (for me at least), the CPU usage is great...  2% rather
> than 48% of a Pentium-D...  but that's because alphapulse isn't pulsing
> anymore.
> Could you update this for current trunk and test again?

I fixed the alphapulse bug in the patch, as you probably saw.

I maintain this patch and run my production systems with it, which is
easy enough to do since that part of the code doesn't change much.
However, I really don't find much benefit.  Even without the patch, it
really doesn't take much processing to walk the tree every 1/70 of a
second and verify that nothing needs pulsing.  On an Atom, real
pulsing consumes >50% CPU on one core, which is an entirely separate
problem (if you want to call it a problem).  The extra complexity of
having to insert an extra call every time an object field changes that
affects the pulsing decision, just doesn't seem worth the minimal
performance gain.

Jim