[PATCH v2][TRIVIAL] osm_sm_state_mgr.c Trivial log changes

2013-06-20 Thread Line Holen
Signed-off-by: Line Holen line.ho...@oracle.com --- diff --git a/opensm/osm_sm_state_mgr.c b/opensm/osm_sm_state_mgr.c index 11defdd..5d4b651 100644 --- a/opensm/osm_sm_state_mgr.c +++ b/opensm/osm_sm_state_mgr.c @@ -157,7 +157,7 @@ static void sm_state_mgr_start_polling(osm_sm_t * sm)

Re: [PATCH] mm: Revert pinned_vm braindamage

2013-06-20 Thread Ingo Molnar
* Thomas Gleixner t...@linutronix.de wrote: On Thu, 13 Jun 2013, Andrew Morton wrote: Let's try to get this wrapped up? On Thu, 6 Jun 2013 14:43:51 +0200 Peter Zijlstra pet...@infradead.org wrote: Patch bc3e53f682 (mm: distinguish between mlocked and pinned pages) broke

Re: [PATCH] mm: Revert pinned_vm braindamage

2013-06-20 Thread Ingo Molnar
* Christoph Lameter c...@gentwo.org wrote: On Mon, 17 Jun 2013, Peter Zijlstra wrote: They did no such thing; being one of those who wrote such code. I expressly used RLIMIT_MEMLOCK for its the one limit userspace has to limit pages that are exempt from paging. Dont remember

Re: [PATCH V2 0/4] Add IPv6 support for iWARP

2013-06-20 Thread Steve Wise
On 6/19/2013 11:08 PM, David Miller wrote: From: Steve Wise sw...@opengridcomputing.com Date: Wed, 19 Jun 2013 21:19:13 -0500 On 6/19/2013 8:01 PM, David Miller wrote: From: Vipul Pandya vi...@chelsio.com Date: Wed, 12 Jun 2013 17:11:38 +0530 We have included all the maintainers of

Re: [PATCH V2 0/4] Add IPv6 support for iWARP

2013-06-20 Thread Vipul Pandya
On 20-06-2013 09:38, David Miller wrote: From: Steve Wise sw...@opengridcomputing.com Date: Wed, 19 Jun 2013 21:19:13 -0500 On 6/19/2013 8:01 PM, David Miller wrote: From: Vipul Pandya vi...@chelsio.com Date: Wed, 12 Jun 2013 17:11:38 +0530 We have included all the maintainers of

Re: [PATCH] libibverbs: Allow arbitrary int values for MTU

2013-06-20 Thread Jeff Squyres (jsquyres)
On Jun 18, 2013, at 2:49 PM, Jason Gunthorpe jguntho...@obsidianresearch.com wrote: +int num_to_ibv_mtu(int num); Probably should be ibv_num_to_mtu() to keep with the naming pattern.. New patch coming momentarily, but I wanted to comment on this one: I used the name num_to_ibv_mtu

[PATCH V2] libibverbs: Allow arbitrary int values for MTU.

2013-06-20 Thread Jeff Squyres
Keep IBV_MTU_* enums values as they are, but pass MTU values around as int's. This is an ABI-compatible change; legacy applications will use the enum values, but newer applications can use an int for values that do not currently exist in the enum set (e.g., 1500, 9000). (if people like the idea

Re: [PATCH for-next 0/8] Add Mellanox mlx5 driver for Connect-IB devices

2013-06-20 Thread Or Gerlitz
On 16/06/2013 15:02, Eli Cohen wrote: From: Eli Cohen e...@mellanox.com The patches that follow constitute the driver for Mellanox's 5th generation of HCAs named Connect-IB. The driver is comprised of two kernel modules: mlx5_ib and mlx5_core. This partitioning resembles what we have for mlx4

Re: [PATCH] mm: Revert pinned_vm braindamage

2013-06-20 Thread Christoph Lameter
On Thu, 20 Jun 2013, Ingo Molnar wrote: Peter clearly pointed it out that in the perf case it's user-space that initiates the pinned memory mapping which is resource-controlled via RLIMIT_MEMLOCK - and this was implemented that way before your commit broke the code. There is no way that user

Re: NFS over RDMA benchmark

2013-06-20 Thread Or Gerlitz
On 19/06/2013 18:47, Wendy Cheng wrote: what kind of HW I would need to run it ? The mlx4 driver supports memory windows as of kernel 3.9 Or. -- 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 v2][TRIVIAL] osm_sm_state_mgr.c Trivial log changes

2013-06-20 Thread Hal Rosenstock
On 6/20/2013 3:44 AM, Line Holen wrote: Signed-off-by: Line Holen line.ho...@oracle.com Thanks. Applied. -- Hal -- 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 V2] libibverbs: Allow arbitrary int values for MTU.

2013-06-20 Thread Doug Ledford
On 06/20/2013 10:21 AM, Jeff Squyres wrote: Keep IBV_MTU_* enums values as they are, but pass MTU values around as int's. This is an ABI-compatible change; legacy applications will use the enum values, I'm not really concerned with what the legacy apps use so much as what they are presented

Re: [PATCH V2] libibverbs: Allow arbitrary int values for MTU.

2013-06-20 Thread Doug Ledford
On 06/20/2013 12:34 PM, Doug Ledford wrote: On 06/20/2013 10:21 AM, Jeff Squyres wrote: Keep IBV_MTU_* enums values as they are, but pass MTU values around as int's. This is an ABI-compatible change; legacy applications will use the enum values, I'm not really concerned with what the

Re: [PATCH V2] libibverbs: Allow arbitrary int values for MTU.

