[ofa-general] ***SPAM*** [PATCHv2] libibmad/rpc.c: Handle redirection status

[Hal Rosenstock]

Also, in mad_rpc, status should be based on management class

Signed-off-by: Hal Rosenstock <hal.rosenstock at gmail.com>
---
Changes since v1:
Always get 16 bits of MAD status and mask when DR SMP
Remove some extraneous braces

diff --git a/libibmad/src/rpc.c b/libibmad/src/rpc.c
index 8c68cf9..965f32c 100644
--- a/libibmad/src/rpc.c
+++ b/libibmad/src/rpc.c
@@ -1,5 +1,6 @@
 /*
  * Copyright (c) 2004-2006 Voltaire Inc.  All rights reserved.
+ * Copyright (c) 2009 HNR Consulting.  All rights reserved.
  *
  * This software is available to you under a choice of one of two
  * licenses.  You may choose to be licensed under the terms of the GNU
@@ -170,7 +171,7 @@ void *mad_rpc(const struct ibmad_port *port, ib_rpc_t * rpc, ib_portid_t * dport
 	      void *payload, void *rcvdata)
 {
 	int status, len;
-	uint8_t sndbuf[1024], rcvbuf[1024], *mad;
+	uint8_t sndbuf[1024], rcvbuf[1024], *mad, mgmtclass;
 
 	len = 0;
 	memset(sndbuf, 0, umad_size() + IB_MAD_SIZE);
@@ -187,7 +188,18 @@ void *mad_rpc(const struct ibmad_port *port, ib_rpc_t * rpc, ib_portid_t * dport
 
 	mad = umad_get_mad(rcvbuf);
 
-	if ((status = mad_get_field(mad, 0, IB_DRSMP_STATUS_F)) != 0) {
+	status = mad_get_field(mad, 0, IB_MAD_STATUS_F);
+	mgmtclass = mad_get_field(mad, 0, IB_MAD_MGMTCLASS_F);
+	if (mgmtclass == IB_SMI_DIRECT_CLASS)
+		status &= 0x7fff;
+	else if (mgmtclass != IB_SMI_CLASS) {
+		if (status & 2) {
+			ERRS("MAD redirection not supported; dport (%s)",
+			     portid2str(dport));
+			return 0;
+		}
+	}
+	if (status) {
 		ERRS("MAD completed with error status 0x%x; dport (%s)",
 		     status, portid2str(dport));
 		return 0;
@@ -227,8 +239,13 @@ void *mad_rpc_rmpp(const struct ibmad_port *port, ib_rpc_t * rpc, ib_portid_t *
 	mad = umad_get_mad(rcvbuf);
 
 	if ((status = mad_get_field(mad, 0, IB_MAD_STATUS_F)) != 0) {
-		ERRS("MAD completed with error status 0x%x; dport (%s)",
-		     status, portid2str(dport));
+		if (status & 2) {
+			ERRS("MAD redirection not supported; dport (%s)",
+			     portid2str(dport));
+		} else {
+			ERRS("MAD completed with error status 0x%x; dport (%s)",
+			     status, portid2str(dport));
+		}
 		return 0;
 	}