-
Notifications
You must be signed in to change notification settings - Fork 122
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix 2 lock-order-inversion bugs as found by TSan on m-c tests #1378
Conversation
This fixes a potential deadlock as found by TSan, when running mozilla-central tests (`lock-order-inversion`). Locks should be acquired in the same order across all code paths.
This fixes a potential deadlock as found by TSan, when running mozilla-central tests (`lock-order-inversion`). Locks should be acquired in the same order across all code paths.
5a4c0c6
to
2881200
Compare
[Full changelog](https://github.com/mozilla/glean/compare/v33.7.0...main) | ||
|
||
* Rust | ||
* Shut down Glean in tests before resetting to make sure they don't mistakenly init Glean twice in parallel. | ||
* Introduce the Memory Distribution metric type in the RLB. [#1376](https://github.com/mozilla/glean/pull/1376) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I went ahead and fixed the changelog while I was here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great catches. Would you consider filing a bug for discussing enabling TSan test coverage for the glean SDK?
|
it's green! |
(Taking this for a try run in m-c to ensure it works before we land this)