diff mbox

[2/2] opensm: use OSM_LOG_INFO on duplicate torus port order parsing

Message ID 1306967581-1015-2-git-send-email-jaschut@sandia.gov (mailing list archive)
State Accepted
Headers show

Commit Message

Jim Schutt June 1, 2011, 10:33 p.m. UTC
Since it doesn't cause OpenSM to exit, it isn't an error.
We just want to give the admin information that will allow them
to clean up their configuration.

Signed-off-by: Jim Schutt <jaschut@sandia.gov>
---
 opensm/osm_torus.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

Comments

Alex Netes June 3, 2011, 10:49 a.m. UTC | #1
On 16:33 Wed 01 Jun     , Jim Schutt wrote:
> Since it doesn't cause OpenSM to exit, it isn't an error.
> We just want to give the admin information that will allow them
> to clean up their configuration.
> 
> Signed-off-by: Jim Schutt <jaschut@sandia.gov>
> ---

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_torus.c b/opensm/osm_torus.c
index 47654ce..d06cea9 100644
--- a/opensm/osm_torus.c
+++ b/opensm/osm_torus.c
@@ -870,8 +870,8 @@  bool parse_port_order(struct torus *t, const char *parse_sep)
 
 		for (j = 0; j < i; j++) {
 			if (t->port_order[j] == t->port_order[i]) {
-				OSM_LOG(&t->osm->log, OSM_LOG_ERROR,
-					"Error: ignoring duplicate port %u in "
+				OSM_LOG(&t->osm->log, OSM_LOG_INFO,
+					"Ignored duplicate port %u in"
 					" port_order parsing\n",
 					t->port_order[j]);
 				i--;	/* Ignore duplicate port number */