Update rkv to 0.16.0
Categories
(Data Platform and Tools :: Glean: SDK, task, P1)
Tracking
(Not tracked)
People
(Reporter: janerik, Assigned: janerik)
References
Details
(Whiteboard: [telemetry:fog:m6])
Attachments
(4 files)
We need to upgrade rkv, then land that update in m-c in coordination with other users of rkv.
Comment 1•4 years ago
|
||
Comment 2•4 years ago
|
||
What's the process? Update glean, publish it to crates, then update rkv and glean together in m-c?
Assignee | ||
Comment 3•4 years ago
|
||
Yup, though we're landing another big Glean update right now, so that needs to happen first.
Comment 4•4 years ago
|
||
Depends on D98482
Comment 5•4 years ago
|
||
Depends on D98594
Comment 7•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 8•4 years ago
|
||
We should have made its own bug for the code you're landing, :vporof. Renaming this bug.
We can keep it here for now, but in the future we should probably track these separately.
Comment 9•4 years ago
|
||
It's a little bit the other way around – the tsan failure is very likely caused by updating to 0.16, which is an issue that I'm looking into fixing right now. I agree with renaming this bug.
Assignee | ||
Comment 10•4 years ago
|
||
Unfortunately the lock-order-inversion
failure seems to be in FOG code.
We might have just been unlucky to find it here.
Assignee | ||
Comment 11•4 years ago
|
||
Talked to Victor in private.
It looks like by upgrading we resurfaced a false-positive (bug 1606804) due the to the change in paths for only Glean/FOG (because we duplicate rkv versions).
A Glean upgrade is coming today or tomorrow also upgrading its rkv dependency, that should fix this without requiring yet another exception for TSAN (as done here: https://searchfox.org/mozilla-central/rev/c14733e74e540b71856cc1d963bc424b38aa4bef/mozglue/build/TsanOptions.cpp#101-102)
Taking over the bug to get that update in.
Comment 12•4 years ago
|
||
Thanks Jan-Erik!
Assignee | ||
Comment 13•4 years ago
|
||
Assignee | ||
Updated•4 years ago
|
Comment 14•4 years ago
|
||
Comment 15•4 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/02dced2018db
https://hg.mozilla.org/mozilla-central/rev/2e99b5099cb5
https://hg.mozilla.org/mozilla-central/rev/55d02702cb10
Description
•