Sasha Khapyorsky wrote:
On 09:36 Thu 15 Oct     , Hal Rosenstock wrote:
Signed-off-by: Hal Rosenstock <[email protected]>
---
diff --git a/opensm/opensm/osm_ucast_cache.c b/opensm/opensm/osm_ucast_cache.c
index 31a5333..8db9df2 100644
--- a/opensm/opensm/osm_ucast_cache.c
+++ b/opensm/opensm/osm_ucast_cache.c
@@ -876,7 +876,7 @@ void osm_ucast_cache_add_link(osm_ucast_mgr_t * p_mgr,
        if (osm_node_get_type(p_node_1) != IB_NODE_TYPE_SWITCH &&
            osm_node_get_type(p_node_2) != IB_NODE_TYPE_SWITCH) {
                OSM_LOG(p_mgr->p_log, OSM_LOG_INFO,
-                       "Dropping CA-2-CA link - cache invalid\n");
+                       "Dropping CA/RTR-2-CA/RTR link - cache invalid\n");

IMHO it looks more confusing after the change.

BTW why should OSM_LOG_INFO level be used here?

If I remember it correctly, there's an INFO message
in the ucast mgr saying something like "switches
configured for unicast" or "<routing engine name>
tables configured", or something like that (I don't
have access to the code right now).
So I wanted to see one more line that says that
cache is not applicable.

-- Yevgeny

Sasha

                osm_ucast_cache_invalidate(p_mgr);
                goto Exit;
        }



--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to