[ofw] RE: [PATCH] umad - add missing item from latest management tree
Sean Hefty
sean.hefty at intel.com
Fri Oct 16 13:41:41 PDT 2009
>> That doesn't really explain anything. How is opensm using this
>> value, and what assumptions does it make about its size? We should
>> make sure that we have the right fix here.
>
>Why are you being so difficult to work with here?
You're asking to change code with zero explanation as to why the code needs to
change.
>OpenSM 3.2.? added the value to umad.h file and referenced it in
>osm_vendor_umad.c.
>If you have issues with the code, then take it up with those who wrote the code
>and stop impeding forward progress of getting a sorely needed newer revision of
>OpenSM to the Windows community.
If you want to change the windows version of libibumad, then submit a patch
against that code with a reason why that change is needed. The #define that you
gave in the patch is wrong. The limit is much, much higher on windows.
- Sean
More information about the ofw
mailing list