New valgrind results for Oct 21, 2009

Dan Kegel dank at kegel.com
Thu Oct 22 07:21:04 CDT 2009


I added suppressions for a few frequent spurious warnings and reran,
the updated results are in
 http://kegel.com/wine/valgrind/logs/2009-10-21-19.42.log
and broken out by test in
 http://kegel.com/wine/valgrind/logs/2009-10-21-19.42/
Please send me the suppression record for any warnings that are
spurious or benign, and mention the reason for each.

There's also an overall diff from the previous run,
http://kegel.com/wine/valgrind/logs/2009-10-21-19.42-diff.txt
but reading it is kind of a slog.

A few suggestions for people who would like to help fix these warnings:

1) Don't fix any error you don't fully understand.   Many
of these warnings are easy, but some are subtle.  When
in doubt, move on to an easier one.

2) Fixing leaks in test code is usually pretty safe, and
it's easier to get Alexandre to accept simple patches to tests,
so start with those if you don't have many patches under your belt.

3) the file
  http://kegel.com/wine/valgrind/logs/2009-10-21-19.42-count-by-file.txt
lists how many warnings are in each test; pick one with
just one error, and fix it.  Voila, one less failing test!

4) the file
 http://kegel.com/wine/valgrind/logs/2009-10-21-19.42-count-by-error.txt
lists how many of each warning there are.  Pick a warning
that happens a lot, and fix it.  Voila, lots fewer warnings!

5) if you know a warning is spurious, send it to me along with
the mangled version below it (that's in suppression format)
and why it's spurious or benign or not our fault, and I'll add it to
http://code.google.com/p/winezeug/source/browse/trunk/valgrind/valgrind-suppressions

Have fun, be careful, and thanks to everyone who's pitching in!
- Dan



More information about the wine-devel mailing list