[committed][gdb] Add tsan-suppressions.txt
Tom de Vries
tdevries@suse.de
Tue Jul 12 11:46:15 GMT 2022
On 7/12/22 12:31, Tom de Vries wrote:
> On 7/12/22 10:39, Tom de Vries wrote:
>> On 7/11/22 12:08, Tom de Vries wrote:
>>> Hi,
>>>
>>> Add a new file tsan-suppressions.txt, to suppress the "unlock
>>> unlocked mutex"
>>> problem in ncurses, filed in PR29328.
>>>
>>> For now, one needs to specify it manually using something like:
>>> ...
>>> $ export TSAN_OPTIONS=suppressions=$(pwd
>>> -P)/src/gdb/tsan-suppressions.txt
>>> ...
>>>
>>> Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=29328
>>>
>>> Any comments?
>>>
>>
>> I just found that we already set ASAN_OPTIONS in lib/gdb.exp, so I've
>> added this TSAN_OPTIONS setting alongside.
>
> I furthermore realized that suppression files are not cumulative, so if
> we have suppressions=a.txt:suppressions=b.txt then only b.txt is used.
>
> This would mean we ignore any user-specified suppression files, which
> may contains more suppressions for whatever reason.
>
> So I've updated the patch to only add the suppressions if the user
> didn't already do that.
Well, that makes the whole thing harmless enough, so committed.
Thanks,
- Tom
More information about the Gdb-patches
mailing list