Re: ib-diags and ib_get_err_str

2011-08-07 Thread Hal Rosenstock
On 8/5/2011 8:17 PM, Ira Weiny wrote: > On Fri, 5 Aug 2011 16:57:02 -0700 > "Hefty, Sean" wrote: > >>> This is picked up by libosmcomp.so >> >> Is this a new library built as part of opensm? > > opensm-libs is the specific package. > > I don't think it is that new. It's not new. It's built a

[PATCH] opensm: Add extended link speeds support

2011-08-07 Thread Hal Rosenstock
Per approved and published MgtWG RefIDs relating to this Signed-off-by: Hal Rosenstock --- diff --git a/include/iba/ib_types.h b/include/iba/ib_types.h index 3dc8a7c..918301a 100644 --- a/include/iba/ib_types.h +++ b/include/iba/ib_types.h @@ -2720,6 +2720,11 @@ typedef struct _ib_path_rec {

[PATCH 6/7] Fix a compiler warning about an unused function

2011-08-07 Thread Bart Van Assche
Signed-off-by: Bart Van Assche --- src/memory.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/src/memory.c b/src/memory.c index 9f13774..7d97e55 100644 --- a/src/memory.c +++ b/src/memory.c @@ -269,6 +269,7 @@ static void __mm_rotate_left(struct ibv_mem_node *node)

[PATCH 3/7] autogen.sh: Introduce autoreconf

2011-08-07 Thread Bart Van Assche
There is no need to invoke the different autotools individually - autoreconf already does that. Signed-off-by: Bart Van Assche --- autogen.sh |7 +-- 1 files changed, 1 insertions(+), 6 deletions(-) diff --git a/autogen.sh b/autogen.sh index fd47839..f6ff02c 100755 --- a/autogen.sh +++

[PATCH 2/7] Makefile.am: Fix an automake warning

2011-08-07 Thread Bart Van Assche
Avoid that automake reports the following warning message: Makefile.am:1: `INCLUDES' is the old name for `AM_CPPFLAGS' (or `*_CPPFLAGS') A quote from the automake manual: INCLUDES This does the same job as AM_CPPFLAGS (or any per-target _CPPFLAGS variable if it is used). It is an older nam

[PATCH 4/7] .gitignore: Ignore files generated by the autotools

2011-08-07 Thread Bart Van Assche
Signed-off-by: Bart Van Assche --- .gitignore |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/.gitignore b/.gitignore index 78effef..a0a13a6 100644 --- a/.gitignore +++ b/.gitignore @@ -15,3 +15,8 @@ Makefile config.status stamp-h1 libtool +config/libtool.m4 +confi

[PATCH 5/7] 32-bit compiler warning fix

2011-08-07 Thread Bart Van Assche
Avoid that the code in src/memory.c triggers a compiler warning when compiled on a 32-bit system. Signed-off-by: Bart Van Assche --- src/memory.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/src/memory.c b/src/memory.c index faa43f3..9f13774 100644 --- a/src/memory.

[PATCH 1/7] Add "foreign" option to AM_INIT_AUTOMAKE

2011-08-07 Thread Bart Van Assche
Tell automake that the libibverbs package does not follow the GNU standards. This change makes it possible to use 'autoreconf' for the libibverbs package. Signed-off-by: Bart Van Assche --- configure.in |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/configure.in b/conf

[PATCH 0/7] Various libibverbs patches

2011-08-07 Thread Bart Van Assche
Hello Roland, This patch series addresses a few issues I noticed after having built libibverbs from the source tree available at kernel.org. Feedback is welcome. Bart. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majord...@vger.kernel.org Mo

[PATCH 7/7] Fix a compiler warning related to NVALGRIND

2011-08-07 Thread Bart Van Assche
Avoid that compiling with NVALGRIND defined and the latest Valgrind header files triggers compiler warnings. Recently the Valgrind client request implementation has been modified in order to not trigger compiler warnings when building with gcc 4.6. A side effect of that change is that Valgrind clie

Re: [PATCH v3] opensm: make loopback console compile on by default.

2011-08-07 Thread Alex Netes
Hi Ira, On 15:42 Wed 03 Aug , Ira Weiny wrote: > > The console is very useful for debugging and should be available in > opensm.conf > as an option. > > Generic socket is still an option which is off for security reasons. > > Changes in V3: >Print warning when libwrap is not found for

Re: [patch v2 34/37] add rxe_net_sysfs.c

2011-08-07 Thread Bart Van Assche
On Sun, Jul 24, 2011 at 9:43 PM, wrote: > +/* Copy argument and remove trailing CR. Return the new length. */ As far as I can see the function below removes a trailing newline instead of a trailing carriage return ? > +static int sanitize_arg(const char *val, char *intf, int intf_len) > +{ > +

Re: [patch v2 32/37] add rxe_net.h

2011-08-07 Thread Bart Van Assche
On Sun, Jul 24, 2011 at 9:43 PM, wrote: > +/* > + * this should be defined in .../include/linux/if_ether.h > + */ > +#define ETH_P_RXE(0x8915) > + > +/* > + * this should be defined in .../include/linux/netfilter.h > + * to a specific value > + */ > +#define NFPROTO_RXE

Re: [patch v2 35/37] add rxe_sample.c

2011-08-07 Thread Bart Van Assche
On Sun, Jul 24, 2011 at 9:43 PM, wrote: > module that implements a soft IB device using ib_rxe in loopback. > +MODULE_DESCRIPTION("RDMA transport over Converged Enhanced Ethernet"); That's the same description as for the ib_rxe_net kernel module, so one of the two probably should be updated. >