Widen suppressoin for 101853

Widen suppressoin for 101853
Add more race/overlaps for 93932.
Renamed 93932 issues in mac to have _mac_ as same suppression exists on suppression.txt

TBR=acolwell@chromium.org,rvargas@chromium.org
BUG=101853, 93932
TEST=none

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=107693

[Timur Iskhodzhanov, 2011-10-28 08:26:20]

http://codereview.chromium.org/8413028/diff/1/tools/valgrind/tsan/suppression...
File tools/valgrind/tsan/suppressions.txt (right):

http://codereview.chromium.org/8413028/diff/1/tools/valgrind/tsan/suppression...
tools/valgrind/tsan/suppressions.txt:635: <Put your suppression name here>
baaah

http://codereview.chromium.org/8413028/diff/1/tools/valgrind/tsan/suppression...
tools/valgrind/tsan/suppressions.txt:641:
fun:media::FFmpegVideoDecodeEngine::DecodeFrame
doesn't this match bug_92932_c ?

http://codereview.chromium.org/8413028/diff/1/tools/valgrind/tsan/suppression...
tools/valgrind/tsan/suppressions.txt:649: <Put your suppression name here>
boo

http://codereview.chromium.org/8413028/diff/1/tools/valgrind/tsan/suppression...
tools/valgrind/tsan/suppressions.txt:652: fun:ff_thread_decode_frame
doesn't that match bug_93932_b?

http://codereview.chromium.org/8413028/diff/1/tools/valgrind/tsan/suppression...
File tools/valgrind/tsan/suppressions_mac.txt (right):

http://codereview.chromium.org/8413028/diff/1/tools/valgrind/tsan/suppression...
tools/valgrind/tsan/suppressions_mac.txt:176: bug_93932_mac_b
any reason to have Mac-only suppression duplicating the cross-platform one?

are you aware tsan/suppressions.txt is used on all platforms
+ Mac is using tsan/suppressions_mac.txt ?

http://codereview.chromium.org/8413028/diff/1/tools/valgrind/tsan/suppression...
tools/valgrind/tsan/suppressions_mac.txt:213: Memcheck:Overlap
Memcheck doesn't check this file - please remove

[oshima, 2011-10-28 17:17:26]

Oops, sorry for mess. I found them late night and did it just before I go to
bed. Andrew actually told me about name problem, and he said he will submit fix
soon, so I didn't look it again.

http://codereview.chromium.org/8413028/diff/1/tools/valgrind/tsan/suppression...
File tools/valgrind/tsan/suppressions.txt (right):

http://codereview.chromium.org/8413028/diff/1/tools/valgrind/tsan/suppression...
tools/valgrind/tsan/suppressions.txt:635: <Put your suppression name here>
These seems to be cut&pasted without my intention. Sorry about that.

http://codereview.chromium.org/8413028/diff/1/tools/valgrind/tsan/suppression...
File tools/valgrind/tsan/suppressions_mac.txt (right):

http://codereview.chromium.org/8413028/diff/1/tools/valgrind/tsan/suppression...
tools/valgrind/tsan/suppressions_mac.txt:176: bug_93932_mac_b
On 2011/10/28 08:26:20, Timur Iskhodzhanov wrote:
> any reason to have Mac-only suppression duplicating the cross-platform one?

The particular error was happening on mac only because the stack was slightly
different from 93932 in suppression.txt. I added to mac first, then realized
that there is similar one in suppression.txt. I updated 93932 but apparently
I forgot to remove this one.

> are you aware tsan/suppressions.txt is used on all platforms
> + Mac is using tsan/suppressions_mac.txt ?

yes

http://codereview.chromium.org/8413028/diff/1/tools/valgrind/tsan/suppression...
tools/valgrind/tsan/suppressions_mac.txt:213: Memcheck:Overlap
On 2011/10/28 08:26:20, Timur Iskhodzhanov wrote:
> Memcheck doesn't check this file - please remove

Sorry about that. I'll be careful next time.
One note: waterfall.sh didn't complain about this, and did hide this error. It'd
be nice if waterfall.sh can catch this type of error.