Closed Bug 1158227 Opened 9 years ago Closed 9 years ago

make running tests with TSan more ergonomic

Categories

(Testing :: General, defect)

defect
Not set
normal

Tracking

(firefox40 fixed)

RESOLVED FIXED
mozilla40
Tracking Status
firefox40 --- fixed

People

(Reporter: froydnj, Assigned: froydnj)

Details

Attachments

(2 files)

      No description provided.
This is exactly what we do for ASan, but with TSan flavoring.
Attachment #8597339 - Flags: review?(jmaher)
This setting is done mostly to encourage TSan to use llvm-symbolizer, which
does a better job of producing stack traces.

I see that ASAN_OPTIONS is set similarly in build/automation.py.in...do I need
to set TSAN_OPTIONS there as well?
Attachment #8597342 - Flags: review?(jmaher)
Comment on attachment 8597339 [details] [diff] [review]
part 1 - don't run TSan on test tools or the xpcshell HTTP server in mochitests

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

oh more *san stuff :)
Attachment #8597339 - Flags: review?(jmaher) → review+
Comment on attachment 8597342 [details] [diff] [review]
part 2 - set TSAN_OPTIONS environment variable in automationutils.py for TSan

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

cool;
Attachment #8597342 - Flags: review?(jmaher) → review+
https://hg.mozilla.org/mozilla-central/rev/6db78683fa3e
https://hg.mozilla.org/mozilla-central/rev/cc1ce0be9226
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: