RE: [for-next 1/2] xprtrdma: take reference of rdma provider module

2014-07-21 Thread Devesh Sharma
Shirley, Once rmmod is issued, the connection corresponding to the active mount is destroyed and all the associated resources Are freed. As per the processing logic of DEVICE-REMOVAL event, nfs-rdma wakes-up all the waiters, This results into Re-establishment efforts, since the device is not

RE: [for-next 1/2] xprtrdma: take reference of rdma provider module

2014-07-21 Thread Devesh Sharma
In rpcrdma_ep_connect(): write_lock(ia-ri_qplock); old = ia-ri_id; ia-ri_id = id; write_unlock(ia-ri_qplock); rdma_destroy_qp(old); rdma_destroy_id(old); = Cm -id is destroyed here. If following code

Re: [for-next 1/2] xprtrdma: take reference of rdma provider module

2014-07-21 Thread Chuck Lever
Hi Devesh- Thanks for drilling into this further. On Jul 21, 2014, at 7:48 AM, Devesh Sharma devesh.sha...@emulex.com wrote: In rpcrdma_ep_connect(): write_lock(ia-ri_qplock); old = ia-ri_id; ia-ri_id = id; write_unlock(ia-ri_qplock);

RE: [for-next 1/2] xprtrdma: take reference of rdma provider module

2014-07-21 Thread Steve Wise
-Original Message- From: Chuck Lever [mailto:chuck.le...@oracle.com] Sent: Monday, July 21, 2014 9:54 AM To: Devesh Sharma Cc: Shirley Ma; Steve Wise; Hefty, Sean; Roland Dreier; linux-rdma@vger.kernel.org Subject: Re: [for-next 1/2] xprtrdma: take reference of rdma provider

[PATCH 0/5] Misc. fix for cxgb4 and iw_cxgb4

2014-07-21 Thread Hariprasad Shenai
Hi, This patch series adds support to enchance error reporting, log detailed warning for negative advice, support query_qp verb and advertise correct device max attributes for iwarp. The patches series is created against 'net-next' tree. And includes patches on cxgb4 and iw_cxgb4 driver. Since

[PATCH 1/5] cxgb4: Add the MC1 registers to read in the interrupt handler

2014-07-21 Thread Hariprasad Shenai
Signed-off-by: Hariprasad Shenai haripra...@chelsio.com --- drivers/net/ethernet/chelsio/cxgb4/t4_hw.c | 16 +--- drivers/net/ethernet/chelsio/cxgb4/t4_regs.h |3 +++ 2 files changed, 16 insertions(+), 3 deletions(-) diff --git a/drivers/net/ethernet/chelsio/cxgb4/t4_hw.c

[PATCH 3/5] iw_cxgb4: Support query_qp() verb

2014-07-21 Thread Hariprasad Shenai
Signed-off-by: Steve Wise sw...@opengridcomputing.com Signed-off-by: Hariprasad Shenai haripra...@chelsio.com --- drivers/infiniband/hw/cxgb4/qp.c |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/drivers/infiniband/hw/cxgb4/qp.c b/drivers/infiniband/hw/cxgb4/qp.c

[PATCH 5/5] iw_cxgb4: Don't limit TPTE count to 32KB

2014-07-21 Thread Hariprasad Shenai
Use the size advertised by FW Signed-off-by: Steve Wise sw...@opengridcomputing.com Signed-off-by: Hariprasad Shenai haripra...@chelsio.com --- drivers/infiniband/hw/cxgb4/iw_cxgb4.h |2 +- drivers/infiniband/hw/cxgb4/t4.h |1 - 2 files changed, 1 insertions(+), 2 deletions(-)

Re: [for-next 1/2] xprtrdma: take reference of rdma provider module

2014-07-21 Thread Chuck Lever
On Jul 21, 2014, at 11:03 AM, Steve Wise sw...@opengridcomputing.com wrote: -Original Message- From: Chuck Lever [mailto:chuck.le...@oracle.com] Sent: Monday, July 21, 2014 9:54 AM To: Devesh Sharma Cc: Shirley Ma; Steve Wise; Hefty, Sean; Roland Dreier;

RE: [for-next 1/2] xprtrdma: take reference of rdma provider module

2014-07-21 Thread Steve Wise
-Original Message- From: Chuck Lever [mailto:chuck.le...@oracle.com] Sent: Monday, July 21, 2014 10:21 AM To: Steve Wise Cc: Devesh Sharma; Shirley Ma; Hefty, Sean; Roland Dreier; linux-rdma@vger.kernel.org Subject: Re: [for-next 1/2] xprtrdma: take reference of rdma provider

RE: [for-next 1/2] xprtrdma: take reference of rdma provider module

2014-07-21 Thread Devesh Sharma
-Original Message- From: Steve Wise [mailto:sw...@opengridcomputing.com] Sent: Monday, July 21, 2014 8:53 PM To: 'Chuck Lever' Cc: Devesh Sharma; 'Shirley Ma'; 'Hefty, Sean'; 'Roland Dreier'; linux- r...@vger.kernel.org Subject: RE: [for-next 1/2] xprtrdma: take reference of rdma

Re: [for-next 1/2] xprtrdma: take reference of rdma provider module

2014-07-21 Thread Chuck Lever
On Jul 21, 2014, at 1:07 PM, Devesh Sharma devesh.sha...@emulex.com wrote: Until that support is in place, obviously I would prefer that the removal of the underlying driver be prevented while there are NFS mounts in place. I think that's what NFS users have come to expect. In other words,

Re: [PATCH 0/5] Misc. fix for cxgb4 and iw_cxgb4

2014-07-21 Thread David Miller
From: Hariprasad Shenai haripra...@chelsio.com Date: Mon, 21 Jul 2014 20:55:11 +0530 This patch series adds support to enchance error reporting, log detailed warning for negative advice, support query_qp verb and advertise correct device max attributes for iwarp. The patches series is