On Oct 17, 2011, at 5:28 PM, Hal Rosenstock wrote:
>
> A couple of minor comments:
>
> 1. Why not move the OSM_EVENT_ID_LIGHT_SWEEP_DONE event to where "LIGHT
> SWEEP COMPLETE" is indicated in osm_state_mgr.c ? That's actually more
> accurate as all the transactions have completed.
>
> 2. The n
Hi Karl,
On 10/17/2011 5:43 PM, Karl Schulz wrote:
> Thanks for explaining the enum convention - quite logical indeed. Also,
> thanks for some
> of the off-list discussion where we tried to see if any of the current event
> callbacks
> could be used to achieve the desired support. As it stan
Thanks for explaining the enum convention - quite logical indeed. Also, thanks
for some of the off-list discussion where we tried to see if any of the current
event callbacks could be used to achieve the desired support. As it stands, it
still sounds like including light sweep events is the si
Hi Karl,
The idea in that if the received event is >= OSM_EVENT_ID_MAX,
then the plug-in knows that something went wrong (wrong version,
something changed in API, etc).
Also, you might want to know about more stuff if you're pulling
topology/routing info from SM, such as when SM became stand
Thanks Ira,
My apologies for not following the conventional standard. Should I generate a
new patch?
-k
On Oct 12, 2011, at 5:13 PM, Ira Weiny wrote:
> I think this is a reasonable idea.
>
> Comment on the patch.
>
> - OSM_EVENT_ID_MAX
> + OSM_EVENT_ID_MAX,
> + OSM_EVENT_ID_LIGH
I think this is a reasonable idea.
Comment on the patch.
- OSM_EVENT_ID_MAX
+ OSM_EVENT_ID_MAX,
+ OSM_EVENT_ID_LIGHT_SWEEP_START,
+ OSM_EVENT_ID_LIGHT_SWEEP_DONE,
Traditionally the "MAX" value in the enum is kept at the end.
Ira
On Wed, 12 Oct 2011 14:31:07 -0700
Ka
Hello,
Would it be possible to consider exposing light-sweep events to the current
OpenSM plugin interface? Something along the lines of the attached patch is
what we are using locally against opensm-3.3.9.
The motivation is to support a plugin module we have written to expose topology
inform