Re: [PATCH 2/3 fixed] infiniband-diags: Allow specification of an mkey on the command line

2012-03-13 Thread Hal Rosenstock
On 3/9/2012 3:09 PM, Ira Weiny wrote: On Fri, 09 Mar 2012 08:00:11 -0500 Hal Rosenstock h...@dev.mellanox.co.il wrote: On 3/6/2012 5:18 PM, Jim Foraker wrote: Signed-off-by: Jim Foraker forak...@llnl.gov --- include/ibdiag_common.h |1 +

Re: [PATCH 2/3 fixed] infiniband-diags: Allow specification of an mkey on the command line

2012-03-09 Thread Hal Rosenstock
On 3/6/2012 5:18 PM, Jim Foraker wrote: Signed-off-by: Jim Foraker forak...@llnl.gov --- include/ibdiag_common.h |1 + libibnetdisc/include/infiniband/ibnetdisc.h |3 ++- libibnetdisc/src/ibnetdisc.c|1 + man/ibaddr.8

Re: [PATCH 2/3 fixed] infiniband-diags: Allow specification of an mkey on the command line

2012-03-09 Thread Ira Weiny
On Fri, 09 Mar 2012 08:00:11 -0500 Hal Rosenstock h...@dev.mellanox.co.il wrote: On 3/6/2012 5:18 PM, Jim Foraker wrote: Signed-off-by: Jim Foraker forak...@llnl.gov --- include/ibdiag_common.h |1 + libibnetdisc/include/infiniband/ibnetdisc.h |3 ++-

Re: [PATCH 2/3 fixed] infiniband-diags: Allow specification of an mkey on the command line

2012-03-09 Thread Jason Gunthorpe
On Fri, Mar 09, 2012 at 12:09:31PM -0800, Ira Weiny wrote: Why do all these tools need mkey ? Isn't it only tools that require SMPs needing this support ? Unfortunately many of the diags issue SMP's when they should not. To keep them working in the short term with mkey's this patch will

[PATCH 2/3 fixed] infiniband-diags: Allow specification of an mkey on the command line

2012-03-06 Thread Jim Foraker
Signed-off-by: Jim Foraker forak...@llnl.gov --- include/ibdiag_common.h |1 + libibnetdisc/include/infiniband/ibnetdisc.h |3 ++- libibnetdisc/src/ibnetdisc.c|1 + man/ibaddr.8|2 ++ src/ibaddr.c