1835280 - Consider increasing the stylo parallelism threshold.

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, defect)

Comment 1

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 1835280 - Double stylo parallelism threshold. r=bholley!,smaug!

This gets us wins on Android:

https://treeherder.mozilla.org/perfherder/comparesubtest?originalProject=try&newProject=try&newRevision=e3778acc9590726d0eff9f9e0b8556693ecb1e14&originalSignature=4696751&newSignature=4696751&framework=13&application=geckoview&originalRevision=46e0f500a3998d351f8c72b787d34414fc4edf25&page=1&showOnlyConfident=1

and Windows:

https://treeherder.mozilla.org/perfherder/comparesubtest?originalProject=try&newProject=try&newRevision=e3778acc9590726d0eff9f9e0b8556693ecb1e14&originalSignature=4702229&newSignature=4702229&framework=13&application=firefox&originalRevision=46e0f500a3998d351f8c72b787d34414fc4edf25&page=1&showOnlyConfident=1

Linux and macOS seem a bit worse but there were no non-low-confidence
results... Maybe I need more runs?

I think the idea is generally sound, tho, which is just not switching to
parallel mode unless we have at least enough work to keep two cores
busy...

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/51d7da4b2646
Double stylo parallelism threshold. r=bholley,smaug

Comment 3

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/51d7da4b2646

Comment 4

[Alexandru Ionescu (needinfo me) [:alexandrui]]

(In reply to Pulsebot from comment #2)

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/51d7da4b2646
Double stylo parallelism threshold. r=bholley,smaug

== Change summary for alert #38515 (as of Tue, 30 May 2023 14:44:38 GMT) ==

Improvements:

Ratio	Test	Platform	Options	Absolute values (old vs new)
4%	facebook-cristiano PerceptualSpeedIndex	android-hw-a51-11-0-aarch64-shippable-qr	warm webrender	709.92 -> 681.70

For up to date results, see: https://treeherder.mozilla.org/perfherder/alerts?id=38515