[openib-general] RE: Separating SA and SM Keys

[Eitan Zahavi]

Hi Hal,

OpenSM can support 2 configuration flags. 
This is different then the wire protocol described in ib_types.h which
should follow the IB spec.

Eitan Zahavi
Design Technology Director
Mellanox Technologies LTD
Tel:+972-4-9097208
Fax:+972-4-9593245
P.O. Box 586 Yokneam 20692 ISRAEL


> -----Original Message-----
> From: Hal Rosenstock [mailto:halr at voltaire.com]
> Sent: Tuesday, December 20, 2005 1:45 PM
> To: Eitan Zahavi
> Cc: Yael Kalka; openib-general at openib.org
> Subject: RE: Separating SA and SM Keys
> 
> Hi Eitan,
> 
> On Tue, 2005-12-20 at 03:17, Eitan Zahavi wrote:
> > Hi Hal,
> >
> > I think we need to stick to the IB spec terminology.
> > Since the spec did not change and only added some note describing
the
> > change we probably just need to add such comment too.
> 
> Ideally the spec would have been changed.
> 
> > I can see people looking at the sa_key and trying to find it in the
> > spec...
> 
> How do you propose handling the setting of the 2 keys differently ?
They
> need different names at least for configuration purposes.
> 
> -- Hal
> 
> > Eitan
> >
> > Eitan Zahavi
> > Design Technology Director
> > Mellanox Technologies LTD
> > Tel:+972-4-9097208
> > Fax:+972-4-9593245
> > P.O. Box 586 Yokneam 20692 ISRAEL
> >
> >
> > > -----Original Message-----
> > > From: Hal Rosenstock [mailto:halr at voltaire.com]
> > > Sent: Monday, December 19, 2005 5:27 PM
> > > To: Eitan Zahavi; Yael Kalka
> > > Cc: openib-general at openib.org
> > > Subject: Separating SA and SM Keys
> > >
> > > Hi,
> > >
> > > In order to support separate SA and SM keys and make this clearer,
I
> > > propose to change ib_types.h as follows.
> > >
> > > -- Hal
> > >
> > > Index: ib_types.h
> > >
===================================================================
> > > --- ib_types.h  (revision 4540)
> > > +++ ib_types.h  (working copy)
> > > @@ -3618,7 +3618,7 @@ typedef struct _ib_sa_mad
> > >         ib_net32_t                              seg_num;
> > >         ib_net32_t                              paylen_newwin;
> > >
> > > -       ib_net64_t                              sm_key;
> > > +       ib_net64_t                              sa_key;
> > >
> > >         ib_net16_t                              attr_offset;
> > >         ib_net16_t                              resv3;