Re: opensm logs can go anywhere in size?

2011-07-08 Thread Alex Netes
Hi Or, On 12:07 Thu 07 Jul , Or Gerlitz wrote: Hi Alex, I just noted that opensm logs can go anywhere in size when one of my nodes had no space left on device after the opensm log level was raised... I see that the rpm provided with RHEL6 is setting an /etc/logrotate.d/opensm entry

Re: [PATCH] libibumad: Remove OpenSM reference from COPYING file

2011-07-08 Thread Alex Netes
Hi Ira, On 14:39 Wed 06 Jul , Ira Weiny wrote: Signed-off-by: Ira Weiny wei...@llnl.gov --- Applied, thanks. -- 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

Re: [PATCH] opensm: enable perfmgr build by default

2011-07-08 Thread Alex Netes
Hi Ira, On 15:11 Wed 06 Jul , Ira Weiny wrote: This should at least be compiled in by default. Signed-off-by: Ira Weiny wei...@llnl.gov --- Sounds like a good idea. If we change the defaults, why should we keep this option for configure script? Is it really needed? What about

Re: [PATCH] IB/CMA: fail qkey setting for RDMA_PS_IPOIB and IB_LINK_LAYER_ETHERNET

2011-07-08 Thread Moni Shoua
Yes, the kernel crashed after watchdog detected a lockup. This happened while running udaddy -p 0x2 with RoCE. The crash isn't 100% reproducible but I have a pair of nodes where that used to crash with probability of 50%. The fix you suggest seems to detect the mismatch between port space and link

Re: [PATCH] opensm/osmtest/osmt_multicast.c: Use proper defines for rate

2011-07-08 Thread Alex Netes
Hi Hal, On 13:24 Tue 05 Jul , Hal Rosenstock wrote: Signed-off-by: Hal Rosenstock h...@mellanox.com --- Applied, thanks. -- 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

Re: [PATCH 04/14] SIWv2: Module initialization: siw_main.c

2011-07-08 Thread Bernard Metzler
Steve, Thanks for the reply. Sounds good, I completely agree with your suggestions. Many thanks, Bernard. Steve Wise sw...@opengridcomputing.com wrote on 07/07/2011 04:04:05 PM: On 06/30/2011 11:23 AM, Bernard Metzler wrote: Good point. I moved the three attributes to debugfs which looks

Re: [PATCH] opensm: enable perfmgr build by default

2011-07-08 Thread Hal Rosenstock
Ira, On 7/8/2011 12:06 PM, Ira Weiny wrote: However, when we first put the PerfMgr in OpenSM there was some concern for developers who may be using OpenSM in an embedded environment. Do you recall the specific concerns ? I forget and would rather not dig them out unless really necessary...

Re: [RFC 6/9]drivers:char:nvram.c Change the copyright info.

2011-07-08 Thread Jiri Kosina
On Fri, 26 Nov 2010, Justin P. Mattock wrote: Change the copyright info from Sun Microsystems to Oracle since Oracle now owns them. Signed-off-by: Justin P. Mattock justinmatt...@gmail.com --- drivers/char/nvram.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff

Re: dapltest question

2011-07-08 Thread Steve Wise
On 07/08/2011 11:14 AM, Smith, Stan wrote: -Original Message- From: linux-rdma-ow...@vger.kernel.org [mailto:linux-rdma-ow...@vger.kernel.org] On Behalf Of Steve Wise Sent: Friday, July 08, 2011 8:23 AM To: Davis, Arlin R Cc: linux-rdma Subject: dapltest question Hey Arlin, I have a

Re: [PATCH] opensm: enable perfmgr build by default

2011-07-08 Thread Ira Weiny
On Fri, 8 Jul 2011 09:23:50 -0700 Hal Rosenstock h...@dev.mellanox.co.il wrote: Ira, On 7/8/2011 12:06 PM, Ira Weiny wrote: However, when we first put the PerfMgr in OpenSM there was some concern for developers who may be using OpenSM in an embedded environment. Do you recall the

Re: [RFC 6/9]drivers:char:nvram.c Change the copyright info.

2011-07-08 Thread Justin P. Mattock
On 07/08/2011 09:33 AM, Jiri Kosina wrote: On Fri, 26 Nov 2010, Justin P. Mattock wrote: Change the copyright info from Sun Microsystems to Oracle since Oracle now owns them. Signed-off-by: Justin P. Mattockjustinmatt...@gmail.com --- drivers/char/nvram.c |2 +- 1 files changed, 1

Re: [PATCH 3/3] libibmad: Add PortExtendedSpeedsCounters support

2011-07-08 Thread Ira Weiny
On Thu, 7 Jul 2011 09:50:21 -0700 Hal Rosenstock h...@dev.mellanox.co.il wrote: Signed-off-by: Hal Rosenstock h...@mellanox.com Thanks applied, Ira --- include/infiniband/mad.h | 48 +- src/dump.c |5 src/fields.c

Re: [PATCH 3/3] libibmad: Add PortExtendedSpeedsCounters support

2011-07-08 Thread Ira Weiny
Hey Hal, On Thu, 7 Jul 2011 09:50:21 -0700 Hal Rosenstock h...@dev.mellanox.co.il wrote: Signed-off-by: Hal Rosenstock h...@mellanox.com --- include/infiniband/mad.h | 48 +- src/dump.c |5 src/fields.c |

