[ofa-general] Issues with osm_state_mgr.c:__osm_state_mgr_get_remote_port_info()

Vincent Ficet jean-vincent.ficet at bull.net
Wed Jun 25 07:09:39 PDT 2008


Hi Sasha,

>>   mad_context.pi_context.port_guid = cl_hton64(p_physp->port_guid);
>>     
>
> cl_hnon64() is not needed.
>   
I generated the patch [1] according to your comments. Please let me know 
if there is any problem.

Cheers,

Vincent

patch [1]:

diff --git a/opensm/opensm/osm_state_mgr.c b/opensm/opensm/osm_state_mgr.c
index f15a4af..780d8db 100644
--- a/opensm/opensm/osm_state_mgr.c
+++ b/opensm/opensm/osm_state_mgr.c
@@ -202,8 +202,7 @@ __osm_state_mgr_get_remote_port_info(IN osm_sm_t * sm,

mad_context.pi_context.node_guid =
osm_node_get_node_guid(osm_physp_get_node_ptr(p_physp));
- mad_context.pi_context.port_guid =
- cl_hton64(osm_physp_get_port_num(p_physp));
+ mad_context.pi_context.port_guid = p_physp->port_guid;
mad_context.pi_context.set_method = FALSE;
mad_context.pi_context.light_sweep = TRUE;
mad_context.pi_context.update_master_sm_base_lid = FALSE;




More information about the general mailing list