d/freebsd-nq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix for compilation issue. Don't use the "abs()" function for unsigned

Browse Source 
computations.

MFC after:	3 days
PR:		196597
Sponsored by:	Mellanox Technologies
This commit is contained in:
Hans Petter Selasky 2015-01-08 10:13:56 +00:00
parent 359f2b4604
commit dda7d972d5
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
contrib/ofed/management/opensm/opensm/osm_ucast_ftree.c
View File

@ -2917,6 +2917,10 @@ __osm_ftree_fabric_construct_hca_ports(IN ftree_fabric_t * p_ftree,
/***************************************************
***************************************************/
static boolean_t __osm_invalid_link_rank_diff(const uint32_t val)
{
return (val != 1U && val != -1U);
}
static int __osm_ftree_fabric_construct_sw_ports(IN ftree_fabric_t * p_ftree,
IN ftree_sw_t * p_sw)
@ -2993,7 +2997,7 @@ static int __osm_ftree_fabric_construct_sw_ports(IN ftree_fabric_t * p_ftree,
p_remote_hca_or_sw = (void *)p_remote_sw;
if (abs(p_sw->rank - p_remote_sw->rank) != 1) {
if (__osm_invalid_link_rank_diff(p_sw->rank - p_remote_sw->rank)) {
OSM_LOG(&p_ftree->p_osm->log, OSM_LOG_ERROR,
"ERR AB16: "
"Illegal link between switches with ranks %u and %u:\n"