Re: [PATCH 1/3] libibmad: Add initial extended speed support

2011-07-08 Thread Ira Weiny
On Thu, 7 Jul 2011 09:50:03 -0700 Hal Rosenstock h...@dev.mellanox.co.il wrote: Add support for new LinkSpeedExt* components in SM PortInfo attribute Signed-off-by: Hal Rosenstock h...@mellanox.com Thanks, _this_ one is applied... Ira --- include/infiniband/mad.h |9 +

Re: [PATCH 3/3] libibmad: Add PortExtendedSpeedsCounters support

2011-07-08 Thread Ira Weiny
Ooops... I mean to say 1/3 was applied... See other email for comments. Sorry, Ira On Fri, 8 Jul 2011 13:34:32 -0700 Ira Weiny wei...@llnl.gov wrote: On Thu, 7 Jul 2011 09:50:21 -0700 Hal Rosenstock h...@dev.mellanox.co.il wrote: Signed-off-by: Hal Rosenstock h...@mellanox.com

Re: [PATCH 2/3] infiniband-diags: Initial changes for extended speed support in libibmad

2011-07-08 Thread Ira Weiny
On Thu, 7 Jul 2011 09:50:12 -0700 Hal Rosenstock h...@dev.mellanox.co.il wrote: Need larger buffer for mad_dump_portinfo due to additional extended speed fields Signed-off-by: Hal Rosenstock Thanks, applied, Ira --- diff --git a/src/saquery.c b/src/saquery.c index 7933fec..53ab1b6

Re: [PATCH 3/3] libibmad: Add PortExtendedSpeedsCounters support

2011-07-08 Thread Hal Rosenstock
Hi Ira, On 7/8/2011 4:35 PM, Ira Weiny wrote: Hey Hal, On Thu, 7 Jul 2011 09:50:21 -0700 Hal Rosenstock h...@dev.mellanox.co.il wrote: Signed-off-by: Hal Rosenstock h...@mellanox.com --- include/infiniband/mad.h | 48 +- src/dump.c

Re: [PATCH 3/3] libibmad: Add PortExtendedSpeedsCounters support

2011-07-08 Thread Ira Weiny
On Fri, 8 Jul 2011 14:23:48 -0700 Hal Rosenstock h...@dev.mellanox.co.il wrote: Hi Ira, On 7/8/2011 4:35 PM, Ira Weiny wrote: Hey Hal, On Thu, 7 Jul 2011 09:50:21 -0700 Hal Rosenstock h...@dev.mellanox.co.il wrote: Signed-off-by: Hal Rosenstock h...@mellanox.com ---

Re: [PATCH 2/4] infiniband-diags: move core saquery functionality to ibdiag_common

2011-07-08 Thread Hal Rosenstock
On 7/5/2011 3:08 PM, Ira Weiny wrote: While this is sub-optimal it is the easiest way to get saquery support to the other diags. Just some nits below... Signed-off-by: Ira Weiny wei...@llnl.gov --- include/ibdiag_common.h | 45 +++ src/ibdiag_common.c | 172

Re: [PATCH 3/4] infiniband-diags: properly resolve node guids

2011-07-08 Thread Hal Rosenstock
On 7/5/2011 3:08 PM, Ira Weiny wrote: Can you explain what issue this solves or is it just to move to the SA way to do this ? Signed-off-by: Ira Weiny wei...@llnl.gov --- include/ibdiag_common.h |3 ++ src/ibdiag_common.c | 55 +++

Re: [PATCH 3/4] infiniband-diags: properly resolve node guids

2011-07-08 Thread Jason Gunthorpe
On Fri, Jul 08, 2011 at 05:42:38PM -0400, Hal Rosenstock wrote: Should the request just be a GET rather than GET_TABLE and avoid this check ? I don't think multiple nodes can register with same Node GUID, can they ? Also, I think it makes eliminates this check and the missing 0 check.

Re: [PATCH 3/4] infiniband-diags: properly resolve node guids

2011-07-08 Thread Hal Rosenstock
On 7/8/2011 5:50 PM, Jason Gunthorpe wrote: On Fri, Jul 08, 2011 at 05:42:38PM -0400, Hal Rosenstock wrote: Should the request just be a GET rather than GET_TABLE and avoid this check ? I don't think multiple nodes can register with same Node GUID, can they ? Also, I think it makes

Re: [PATCH 3/4] infiniband-diags: properly resolve node guids

2011-07-08 Thread Ira Weiny
On Fri, 8 Jul 2011 14:59:01 -0700 Hal Rosenstock h...@dev.mellanox.co.il wrote: On 7/8/2011 5:50 PM, Jason Gunthorpe wrote: On Fri, Jul 08, 2011 at 05:42:38PM -0400, Hal Rosenstock wrote: Should the request just be a GET rather than GET_TABLE and avoid this check ? I don't think

Re: [PATCH 3/4] infiniband-diags: properly resolve node guids

2011-07-08 Thread Jason Gunthorpe
On Fri, Jul 08, 2011 at 03:29:53PM -0700, Ira Weiny wrote: On Fri, 8 Jul 2011 14:59:01 -0700 Hal Rosenstock h...@dev.mellanox.co.il wrote: On 7/8/2011 5:50 PM, Jason Gunthorpe wrote: On Fri, Jul 08, 2011 at 05:42:38PM -0400, Hal Rosenstock wrote: Should the request just be a GET