Skip to contentSkip to conversation

osm_mcast_tbl.c: Wrong assert placement in osm_mcast_tbl_get_block

PATCH New Hal Rosenstock 1 participant 1 message May 29, 2015 InfiniBand

lore
Share & tools
lore

Hal Rosenstock

author
#0 lore raw

PATCH 1/1 osm_mcast_tbl.c: Wrong assert placement in osm_mcast_tbl_get_block

From: Ilya Nelkenbaum <ilyan-VPRAkNaXOzVWk0Htik3J/[email protected]>
Date: Mon, 28 Jan 2013 13:53:38 +0200

Signed-off-by Ilya Nelkenbaum <ilyan-VPRAkNaXOzVWk0Htik3J/[email protected]>
Signed-off-by Hal Rosenstock <hal-VPRAkNaXOzVWk0Htik3J/[email protected]>

---

opensm/osm_mcast_tbl.c +2 -1
index c4a64e2..617fd50 100644
@@ -242,7 +242,6 @@ boolean_t osm_mcast_tbl_get_block(IN osm_mcast_tbl_t * p_tbl,
 
 	CL_ASSERT(p_tbl);
 	CL_ASSERT(p_block);
-	CL_ASSERT(block_num * IB_MCAST_BLOCK_SIZE <= p_tbl->mft_depth);
 
 	if (block_num > p_tbl->max_block_in_use)
 		return FALSE;

@@ -255,6 +254,8 @@ boolean_t osm_mcast_tbl_get_block(IN osm_mcast_tbl_t * p_tbl,
 		return TRUE;
 	}
 
+	CL_ASSERT(block_num * IB_MCAST_BLOCK_SIZE <= p_tbl->mft_depth);
+
 	mlid_start_ho = (uint16_t) (block_num * IB_MCAST_BLOCK_SIZE);
 
 	for (i = 0; i < IB_MCAST_BLOCK_SIZE; i++)

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