[pulseaudio-discuss] [PATCH] Add a Valgrind suppression file

David Henningsson david.henningsson at canonical.com
Sun Sep 21 02:25:16 PDT 2014



On 2014-09-21 08:08, Tanu Kaskinen wrote:
> On Fri, 2014-09-19 at 14:34 +0200, David Henningsson wrote:
>> Cool, I did not know this was possible. So how does valgrind know that
>> it should pick this file up and use it? Does it also work with
>> out-of-tree builds?
>
> Valgrind picks up the file when you give the option
> --suppressions=<file>. I don't see why this wouldn't work in out-of-tree
> builds, since the matching is (in this case) made only based on the
> function name where the error happens.

Right.

>> Could there be a comment in the file as well what it
>> is about and how to use it?
>
> Probably. I didn't notice any information about comments when I read the
> syntax documentation, but I didn't particularly look for that either.
> I'll add a comment in v2 if I figure out how to do that.

Looking at /usr/lib/valgrind/default.supp, it seems like # indicates a 
comment line.

(Or maybe we could even get some valgrind developer to fix the root 
cause, that would be even better...maybe we can grab someone in 
Düsseldorf, if they're present?)

-- 
David Henningsson, Canonical Ltd.
https://launchpad.net/~diwic


More information about the pulseaudio-discuss mailing list