Cache validator scans all switches by looping over all fabric nodes
table. This patch improves this slightly by using a fabric switches
table instead.
Signed-off-by: Sasha Khapyorsky <[EMAIL PROTECTED]>
---
opensm/opensm/osm_ucast_cache.c | 16 ++++++----------
1 files changed, 6 insertions(+), 10 deletions(-)
diff --git a/opensm/opensm/osm_ucast_cache.c b/opensm/opensm/osm_ucast_cache.c
index e5264cd..df8b7d2 100644
--- a/opensm/opensm/osm_ucast_cache.c
+++ b/opensm/opensm/osm_ucast_cache.c
@@ -525,7 +525,7 @@ static void ucast_cache_validate(osm_ucast_cache_t *
p_cache)
osm_physp_t * p_physp;
osm_physp_t * p_remote_physp;
osm_port_t * p_remote_port;
- cl_qmap_t * p_node_guid_tbl;
+ cl_qmap_t * p_sw_tbl;
OSM_LOG_ENTER(p_cache->p_ucast_mgr->p_log);
if (!p_cache->valid)
@@ -537,18 +537,14 @@ static void ucast_cache_validate(osm_ucast_cache_t *
p_cache)
* it's just some node/port reset or a cached topology
* change. Otherwise the cache is invalid.
*/
- p_node_guid_tbl = &p_cache->p_ucast_mgr->p_subn->node_guid_tbl;
- for (p_node = (osm_node_t *) cl_qmap_head(p_node_guid_tbl);
- p_node != (osm_node_t *) cl_qmap_end(p_node_guid_tbl);
- p_node = (osm_node_t *) cl_qmap_next(&p_node->map_item)) {
-
- if (osm_node_get_type(p_node) != IB_NODE_TYPE_SWITCH)
- continue;
+ p_sw_tbl = &p_cache->p_ucast_mgr->p_subn->sw_guid_tbl;
+ for (p_sw = (osm_switch_t *) cl_qmap_head(p_sw_tbl);
+ p_sw != (osm_switch_t *) cl_qmap_end(p_sw_tbl);
+ p_sw = (osm_switch_t *) cl_qmap_next(&p_sw->map_item)) {
+ p_node = p_sw->p_node;
lid_ho = cl_ntoh16(osm_node_get_base_lid(p_node,0));
p_cache_sw = __cache_get_sw(p_cache, lid_ho);
-
- p_sw = p_node->sw;
max_ports = osm_node_get_num_physp(p_node);
/* skip port 0 */
--
1.6.0.1.196.g01914
_______________________________________________
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
