[openib-general] RE: [PATCH] OpenSM/ib_types.h: Modify ib_port_info_compute_rate

Hal Rosenstock halr at voltaire.com
Sun Dec 18 07:48:04 PST 2005


Hi Eitan,

On Sun, 2005-12-18 at 06:53, Eitan Zahavi wrote:
> Hi Hal,
> 
> The attached patch is fine. Please go ahead and commit it.
> 
> BTW: 
> In the following commit 4509 you have changed the name of a switch info
> record field.
> Note this is an API change and have severe effect on any application
> using ib_types.h
> (and there are plenty of these)

Is this "API" frozen for all time ? How would you propose that this
"API" evolve ? I do not see where there is any versioning to the API.

> I would appreciate if you will revert this un-necessary change.

I reverted this change.

> Also in
> the future please post a patch before changing ib_types.h
> osm_vendor_api.h osm_vendor_sa_api.h  and any of the complib H files.

The previous request on this was for any non cosmetic changes. This was
viewed as a cosmetic change (a simple variable name change).I'll now
post any changes to these header files.

-- Hal




More information about the general mailing list