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 >

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 > --- 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 http://vger.kernel.org/majordomo-info.

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

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 > --- 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 http://vger.kernel.org/majordom

dapltest question

2011-07-08 Thread Steve Wise
Hey Arlin, I have a dumb question: Is dapltest designed to support multiple dapltest client processes running on multiple physical systems to connect to a single dapltest server process/system? I'm running tests where we have 4 client nodes running dapltest transaction clients connecting to t

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 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 much more appropriate.

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

2011-07-08 Thread Ira Weiny
On Fri, 8 Jul 2011 03:49:27 -0700 Alex Netes wrote: > 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 > > --- > > Sounds like a good idea. > If we change the defaults, why should we keep this opti

RE: dapltest question

2011-07-08 Thread Smith, Stan
>-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 dumb question: Is dapltest designed

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 > > --- > drivers/char/nvram.c |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/drivers/c

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 du

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 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 specific concerns ?

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. Mattock --- drivers/char/nvram.c |2 +- 1 files changed, 1 insertions(+), 1 deletions

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 wrote: > > Signed-off-by: Hal Rosenstock Thanks applied, Ira > --- > include/infiniband/mad.h | 48 > +- > src/dump.c |5 > src/fields.c | 45 +

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 wrote: > > Signed-off-by: Hal Rosenstock > --- > include/infiniband/mad.h | 48 > +- > src/dump.c |5 > src/fields.c | 45 +

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 wrote: > Add support for new LinkSpeedExt* components in SM PortInfo attribute > > Signed-off-by: Hal Rosenstock Thanks, _this_ one is applied... Ira > --- > include/infiniband/mad.h |9 + > src/dump.c | 75 +++

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 wrote: > On Thu, 7 Jul 2011 09:50:21 -0700 > Hal Rosenstock wrote: > > > > > Signed-off-by: Hal Rosenstock > > Thanks applied, > Ira > > > --- > > include/infi

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 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 100644 > --- a/src/

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 wrote: > >> >> Signed-off-by: Hal Rosenstock >> --- >> include/infiniband/mad.h | 48 >> +- >> src/dump.c |5 >>

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 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 wrote: > > > >> > >> Signed-off-by: Hal Rosenstock > >> --- > >> include/infiniband/mad.h | 48 > >> ++

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 > --- > 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 > --- > include/ibdiag_common.h |3 ++ > src/ibdiag_common.c | 55 > +++ > src/iblin

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

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 el

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 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 multiple nodes can r

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