[ofw] RE: [PATCH] umad - add missing item from latest management tree

Smith, Stan stan.smith at intel.com
Fri Oct 16 08:22:00 PDT 2009


Hefty, Sean wrote:
>>> Can you explain again why this is needed?
>>>
>>> Windows has no restriction on the number of MAD agents, so this
>>> should be set to something huge, like 4 billion.  If the user is
>>> assuming that this is a small value, then I think we need to fix
>>> that code.
>>
>> Opensm 3.2.2
>
> 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?

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 Sasha & company decide it's the right thing to change, then we can adjust the Windows code at that time.



More information about the ofw mailing list