diff mbox

opensm/osm_dump.c: Remove incorrect assert

Message ID 5101491C.9060607@dev.mellanox.co.il (mailing list archive)
State Accepted
Delegated to: Alex Netes
Headers show

Commit Message

Hal Rosenstock Jan. 24, 2013, 2:45 p.m. UTC
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@mellanox.com>
---
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Alex Netes Feb. 3, 2013, 5:49 p.m. UTC | #1
Hi Hal,

On 09:45 Thu 24 Jan     , Hal Rosenstock wrote:
> 
> 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@mellanox.com>
> ---

Applied, thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/opensm/osm_dump.c b/opensm/osm_dump.c
index b29d957..69cc360 100644
--- a/opensm/osm_dump.c
+++ b/opensm/osm_dump.c
@@ -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",