I am a bit concerned about how this might affect callers of the function but I can't find anywhere that it would be an issue in infiniband-diags.
We will go with it and see if anything comes up. Applied, thanks, Ira On Tue, 5 Apr 2011 12:23:28 -0700 Hal Rosenstock <[email protected]> wrote: > > smp_query_via() does not allow setting the sent mad data. > Change the data to be both in and out arguments. > > Signed-off-by: Oren Kladnitsky <[email protected]> > Signed-off-by: Hal Rosenstock <[email protected]> > --- > Changes since v1: > Proper formatting > > diff --git a/src/smp.c b/src/smp.c > index 3ff58ce..2809ffe 100644 > --- a/src/smp.c > +++ b/src/smp.c > @@ -102,7 +102,7 @@ uint8_t *smp_query_via(void *rcvbuf, ib_portid_t * > portid, unsigned attrid, > portid->sl = 0; > portid->qp = 0; > > - return mad_rpc(srcport, &rpc, portid, 0, rcvbuf); > + return mad_rpc(srcport, &rpc, portid, rcvbuf, rcvbuf); > } > > uint8_t *smp_query(void *rcvbuf, ib_portid_t * portid, unsigned attrid, > -- > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > the body of a message to [email protected] > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Ira Weiny Math Programmer/Computer Scientist Lawrence Livermore National Lab 925-423-8008 [email protected] -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
