[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