[ofa-general] Re: valgrind warnings in libibverbs & PVFS

Pete Wyckoff pw at osc.edu
Mon Mar 10 06:23:02 PDT 2008


rdreier at cisco.com wrote on Sun, 09 Mar 2008 20:58 -0700:
>  > This change makes sense, although I probably would have used &resp,
>  > sizeof(resp).  There are a bunch of places that use
>  > IBV_INIT_CMD_RESP, some of which have VALGRIND annotations too.  But
>  > there's a bunch that do not.  Would make a nice cleanup to do them
>  > all.
> 
> I just audited my tree, and unless I'm missing something, Troy's case
> in verbs.c was the last place that still needed annotation.  Or am I
> blind?

I think you found them all.  My comment was made when looking at
v1.1.1, missing a couple of recent patches from Dotan addressing
the same issue.

		-- Pete



More information about the general mailing list