2013-06-20 Thread Jason Gunthorpe
On Thu, Jun 20, 2013 at 12:34:04PM -0400, Doug Ledford wrote: On 06/20/2013 10:21 AM, Jeff Squyres wrote: Keep IBV_MTU_* enums values as they are, but pass MTU values around as int's. This is an ABI-compatible change; legacy applications will use the enum values, I'm not really

RE: [PATCH] libibverbs: Allow arbitrary int values for MTU

2013-06-20 Thread Hefty, Sean
I used the name num_to_ibv_mtu because it is in the spirit of the other enum- to-int/int-to-enum function pair naming conventions: int ibv_rate_to_mult(enum ibv_rate rate); enum ibv_rate mult_to_ibv_rate(int mult); int ibv_rate_to_mbps(enum ibv_rate rate); enum ibv_rate

Re: [PATCH] libibverbs: Allow arbitrary int values for MTU

2013-06-20 Thread Jeff Squyres (jsquyres)
On Jun 20, 2013, at 1:09 PM, Hefty, Sean sean.he...@intel.com wrote: int ibv_rate_to_mult(enum ibv_rate rate); enum ibv_rate mult_to_ibv_rate(int mult); int ibv_rate_to_mbps(enum ibv_rate rate); enum ibv_rate mbps_to_ibv_rate(int mbps); libibverbs uses the ibv_ prefix for pretty much

Re: [PATCH V2] libibverbs: Allow arbitrary int values for MTU.

2013-06-20 Thread Doug Ledford
On 06/20/2013 12:53 PM, Jason Gunthorpe wrote: On Thu, Jun 20, 2013 at 12:34:04PM -0400, Doug Ledford wrote: On 06/20/2013 10:21 AM, Jeff Squyres wrote: Keep IBV_MTU_* enums values as they are, but pass MTU values around as int's. This is an ABI-compatible change; legacy applications will use

Re: [PATCH] libibverbs: Allow arbitrary int values for MTU

2013-06-20 Thread Doug Ledford
On 06/18/2013 02:49 PM, Jason Gunthorpe wrote: This is simpler: { static char str[16]; snprintf(str, sizeof(str), %d, ibv_mtu_to_num(max_mtu)); return str; } That is not, however, multi-thread safe nor advisable unless you clearly indicate in the man page to the

Re: [PATCH] libibverbs: Allow arbitrary int values for MTU

2013-06-20 Thread Jeff Squyres (jsquyres)
On Jun 20, 2013, at 4:40 PM, Doug Ledford dledf...@redhat.com wrote: { static char str[16]; snprintf(str, sizeof(str), %d, ibv_mtu_to_num(max_mtu)); return str; } That is not, however, multi-thread safe nor advisable unless you clearly indicate in the man page to the

Re: [PATCH V2] libibverbs: Allow arbitrary int values for MTU.

2013-06-20 Thread Jason Gunthorpe
On Thu, Jun 20, 2013 at 04:31:14PM -0400, Doug Ledford wrote: happened for iwarp, rocee, etc. If it happened once, then I would agree with you above. That it *keeps* happening is the issue. To me, that's a clear indication that instead of fixing the shortcomings of the current API

linux-next: just chaeking correctness of the infiniband tree

2013-06-20 Thread Stephen Rothwell
Hi all, I noticed that the infiniband tree is now based on the net-next tree. I assume that is deliberate? I do have to question how much testing that tree has had since it is now based on a tree that Dave only released in the last 24 hours ... -- Cheers, Stephen Rothwell

Re: [PATCH V2] libibverbs: Allow arbitrary int values for MTU.

2013-06-20 Thread Doug Ledford
On 06/20/2013 05:14 PM, Jason Gunthorpe wrote: On Thu, Jun 20, 2013 at 04:31:14PM -0400, Doug Ledford wrote: happened for iwarp, rocee, etc. If it happened once, then I would agree with you above. That it *keeps* happening is the issue. To me, that's a clear indication that instead of

RE: [PATCH V2] libibverbs: Allow arbitrary int values for MTU.

2013-06-20 Thread Hefty, Sean
I would argue that this is because the libraries are so disjoint (that librdmacm needs the deep internal knowledge it needs of libibverbs indicates that maybe these two shouldn't be separate from each other for example, or that maybe libibverbs should provide a unified connection API to the

[PATCH] IB/core: Fix error return code in add_port()

2013-06-20 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -ENOMEM in the alloc_group_attrs() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/infiniband/core/sysfs.c | 8 ++-- 1 file changed, 6

RE: [PATCH] IB/core: Fix error return code in add_port()

2013-06-20 Thread Hefty, Sean
Subject: [PATCH] IB/core: Fix error return code in add_port() From: Wei Yongjun yongjun_...@trendmicro.com.cn Fix to return -ENOMEM in the alloc_group_attrs() error handling Patch itself looks fine, but please change alloc_group_attrs() - add_port() in the description. - Sean -- To

[PATCH v2] IB/core: Fix error return code in add_port()

2013-06-20 Thread Wei Yongjun
Fix to return -ENOMEM in the add_port() error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- v1 - v2: change patch description --- drivers/infiniband/core/sysfs.c | 8 ++-- 1 file changed, 6 insertions(+), 2

Re: linux-next: just chaeking correctness of the infiniband tree

2013-06-20 Thread Roland Dreier
On Thu, Jun 20, 2013 at 5:09 PM, Stephen Rothwell s...@canb.auug.org.au wrote: I noticed that the infiniband tree is now based on the net-next tree. I assume that is deliberate? I do have to question how much testing that tree has had since it is now based on a tree that Dave only released in