Thread (1 message) 1 message, 1 author, 2012-09-21

Re: [PATCH] infiniband-diags/src/perfquery.c: Fix all_ports corner case

From: Ira Weiny <hidden>
Date: 2012-09-21 00:32:09

On Thu, 13 Sep 2012 23:16:14 -0700
Al Chu [off-list ref] wrote:
if AllPortSelect is not supported and --all_ports is specified, it should
only emulate AllPortSelect if no port is specified or the all ports
port number (255) is specified.

Signed-off-by: Albert Chu <redacted>
Thanks applied,
Ira
quoted hunk
---
 src/perfquery.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/src/perfquery.c b/src/perfquery.c
index 27ec4f7..605ece9 100644
--- a/src/perfquery.c
+++ b/src/perfquery.c
@@ -783,7 +783,7 @@ int main(int argc, char **argv)
 	if (!(cap_mask & IB_PM_ALL_PORT_SELECT)) {	/* bit 8 is AllPortSelect */
 		if (!all_ports && port == ALL_PORTS)
 			IBERROR("AllPortSelect not supported");
-		if (all_ports)
+		if (all_ports && port == ALL_PORTS)
 			all_ports_loop = 1;
 	}
 
-- 
1.7.1



--
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

-- 
Ira Weiny
Member of Technical Staff
Lawrence Livermore National Lab
925-423-8008
weiny2-i2BcT+NCU+M@public.gmane.org
--
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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help