1330697 - verify xperf_whitelist.json entries are still needed

Status: RESOLVED FIXED

Description

[Joel Maher ( :jmaher ) (UTC-4)]

right now we have a long laundry list of files that we expect to be accessed on startup (measured by the windows xperf talos test):
https://dxr.mozilla.org/mozilla-central/source/testing/talos/talos/xtalos/xperf_whitelist.json?q=path%3Axperf_whitelist.json&redirect_type=single#16

I suspect some of these files are not needed anymore, keeping this list up to date would be important.

we also have mainthreadIO fileIO measurements in this file:
https://dxr.mozilla.org/mozilla-central/source/testing/talos/talos/mtio-whitelist.json

again, keeping that list up to date (i.e. removing old entries) seems important.

as a note, sometimes things are intermittent or pgo only (bug 1324062, comment 79)

Comment 1

[Joel Maher ( :jmaher ) (UTC-4)]

a meager attempt to do this for xperf_whitelist.json:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=8008b1bb89bc7a402728f460ea2382d7c3f62ac8

Comment 2

[Joel Maher ( :jmaher ) (UTC-4)]

Attachment: xperf_sanity.patch

remove some obvious xperf_whitelist.json entries.

a failure case here:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=8008b1bb89bc7a402728f460ea2382d7c3f62ac8

adding those missing ones back in here:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=20cc8bc16b8a9bdc55776affd8b3f6937af1fd9e

Comment 3

[Robert Wood [:rwood]]

Comment on attachment 8826280 [details] [diff] [review]
xperf_sanity.patch

LGTM, R+ pending green on try

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/e8ae38b46276
remove old xperf_whitelist.json entries. r=rwood

Comment 5

[Joel Maher ( :jmaher ) (UTC-4)]

I had mentioned mainthread_io, I had removed almost all the entries in a try server push and we are not detecting anything- possibly this isn't running anymore.

Comment 6

[Joel Maher ( :jmaher ) (UTC-4)]

as a note, I removed what I thought would be easy to remove, possibly other files could be removed, although I believe it would be better served in the future as an annual cleanup.

Comment 7

[Wes Kocher (:KWierso)]

I had to back this out because some Windows pgo xperf runs failed with it in: https://treeherder.mozilla.org/logviewer.html#?job_id=68825854&repo=mozilla-inbound

https://hg.mozilla.org/integration/mozilla-inbound/rev/4ef05cc6af4e

Comment 8

[Joel Maher ( :jmaher ) (UTC-4)]

thanks for backing this out, I am testing a patch on pgo and non pgo to validate all is well.  it looks like a small chunk of my removals are needed.

Comment 9

[Joel Maher ( :jmaher ) (UTC-4)]

Attachment: cleanup old xperf whitelist file

lets try this again, now I made sure to test on pgo before calling this good.  Removed some old entries, put things in a more logical order, and added some new entries (for intermittent issues)

Comment 10

[Robert Wood [:rwood]]

Comment on attachment 8829207 [details] [diff] [review]
cleanup old xperf whitelist file

LGTM

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/e0f2dbc7c3cb
verify xperf_whitelist.json entries are still needed. r=rwood

Comment 12

[Wes Kocher (:KWierso)]

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