1082205 - xperf is permaorange on mozilla-beta, " mainthreadio | File '{profile}\pluginreg.dat.tmp' was accessed and we were not expecting it"

Status: RESOLVED FIXED

(Testing :: Talos, defect)

Description

[Phil Ringnalda (:philor)]

See, e.g., https://treeherder.mozilla.org/ui/logviewer.html#?job_id=98931&repo=mozilla-beta

mozilla-beta is closed.

Comment 1

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

Aaron,

this was live on aurora, but now with the uplift to beta we have problems; any idea why we would have that?

:philor, you do not need to close the tree for this; we can add this to a whitelist, although it would be nice to know why we see this only on beta.

Comment 2

[Phil Ringnalda (:philor)]

Oh, okay. Did you want me to hide xperf on beta instead of closing the tree, then? Those are your choices, there's no "oh, we'll just leave the known permaorange there and look at it every time to see whether it's different permaorange or just the same permaorange" option.

Comment 3

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

lets hide xperf for now;

Comment 4

[Phil Ringnalda (:philor)]

Done.

Comment 5

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

Attachment: whitelist more files in mainthread io (1.0)

so it appears we have some new files between this bug and bug 1081354:
* "{profile}\\pluginreg.dat.tmp": {}
* all parent dirs of the profile
* I am ignoring these from bug 1081354:
TEST-UNEXPECTED-FAIL | mainthreadio | File '{profile}\startupcache\startupcache.4.little' was accessed and we were not expecting it: 'Duration 11.795327 > 1.0'
TEST-UNEXPECTED-FAIL | mainthreadio | File '{profile}\telemetry.shutdowntime.txt.tmp' was accessed and we were not expecting it: 'Duration 11.386187 > 1.0'
TEST-UNEXPECTED-FAIL | mainthreadio | File '{profile}' was accessed and we were not expecting it: 'Duration 6.583649 > 1.0'

those 3 errors are related to Duration, not to access; should we allow those durations to be larger?

Comment 6

[(No longer employed by Mozilla) Aaron Klotz]

Comment on attachment 8504809 [details] [diff] [review]
whitelist more files in mainthread io (1.0)

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

Sorry for the delay.

Comment 7

[(No longer employed by Mozilla) Aaron Klotz]

It's a reasonable file to whitelist, but I'm not sure why we're only seeing it now.

Comment 8

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

https://hg.mozilla.org/build/talos/rev/04081c46ff03