Closed
Bug 1656065
Opened 4 years ago
Closed 4 years ago
SUMMARY: ThreadSanitizer: data race /builds/worker/checkouts/gecko/media/mtransport/third_party/nrappkit/src/log/r_log.c:349:8 in r_vlog
Categories
(Core :: WebRTC: Networking, defect, P5)
Core
WebRTC: Networking
Tracking
()
RESOLVED
FIXED
81 Branch
Tracking | Status | |
---|---|---|
firefox81 | --- | fixed |
People
(Reporter: intermittent-bug-filer, Assigned: bwc)
References
Details
Attachments
(1 file)
Filed by: bcampen [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer.html#?job_id=311427428&repo=try
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/ML1-bUG7S_uyOMyYy3NqEw/runs/0/artifacts/public/logs/live_backing.log
Assignee | ||
Updated•4 years ago
|
Blocks: mochitest_media_tsan
Assignee | ||
Comment 1•4 years ago
|
||
Looks like we're racing on this:
Looks pretty much benign, but we may be racing against other parts of r_log_init.
Assignee | ||
Comment 2•4 years ago
|
||
So it looks like we run this code in _r_log_init over and over again:
We should probably be doing this only once, right?
Flags: needinfo?(ekr)
Comment 3•4 years ago
|
||
Yeah.
r_log_init() is only supposed to be called once, I think. Or we should add an "i have already been called" thing.
Flags: needinfo?(ekr)
Assignee | ||
Comment 4•4 years ago
|
||
Assignee | ||
Comment 5•4 years ago
|
||
Depends on D85295
Updated•4 years ago
|
Assignee: nobody → docfaraday
Status: NEW → ASSIGNED
Assignee | ||
Comment 6•4 years ago
|
||
Try looks good.
Comment hidden (Intermittent Failures Robot) |
Pushed by bcampen@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/f39ddb6e8d4c Init r_log_env_verbose only once. r=mjf
Comment 9•4 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
status-firefox81:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 81 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•