Markus Stange [:mstange] Reporter
	Description • 10 years ago

      No description provided.

	Joel Maher ( :jmaher ) (UTC -8) Assignee
	Comment 1 • 10 years ago

Comment on attachment 8551959 [details] [diff] [review]
patch

Review of attachment 8551959 [details] [diff] [review]:
-----------------------------------------------------------------

this is great.  2 things:
1) this also includes a change to make everything in pageloader use the messagemanager- so I expect numbers to change- ideally I would like to uplift this to aurora after we see the effects all this has
2) on the next update to talos.json, I planned on fixing the *snow* jobs to all have a --e10s, do you mind adding that to talos.json?  It has it for the chromez job, just not the other 3 jobs.  If not, not big deal.

	Markus Stange [:mstange] Reporter
	Comment 2 • 10 years ago

like this?

	Joel Maher ( :jmaher ) (UTC -8) Assignee
	Comment 3 • 10 years ago

Comment on attachment 8551971 [details] [diff] [review]
patch

Review of attachment 8551971 [details] [diff] [review]:
-----------------------------------------------------------------

thanks!

	Markus Stange [:mstange] Reporter
	Comment 4 • 10 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/675a7c4287a5

	Ryan VanderMeulen [:RyanVM]
	Comment 5 • 10 years ago

Backed out for xperf permafail. Did we actually run this through Try before pushing?
https://hg.mozilla.org/integration/mozilla-inbound/rev/3c1deb10f8eb

https://treeherder.mozilla.org/logviewer.html#?job_id=5673332&repo=mozilla-inbound

	Markus Stange [:mstange] Reporter
	Comment 6 • 10 years ago

Oh. Yes, we did, but not without profiling, and I had attributed those errors to profiling. Sorry.

Joel, it looks like we need to adjust the main thread IO whitelist for tscroll.js and Profiler.js because loadFrameScript does main thread IO (!?).

	Markus Stange [:mstange] Reporter
	Comment 7 • 10 years ago

So, there's both xperf_whitelist.json and mtio-whitelist.json. Which one of those needs changing?

	Joel Maher ( :jmaher ) (UTC -8) Assignee
	Comment 8 • 10 years ago

this just needs mtio-whitelist.json changes, should be easy to add, push to try and verify.  Lets keep track in this bug, I can get 15 minutes a bit later tonight to do it if others don't beat me to it.

	Joel Maher ( :jmaher ) (UTC -8) Assignee
	Comment 9 • 10 years ago

https://tbpl.mozilla.org/?tree=Try&rev=0595181a43b5

	Joel Maher ( :jmaher ) (UTC -8) Assignee
	Comment 11 • 10 years ago

https://hg.mozilla.org/build/talos/rev/39c9c9b33cfc

	Joel Maher ( :jmaher ) (UTC -8) Assignee
	Comment 13 • 10 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/dcce08cf9c7d

	Markus Stange [:mstange] Reporter
	Comment 14 • 10 years ago

Thanks Joel!

	Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)
	Comment 15 • 10 years ago

https://hg.mozilla.org/mozilla-central/rev/dcce08cf9c7d

	Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)
	Comment 16 • 10 years ago

Backed out in https://hg.mozilla.org/integration/mozilla-inbound/rev/568bbf124221 for causing bug 1124740.

	Joel Maher ( :jmaher ) (UTC -8) Assignee
	Comment 17 • 9 years ago

the root cause for this is http://hg.mozilla.org/build/talos/rev/d6bcfd393641.  bummer, but we can remove this and land again.  There is on other patch which fixes a font issue for xperf file access which needs to land and get deployed.

	Joel Maher ( :jmaher ) (UTC -8) Assignee
	Comment 18 • 9 years ago

updated talos.json after backing out bustage patch for pgo.  Try server runs:
non-pgo: https://treeherder.mozilla.org/#/jobs?repo=try&revision=16fe6f022b9d
pgo: https://treeherder.mozilla.org/#/jobs?repo=try&revision=190fa2dcb1b2

	Ryan VanderMeulen [:RyanVM]
	Comment 19 • 9 years ago

Comment on attachment 8554153 [details] [diff] [review]
talos.json modifications with only updated

rs=me

	Joel Maher ( :jmaher ) (UTC -8) Assignee
	Comment 20 • 9 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/babd56077826

	Ryan VanderMeulen [:RyanVM]
	Comment 21 • 9 years ago

https://hg.mozilla.org/mozilla-central/rev/babd56077826