Hi Sasha,

Function osm_switch_get_port_by_lid() was using the switch's
LFT, so this LFT might not be updated to recent routing.

I think that this was also relevant before the LFT simplification.
One immediate outcome of this bug is opensm.fdbs file - when it
is dumped from the switch LFT (and not from lft_buf), it sometimes
doesn't match the lst file.

Signed-off-by: Yevgeny Kliteynik <[EMAIL PROTECTED]>
---
 opensm/include/opensm/osm_switch.h |    6 +++++-
 1 files changed, 5 insertions(+), 1 deletions(-)

diff --git a/opensm/include/opensm/osm_switch.h 
b/opensm/include/opensm/osm_switch.h
index caa0bc5..f06931c 100644
--- a/opensm/include/opensm/osm_switch.h
+++ b/opensm/include/opensm/osm_switch.h
@@ -411,7 +411,11 @@ osm_switch_get_port_by_lid(IN const osm_switch_t * const 
p_sw,
 {
        if (lid_ho == 0 || lid_ho > IB_LID_UCAST_END_HO)
                return OSM_NO_PATH;
-       return p_sw->lft[lid_ho];
+
+       if (p_sw->lft_buf)
+               return p_sw->lft_buf[lid_ho];
+       else
+               return p_sw->lft[lid_ho];
 }
 /*
 * PARAMETERS
-- 
1.5.1.4


_______________________________________________
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