Experimental Physics and
Industrial Control System

Andrew Johnson <[email protected]> · Mon, 6 Apr 2015 10:22:12 -0500

Hi Michael,

On 04/03/2015 05:22 PM, Michael Davidsaver wrote:
> After discovering the (I hope) last memory leak in dbYacc.y and related
> I spent some time adding valgrind magic instructions to dbmf.c and
> freeListLib.c (more to come).
> 
> https://github.com/mdavidsaver/epics-base/commits/valgrind
> 
> Should/could we include valgrind.h in Base?  It's BSD licensed, and is a
> no-op on targets which valgrind doesn't support.
> 
> http://valgrind.org/docs/manual/manual-core-adv.html#manual-core-adv.clientreq

Looks possible, but I do have a couple of questions:

* Which version of the header we would include? On RHEL 6.6 I have
valgrind-3.8.1 installed, but I see the latest release is 3.10.1; does
the include file need to match the valgrind version? I assume not given
their instructions for it, but is it guaranteed to be both backwards and
forwards-compatible? Major version only? My version doesn't have 64-bit
Windows support but does have 32-bit MS, Cygwin and MinGW.

* Would we normally define NVALGRIND, so you'd have to explicitly turn
it on for debugging (maybe in our debug builds), or not and leave the
full macros in always?

Anyone else want to comment on this idea?

- Andrew

-- 
Light thinks it travels faster than anything but it is wrong.
No matter how fast light travels, it finds the darkness has
always got there first, and is waiting for it.
    -- Terry Pratchett, Reaper Man

Experimental Physics and Industrial Control System

Experimental Physics and
Industrial Control System