Re: opensm/osm_opensm.c: no report when SM is exiting

2010-08-19 Thread Yevgeny Kliteynik
On 19-Aug-10 3:59 AM, Ira Weiny wrote: On Wed, 18 Aug 2010 14:58:44 -0700 Yevgeny Kliteynikklit...@gmail.com wrote: On 18-Aug-10 7:06 PM, Ira Weiny wrote: On Wed, 18 Aug 2010 06:38:14 -0700 Yevgeny Kliteynikklit...@dev.mellanox.co.il wrote: Don't bother reporting events to plug-ins

Re: [PATCH v2] rdma/ib_pack.h: add new bth opcodes

2010-08-19 Thread Or Gerlitz
Bob Pearson wrote: I was curious to see if I could force a ConnectX device to slow down from a remote application. But since the MADs have been crippled for IBOE there is no way to configure it. QP1 MADs are working for ConnectX, e.g the IB CM is fully functional for IBoE, and I don't

Re: [PATCH v2] rdma/ib_pack.h: add new bth opcodes

2010-08-19 Thread Eli Cohen
On Thu, Aug 19, 2010 at 11:54:17AM +0300, Or Gerlitz wrote: Bob Pearson wrote: I was curious to see if I could force a ConnectX device to slow down from a remote application. But since the MADs have been crippled for IBOE there is no way to configure it. QP1 MADs are working for

Re: [PATCH 2/3 v5] IB/srp: Implement SRP_CRED_REQ and SRP_AER_REQ

2010-08-19 Thread Bart Van Assche
On Thu, Aug 19, 2010 at 2:48 AM, David Dillow d...@thedillows.org wrote: On Mon, 2010-08-16 at 20:55 +0200, Bart Van Assche wrote: Implements SRP_CRED_REQ and SRP_AER_REQ, which are information units defined in the SRP (draft) standard.  Adds declarations for the SRP_CRED_REQ, SRP_CRED_RSP,

Re: [PATCH 1/3 v5] IB/srp: Preparation for transmit ring response allocation

2010-08-19 Thread David Dillow
On Thu, 2010-08-19 at 08:04 +0200, Bart Van Assche wrote: On Thu, Aug 19, 2010 at 1:44 AM, David Dillow d...@thedillows.org wrote: On Mon, 2010-08-16 at 20:54 +0200, Bart Van Assche wrote: @@ -989,19 +989,21 @@ static void srp_send_completion(struct ib_cq *cq, void *target_ptr)

Re: [PATCH 2/3 v5] IB/srp: Implement SRP_CRED_REQ and SRP_AER_REQ

2010-08-19 Thread David Dillow
On Thu, 2010-08-19 at 12:10 +0200, Bart Van Assche wrote: On Thu, Aug 19, 2010 at 2:48 AM, David Dillow d...@thedillows.org wrote: On Mon, 2010-08-16 at 20:55 +0200, Bart Van Assche wrote: Implements SRP_CRED_REQ and SRP_AER_REQ, which are information units defined in the SRP (draft)

Re: [PATCH 2/3 v5] IB/srp: Implement SRP_CRED_REQ and SRP_AER_REQ

2010-08-19 Thread Bart Van Assche
On Thu, Aug 19, 2010 at 12:52 PM, David Dillow d...@thedillows.org wrote: On Thu, 2010-08-19 at 12:10 +0200, Bart Van Assche wrote: On Thu, Aug 19, 2010 at 2:48 AM, David Dillow d...@thedillows.org wrote: [ ... ] If I do not receive any further feedback, I will modify srp_handle_req()

Userspace MMU Notification

2010-08-19 Thread Eric B Munson
During the .35 merge window I attempted to merge the ummunotify driver and was told by Linus that he didn't like the approach. At that point several members of the RDMA community were hashing out a way to do the same thing that would get Linus' blessing. I haven't seen anything about it since

Re: [PATCH 1/3 v5] IB/srp: Preparation for transmit ring response allocation

2010-08-19 Thread Bart Van Assche
On Thu, Aug 19, 2010 at 12:44 PM, David Dillow d...@thedillows.org wrote: On Thu, 2010-08-19 at 08:04 +0200, Bart Van Assche wrote: On Thu, Aug 19, 2010 at 1:44 AM, David Dillow d...@thedillows.org wrote: On Mon, 2010-08-16 at 20:54 +0200, Bart Van Assche wrote: @@ -989,19 +989,21 @@

Re: Userspace MMU Notification

2010-08-19 Thread Roland Dreier
During the .35 merge window I attempted to merge the ummunotify driver and was told by Linus that he didn't like the approach. At that point several members of the RDMA community were hashing out a way to do the same thing that would get Linus' blessing. I haven't seen anything about it

Re: [PATCH 1/3 v5] IB/srp: Preparation for transmit ring response allocation

2010-08-19 Thread David Dillow
On Thu, 2010-08-19 at 18:53 +0200, Bart Van Assche wrote: On Thu, Aug 19, 2010 at 12:44 PM, David Dillow d...@thedillows.org wrote: Put the assignment on the same line as the declaration and I'll deal with the name. Putting the rsv assignment on the same line as its declaration would make

Re: [PATCH 1/3 v5] IB/srp: Preparation for transmit ring response allocation

2010-08-19 Thread Bart Van Assche
On Thu, Aug 19, 2010 at 7:25 PM, David Dillow d...@thedillows.org wrote: On Thu, 2010-08-19 at 18:53 +0200, Bart Van Assche wrote: On Thu, Aug 19, 2010 at 12:44 PM, David Dillow d...@thedillows.org wrote: Put the assignment on the same line as the declaration and I'll deal with the name.

Re: [PATCH 1/3 v5] IB/srp: Preparation for transmit ring response allocation

2010-08-19 Thread Roland Dreier
Putting the rsv assignment on the same line as its declaration would make that line exceed 80 columns and hence would trigger a checkpatch complaint, so it's better to keep it as it is now. I didn't check it with checkpatch, but vi tells me it is 78 characters. In any case I think the

ipoib neighbour free race?

2010-08-19 Thread Zach Brown
Hi gang, We're chasing a bug that we can hit when we pull IB cables with CONFIG_DEBUG_PAGE_ALLOC enabled. It appears as though the to_ipoib_neigh() in ipoib_neigh_free() under ipoib_mcast_free() is referencing a freed neighbour struct. The invariant here, as far as I can tell, is that