[PATCH] libibmad: Fix build on win8
From: Hefty, Sean <hidden>
Date: 2011-08-18 16:46:49
Subsystem:
the rest · Maintainer:
Linus Torvalds
The win8 compiler appears to be even more brain dead than its predecessor. It complains of an uninitialized variable mad in rpc.c around line 213. Fix this by removing the redirect variable and using while (1) / break instead. Problem was reported by Leonid Keller [off-list ref] Signed-off-by: Sean Hefty <redacted> Reviewed-by: Hal Rosenstock <hal-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org> --- src/rpc.c | 9 ++++----- 1 files changed, 4 insertions(+), 5 deletions(-)
diff --git a/src/rpc.c b/src/rpc.c
index d20c321..19a5a70 100644
--- a/src/rpc.c
+++ b/src/rpc.c@@ -213,13 +213,12 @@ void *mad_rpc(const struct ibmad_port *port, ib_rpc_t * rpc, { int status, len; uint8_t sndbuf[1024], rcvbuf[1024], *mad; - int redirect = 1; ib_rpc_v1_t *rpcv1 = (ib_rpc_v1_t *)rpc; int error = 0; if ((rpc->mgtclass & IB_MAD_RPC_VERSION_MASK) == IB_MAD_RPC_VERSION1) rpcv1->error = 0; - while (redirect) { + do { len = 0; memset(sndbuf, 0, umad_size() + IB_MAD_SIZE);
@@ -247,10 +246,10 @@ void *mad_rpc(const struct ibmad_port *port, ib_rpc_t * rpc, /* update dport for next request and retry */ /* bail if redirection fails */ if (redirect_port(dport, mad)) - redirect = 0; + break; } else - redirect = 0; - } + break; + } while (1); if ((rpc->mgtclass & IB_MAD_RPC_VERSION_MASK) == IB_MAD_RPC_VERSION1) rpcv1->error = error; --
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html