opensm/osm_dump.c: Remove incorrect assert

PATCH In review Hal Rosenstock 2 participants 2 messages Jan 24, 2013 – Feb 3, 2013 1 message not in the archive InfiniBand

lore

Share & tools

lore

Hal Rosenstock

author

2013-01-24 14:45 UTC #0 lore raw

PATCH 1/1 opensm/osm_dump.c: Remove incorrect assert

In dump_sl2vl_tbl, no need to check remote physp for CA and router ports. It's not used in code flow there.

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

---

opensm/osm_dump.c +0 -1

index b29d957..69cc360 100644
@@ -524,7 +524,6 @@ static void dump_sl2vl_tbl(cl_map_item_t * item, FILE * file, void *cxt)
 		}
 	} else {
 		p_physp = p_port->p_physp;
-		CL_ASSERT(p_physp->p_remote_physp);
 		p_tbl = osm_physp_get_slvl_tbl(p_physp, 0);
 		for (i = 0, n = 0; i < 16; i++)
 			n += sprintf(buf + n, " %-2d",

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

A message that is not in the archive (sent off-list or not yet received): 5101491C.9060607-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb@public.gmane.org

Alex Netes

2013-02-03 17:49 UTC #2 lore raw

Hi Hal,

Quoted 6 lines · Hal Rosenstock

In dump_sl2vl_tbl, no need to check remote physp for CA and router ports. It's not used in code flow there.
Signed-off-by Hal Rosenstock <hal-VPRAkNaXOzVWk0Htik3J/[email protected]>
---

Applied, thanks.
--
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