[ofa-general] Re: [PATCH] opensm: add osm_version field to osm_opensm_t object

Sasha Khapyorsky sashak at voltaire.com
Fri Jun 27 10:50:24 PDT 2008


On 10:10 Thu 26 Jun     , Hal Rosenstock wrote:
> On Thu, 2008-06-26 at 01:21 +0300, Sasha Khapyorsky wrote:
> > This adds osm_version field to osm_opensm_t object.
> 
> Is part of the intention here to use this in plugins ?

It can be used in plugin for OpenSM version checking (like it is done
in osmeventplugin in event plugin API v2 patch), but I didn't think to
limit its usage by something specific.

> Is this
> versioning of osm_opensm_t for that (and other uses too) ?

No, it is OpenSM version itself.

> If so, that may be a little too granular IMO. Perhaps it should be more
> like the libraries in terms of versioning.

Yes, it could be useful to do so if it is about API/ABI versioning, but
here it is just OpenSM version as is.

Sasha



More information about the general mailing list