***SPAM*** Re: [ofw] [patch] [IPoIB] Adding support to dell guid and user-define mask for guid generation

[Hal Rosenstock]

On Fri, Sep 19, 2008 at 11:59 AM, Fab Tillier
<ftillier at windows.microsoft.com> wrote:
>>> Index: ulp/opensm/user/include/iba/ib_types.h
>>> ===================================================================
>>> --- ulp/opensm/user/include/iba/ib_types.h (revision 3193)
>>> +++ ulp/opensm/user/include/iba/ib_types.h (working copy)
>>> @@ -7928,6 +7928,7 @@
>>>   IB_INVALID_GID, IB_INVALID_LID, IB_INVALID_GUID, +
>>>   IB_INVALID_GUID_MASK, IB_INVALID_CA_HANDLE, IB_INVALID_AV_HANDLE,
>>>   IB_INVALID_CQ_HANDLE,
>>
>> Are we keeping ib_types.h in sync with Linux ? I thought this file was
>> intended on being platform independent. Should this change be made
>> there as well ?
>
> There's 2 ib_types.h files, one for the windows drivers, one in OpenSM's own directory.  Only the second needs to be kept in sync with Linux.

I saw that and that's the one I was commenting on:
ulp/opensm/user/include/iba/ib_types.h

That's the one which should be kept in sync, right ?

>
> There's also ib_types_extended.h in OpenSM's directory, which is a copy of ib_types.h.  I don't quite understand what's going on here, though...

Me neither but I think there's been some discussion of this on the
list in the past; I just don't remember. It doesn't exist in Linux so
is either due to some change or some Windows thing.

-- Hal

> -Fab
>