On Wed, 2012-05-30 at 11:06 -0700, Hal Rosenstock wrote:
> On 5/30/2012 1:21 PM, Jim Foraker wrote:
> > Defines a default for the smkey used in SA requests
> > 
> > Signed-off-by: Jim Foraker <[email protected]>
> > ---
> >  etc/ibdiag.conf     |    3 +++
> >  src/ibdiag_common.c |    3 +++
> >  2 files changed, 6 insertions(+)
> > 
> > diff --git a/etc/ibdiag.conf b/etc/ibdiag.conf
> > index 2a2334f..1e6149d 100644
> > --- a/etc/ibdiag.conf
> > +++ b/etc/ibdiag.conf
> > @@ -17,3 +17,6 @@
> >  
> >  # define a default m_key
> >  #m_key=0x00
> > +
> > +# default smkey to be used for SA requests
> > +#sa_smkey=0x00
> > diff --git a/src/ibdiag_common.c b/src/ibdiag_common.c
> > index c40490a..1301810 100644
> > --- a/src/ibdiag_common.c
> > +++ b/src/ibdiag_common.c
> > @@ -160,6 +160,9 @@ void read_ibdiag_config(const char *file)
> >                     }
> >             } else if (strncmp(name, "m_key", strlen("m_key")) == 0) {
> >                     ibd_mkey = strtoull(val_str, 0, 0);
> > +           } else if (strncmp(name, "sa_smkey",
> > +                              strlen("sa_smkey")) == 0) {
> 
> For consistency with OpenSM naming, how about sa_key for this ?
     That's fine by me.  IIRC, that was actually my first choice, but I
thought there might be some objections, since it's called smkey in the
spec.

     Jim

> 
> -- Hal
> 
> > +                   ibd_sa_smkey = strtoull(val_str, 0, 0);
> >             }
> >     }
> >  
> 
> --
> 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

--
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

Reply via email to