[openib-general] [libibcm] compilation of code that uses libibcm may fail

[Hal Rosenstock]

On Mon, 2006-08-07 at 14:02, Sean Hefty wrote:
> Dotan Barak wrote:
> > enum ib_cm_sidr_status { 0, 1, 2, 3, 4, IB_SIDR_UNSUPPORTED_VERSION };
> > 
> > it seems that the enumerations values were replaced with integers.
> > when i searched for the values that were enumerated in the headre files i
> > found the following defines in ib_types.h:
> > 
> > #define IB_SIDR_SUCCESS                                         0 #define
> > IB_SIDR_UNSUPPORTED                                     1 #define
> > IB_SIDR_REJECT                                          2 #define
> > IB_SIDR_NO_QP                                           3 #define
> > IB_SIDR_REDIRECT                                        4
> > 
> > 
> > I think that the problem was that ib_types.h was included in a file that
> > includes the cm.h and the preprocessor replaced the enumeration names with
> > the integer values.
> > 
> > who can check this issue?
> 
> I think the solution is to remove CM definitions out of ib_types.h.

Perhaps.

>   What is the 
> reason for including ib_types.h and cm.h?  ib_types looks like an internal 
> opensm include file.

There are many userspace applications which use ib_types.h so it is not
just a opensm internal file. Anything using the SA client API also uses
this. There has been discussion of this on the list before.

It does appear there is some overlap here though.

-- Hal

> - Sean