Hi Yevgeny,

On 23:42 Sun 11 Jan     , Yevgeny Kliteynik wrote:
> Hi Sasha,
>
> Sasha Khapyorsky wrote:
>> When we are going to setup LFTs we need to ignore its previous images if
>> OpenSM enters master after standby, so need to check for subnet
>> need_update flag too.
>
> Nice catch. I think there will be a similar problem with
> cached routing too - need to invalidate the cache when SM
> enters master state.

Yes, obviously. Maybe something like this will do:


diff --git a/opensm/opensm/osm_state_mgr.c b/opensm/opensm/osm_state_mgr.c
index 625e026..fc7ceb9 100644
--- a/opensm/opensm/osm_state_mgr.c
+++ b/opensm/opensm/osm_state_mgr.c
@@ -1089,7 +1089,7 @@ static void do_sweep(osm_sm_t * sm)
         */
        if (sm->p_subn->opt.use_ucast_cache &&
            (sm->p_subn->subnet_initialization_error ||
-            sm->p_subn->force_reroute))
+            sm->p_subn->force_reroute || sm->p_subn->coming_out_of_standby))
                osm_ucast_cache_invalidate(&sm->ucast_mgr);
 
        /*


(and basically I think tah all those "flags flow" mess requires cleanup
already :)).

Sasha
_______________________________________________
general mailing list
[email protected]
http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general

To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general

Reply via email to