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

[Ira Weiny]

On Thu, 13 Sep 2012 23:16:14 -0700
Al Chu ch...@llnl.gov 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 ch...@llnl.gov

Thanks applied,
Ira

 ---
  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 majord...@vger.kernel.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
wei...@llnl.gov
--
To unsubscribe from this list: send the line unsubscribe linux-rdma in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

[Al Chu]

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 ch...@llnl.gov
---
 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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html