Skip to contentSkip to conversation

opensm/osm_ucast_updn.c: cosmetics - rename function

PATCH Applied Yevgeny Kliteynik 2 participants 2 messages Oct 22, 2009 – Oct 23, 2009 1 message not in the archive InfiniBand

lore
Share & tools
lore

Yevgeny Kliteynik

author
#0 lore raw

PATCH 1/1 opensm/osm_ucast_updn.c: cosmetics - rename function

Hi Sasha,

There's a static function in the up/down routing, called "updn_clear_root_hops". This function is supposed to clear the hops to all targets but root switches.
So the more correct name for this function would be "updn_clear_non_root_hops".

Signed-off-by Yevgeny Kliteynik <[email protected]>
---
 opensm/opensm/osm_ucast_updn.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)
opensm/opensm/osm_ucast_updn.c +2 -2
index e81a299..e567a46 100644
@@ -286,7 +286,7 @@ static int updn_subn_rank(IN updn_t * p_updn)
 /**********************************************************************
  **********************************************************************/
 /* hack: preserve min hops entries to any other root switches */
-static void updn_clear_root_hops(updn_t * p_updn, osm_switch_t * p_sw)
+static void updn_clear_non_root_hops(updn_t * p_updn, osm_switch_t * p_sw)
 {
 	osm_port_t *p_port;
 	unsigned i;

@@ -325,7 +325,7 @@ static int updn_set_min_hop_table(IN updn_t * p_updn)
 		p_sw = (osm_switch_t *)item;
 		/* Clear Min Hop Table */
 		if (p_subn->opt.connect_roots)
-			updn_clear_root_hops(p_updn, p_sw);
+			updn_clear_non_root_hops(p_updn, p_sw);
 		else
 			osm_switch_clear_hops(p_sw);
 	}
Signature · 6 lines
1.5.1.4

--
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

A message that is not in the archive (sent off-list or not yet received): 4AE07B2F.8030004-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb@public.gmane.org

Applied
#2 lore raw
Quoted 9 lines · Yevgeny Kliteynik

Hi Sasha,

There's a static function in the up/down routing, called "updn_clear_root_hops". This function is supposed to clear the hops to all targets but root switches.
So the more correct name for this function would be "updn_clear_non_root_hops".

Signed-off-by Yevgeny Kliteynik <[email protected]>

Applied. Thanks.

Sasha
--
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