Bug 179107 – mozilla-firefox-2 wastes power in loop

• Product: Gentoo Linux
• Status: RESOLVED UPSTREAM

[Description Ryan Hope 2007-05-19 14:20:43 UTC]

Firefox had 2 condition variables that were waited on with a timeout of 250
miliseconds in a loop.
It appears that the author of the code uses 250 milliseconds as "inifite",
however, the PR_WaitCondVar() API has an actual value for inifite,
PR_INTERVAL_NO_TIMEOUT. Use that instead.

Reproducible: Always

[Comment 1 Ryan Hope 2007-05-19 14:21:43 UTC]

Created attachment 119706 [details, diff]
A patch to fix timer problem

[Comment 2 Ryan Hope 2007-05-19 14:22:43 UTC]

Created attachment 119707 [details, diff]
A diff of the changed required in mozilla-filrefox-2.0.0.3.ebuild

[Comment 3 Jakub Moc (RETIRED) 2007-05-19 14:27:27 UTC]

Did you submit this upstream? Apparently not a Gentoo-specific issue.

https://bugzilla.mozilla.org/enter_bug.cgi?product=Firefox&format=guided

[Comment 4 Raúl Porcel (RETIRED) 2007-05-19 15:53:41 UTC]

Send this to upstream.

[Comment 5 Petteri Räty (RETIRED) 2007-05-19 15:55:15 UTC]

(In reply to comment #3)
> Did you submit this upstream? Apparently not a Gentoo-specific issue.
> 
> https://bugzilla.mozilla.org/enter_bug.cgi?product=Firefox&format=guided
> 

The patches are are the result of the Powertop work from Intel:
http://www.linuxpowertop.org/known.php#firefox

Next time please tell where you go the patches when you didn't write them yourself. The upstream bug report is:
https://bugzilla.mozilla.org/show_bug.cgi?id=380558

[Comment 6 Jakub Moc (RETIRED) 2007-06-18 19:42:16 UTC]

*** Bug 182490 has been marked as a duplicate of this bug. ***