[openib-general] [PATCH] osm: management class constants are unit8 not uint16

Eitan Zahavi eitan at mellanox.co.il
Mon Jun 5 05:40:53 PDT 2006


Hi Hal

Cleaning up compilation warnings I found that the osm_vendor_mlx_svc.h
was using NTOH16 on the class constants.

Eitan

Signed-off-by:  Eitan Zahavi <eitan at mellanox.co.il>

Index: include/vendor/osm_vendor_mlx_svc.h
===================================================================
--- include/vendor/osm_vendor_mlx_svc.h	(revision 7542)
+++ include/vendor/osm_vendor_mlx_svc.h	(working copy)
@@ -119,8 +119,8 @@ osmv_mad_is_rmpp(IN const ib_mad_t *p_ma
 
     rmpp_flags = ((ib_rmpp_mad_t*)p_mad)->rmpp_flags;
     /* HACK - JUST SA and DevMgt for now - need to add BIS and DevAdm */
-    if ( (p_mad->mgmt_class != CL_NTOH16(IB_MCLASS_SUBN_ADM)) &&
-         (p_mad->mgmt_class != CL_NTOH16(IB_MCLASS_DEV_MGMT)) )
+    if ( (p_mad->mgmt_class != IB_MCLASS_SUBN_ADM) &&
+         (p_mad->mgmt_class != IB_MCLASS_DEV_MGMT) )
       return(0);
     return (0 != (rmpp_flags & IB_RMPP_FLAG_ACTIVE));
 }




More information about the general mailing list