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, rpear...@systemfabricworks.com 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

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, rpear...@systemfabricworks.com 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 + */

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, rpear...@systemfabricworks.com 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

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 loopback

[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

[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

[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 bvanass...@acm.org --- configure.in |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git

[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 bvanass...@acm.org --- src/memory.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/src/memory.c b/src/memory.c index faa43f3..9f13774

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

2011-08-07 Thread Bart Van Assche
Signed-off-by: Bart Van Assche bvanass...@acm.org --- .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

[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

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

2011-08-07 Thread Bart Van Assche
Signed-off-by: Bart Van Assche bvanass...@acm.org --- 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

[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 bvanass...@acm.org --- autogen.sh |7 +-- 1 files changed, 1 insertions(+), 6 deletions(-) diff --git a/autogen.sh b/autogen.sh index fd47839..f6ff02c 100755

[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 h...@mellanox.com --- 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