[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