diff mbox

opensm/osm_torus.c: torus routing should fail with VLCap 1 on switch external ports

Message ID 20130204093644.GB13151@calypso.mtl.com (mailing list archive)
State Accepted
Delegated to: Hal Rosenstock
Headers show

Commit Message

Alex Netes Feb. 4, 2013, 9:36 a.m. UTC 
Signed-off-by: Hal Rosenstock <hal@mellanox.com>
Signed-off-by: Alex Netes <alexne@mellanox.com>
---
 opensm/osm_torus.c | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

Comments

Jim Schutt Feb. 4, 2013, 9:56 p.m. UTC | #1 
On 02/04/2013 02:36 AM, Alex Netes wrote:
> Signed-off-by: Hal Rosenstock <hal@mellanox.com>
> Signed-off-by: Alex Netes <alexne@mellanox.com>

Acked-by: Jim Schutt <jaschut@sandia.gov>

--
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_torus.c b/opensm/osm_torus.c
index e63cb40..757a32a 100644
--- a/opensm/osm_torus.c
+++ b/opensm/osm_torus.c
@@ -7021,11 +7021,13 @@  bool verify_setup(struct torus *t, struct fabric *f)
 			"ERR 4E20: missing required torus size specification!\n");
 		goto out;
 	}
-	if (t->osm->subn.min_sw_data_vls < 2)
-		OSM_LOG(&t->osm->log, OSM_LOG_INFO,
-			"Warning: Too few data VLs to support torus routing "
+	if (t->osm->subn.min_sw_data_vls < 2) {
+		OSM_LOG(&t->osm->log, OSM_LOG_ERROR,
+			"ERR 4E48: Too few data VLs to support torus routing "
 			"without credit loops (have switchport %d need 2)\n",
 			(int)t->osm->subn.min_sw_data_vls);
+		goto out;
+	}
 	if (t->osm->subn.min_sw_data_vls < 4)
 		OSM_LOG(&t->osm->log, OSM_LOG_INFO,
 			"Warning: Too few data VLs to support torus routing "