Use RKV safe mode for xulstore
Categories
(Toolkit :: Storage, enhancement, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox85 | --- | fixed |
People
(Reporter: vporof, Assigned: vporof)
References
Details
Attachments
(3 files)
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Comment 1•4 years ago
|
||
Updated•4 years ago
|
Assignee | ||
Comment 2•4 years ago
|
||
Depends on D85314
Comment 4•4 years ago
|
||
Backed out 2 changesets (Bug 1654192) for causing failures in browser_startup_mainthreadio.js CLOSED TREE
Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=311444845&repo=autoland&lineNumber=2491
https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=311449853&repo=autoland&lineNumber=1783
Backout: https://hg.mozilla.org/integration/autoland/rev/2276446c81d6afe0efe07040a9f584ffc6b04792
Assignee | ||
Updated•4 years ago
|
Comment 6•4 years ago
|
||
Backed out 2 changesets (bug 1654192) for Talos failures in \xulstore\data.safe.bin. CLOSED TREE
Log:
https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=311666256&repo=autoland&lineNumber=1748
Push with failures:
https://treeherder.mozilla.org/#/jobs?repo=autoland&group_state=expanded&revision=bad6faf26935ebcc07e602769e43286ddd5d183a
Backout:
https://hg.mozilla.org/integration/autoland/rev/2038f000aed1471bb91ff711cabb3e32b7982279
Comment 7•4 years ago
|
||
There are some r+ patches which didn't land and no activity in this bug for 2 weeks.
:vporof, could you have a look please?
For more information, please visit auto_nag documentation.
Updated•4 years ago
|
Assignee | ||
Comment 9•4 years ago
|
||
Depends on D85315
Comment 10•4 years ago
|
||
(In reply to Dorel Luca [:dluca] from comment #6)
Backed out 2 changesets (bug 1654192) for Talos failures in \xulstore\data.safe.bin. CLOSED TREE
Log:
https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=311666256&repo=autoland&lineNumber=1748Push with failures:
https://treeherder.mozilla.org/#/jobs?repo=autoland&group_state=expanded&revision=bad6faf26935ebcc07e602769e43286ddd5d183aBackout:
https://hg.mozilla.org/integration/autoland/rev/2038f000aed1471bb91ff711cabb3e32b7982279
:dluca, is there any way I can see that this patch no longer causes the failures before it was backed out? If there's a push to try please guide :vporof through that.
Assignee | ||
Comment 11•4 years ago
|
||
Assignee | ||
Updated•4 years ago
|
Comment 12•4 years ago
|
||
Seems like the push is failing talos slightly different from the previous \xulstore\data.mdb
https://treeherder.mozilla.org/jobs?repo=try&revision=4c53597c41c86e4de8f0a16f23998375fb2a7cf8&searchStr=talos
Assignee | ||
Comment 13•4 years ago
|
||
Right, bizarre that didn't show up in my try run.
So what's happening is that we've transitioned to using a different storage backend, and therefore a one-time migration needs to happen before the data can be accessed. In other words, we went from '\xulstore\data.mdb' to the newly added '\xulstore\data.safe.bin'.
We plan to remove the old storage backend fairly soon in the future. But even if we don't, to be clear, the old data is accessed only one for the one-time migration. And it won't happen if there's no 'data.mdb' available (which is the case for users on release – this has only been running in nightly and beta).
I would expect it's reasonable to keep the old '\xulstore\data.mdb' in 'xtalos/xperf_whitelist.json' in the meantime?
Comment 14•4 years ago
|
||
I added them because your patches were backed out because of these intermittents so I wanted to make sure the reason why they were backed out is gone.
If you think that is reasonable and will get rid of the intermittents, go ahead.
Assignee | ||
Comment 15•4 years ago
|
||
Alright, this is ready for a re-review.
Try: https://treeherder.mozilla.org/jobs?repo=try&revision=26ddf28433b4d7f13267e16fc78ddc664b7a9d84
Assignee | ||
Comment 16•4 years ago
|
||
Another try, still green: https://treeherder.mozilla.org/jobs?repo=try&revision=acf49fa546a9e43cfdb2fb4fe69180214fa307f1
Comment 17•4 years ago
|
||
I can see in the comments your question has been answered, removed NI as such.
Updated•4 years ago
|
Comment 18•4 years ago
|
||
Comment 19•4 years ago
|
||
Backed out 5 changesets (Bug 1654192, Bug 1680230) for causing tsan bc failures in RWLock.
Backout link: https://hg.mozilla.org/integration/autoland/rev/821f3a55cb71c53d361d962c5060c972ef34e52b
Failure log: https://treeherder.mozilla.org/logviewer?job_id=323742182&repo=autoland&lineNumber=2293
Assignee | ||
Comment 20•4 years ago
|
||
Thanks! The error is isolated to bug 1680230, not this one, but the two should land simultaneously. On it!
Comment 21•4 years ago
|
||
Comment 22•4 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/54d791bb417b
https://hg.mozilla.org/mozilla-central/rev/16f75cdb5ea0
https://hg.mozilla.org/mozilla-central/rev/15a3d7e84a53
Comment 23•4 years ago
|
||
== Change summary for alert #28050 (as of Fri, 11 Dec 2020 05:44:11 GMT) ==
Improvements:
Ratio | Suite | Test | Platform | Options | Absolute values (old vs new) |
---|---|---|---|---|---|
2% | startup_about_home_paint_realworld_webextensions | startup_about_home_paint_realworld_webextensions | linux64-shippable-qr | e10s stylo webrender | 843.92 -> 823.50 |
2% | startup_about_home_paint_realworld_webextensions | startup_about_home_paint_realworld_webextensions | linux64-shippable-qr | e10s stylo webrender | 843.25 -> 826.08 |
For up to date results, see: https://treeherder.mozilla.org/perfherder/alerts?id=28050
Assignee | ||
Comment 24•4 years ago
|
||
Hey, 2% is 2%. Add that to the 4-7% from bug 1594995, at least it's something.
Description
•