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 + >>> libibnetdisc/include/infiniband/ibnetdisc.h | 3 ++- >>> libibnetdisc/src/ibnetdisc.c | 1 + >>> man/ibaddr.8 | 2 ++ >>> src/ibaddr.c | 2 ++ >>> src/ibccconfig.c | 2 ++ >>> src/ibccquery.c | 2 ++ >>> src/ibdiag_common.c | 5 +++++ >>> src/iblinkinfo.c | 3 +++ >>> src/ibping.c | 2 ++ >>> src/ibportstate.c | 2 ++ >>> src/ibqueryerrors.c | 3 +++ >>> src/ibroute.c | 2 ++ >>> src/ibsendtrap.c | 2 ++ >>> src/ibsysstat.c | 2 ++ >>> src/ibtracert.c | 2 ++ >>> src/mcm_rereg_test.c | 3 +++ >>> src/perfquery.c | 2 ++ >>> src/sminfo.c | 2 ++ >>> src/smpquery.c | 2 ++ >>> src/vendstat.c | 2 ++ >>> 21 files changed, 46 insertions(+), 1 deletions(-) >> >> 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 have to be applied > until an audit can be performed of every tool. You may have noted I already > fixed saquery with previous patches.
Yes, I saw that. It doesn't seem like it would be much work to fix the ones that should be fixed for this (easy for me to say ;-) and IMO this should be done prior to this rather than adding MKey in places where it shouldn't be and then removing that later. -- Hal > > Ira -- 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