linux-toradex.git/net/sunrpc/xprtrdma, branch v3.17.1

Merge tag 'nfs-for-3.17-1' of git://git.linux-nfs.org/projects/trondmy/linux-nfs

2014-08-14T00:13:19+00:00

Pull NFS client updates from Trond Myklebust:
 "Highlights include:

   - stable fix for a bug in nfs3_list_one_acl()
   - speed up NFS path walks by supporting LOOKUP_RCU
   - more read/write code cleanups
   - pNFS fixes for layout return on close
   - fixes for the RCU handling in the rpcsec_gss code
   - more NFS/RDMA fixes"

* tag 'nfs-for-3.17-1' of git://git.linux-nfs.org/projects/trondmy/linux-nfs: (79 commits)
  nfs: reject changes to resvport and sharecache during remount
  NFS: Avoid infinite loop when RELEASE_LOCKOWNER getting expired error
  SUNRPC: remove all refcounting of groupinfo from rpcauth_lookupcred
  NFS: fix two problems in lookup_revalidate in RCU-walk
  NFS: allow lockless access to access_cache
  NFS: teach nfs_lookup_verify_inode to handle LOOKUP_RCU
  NFS: teach nfs_neg_need_reval to understand LOOKUP_RCU
  NFS: support RCU_WALK in nfs_permission()
  sunrpc/auth: allow lockless (rcu) lookup of credential cache.
  NFS: prepare for RCU-walk support but pushing tests later in code.
  NFS: nfs4_lookup_revalidate: only evaluate parent if it will be used.
  NFS: add checks for returned value of try_module_get()
  nfs: clear_request_commit while holding i_lock
  pnfs: add pnfs_put_lseg_async
  pnfs: find swapped pages on pnfs commit lists too
  nfs: fix comment and add warn_on for PG_INODE_REF
  nfs: check wait_on_bit_lock err in page_group_lock
  sunrpc: remove "ec" argument from encrypt_v2 operation
  sunrpc: clean up sparse endianness warnings in gss_krb5_wrap.c
  sunrpc: clean up sparse endianness warnings in gss_krb5_seal.c
  ...

svcrdma: remove rdma_create_qp() failure recovery logic

2014-08-05T20:09:21+00:00

In svc_rdma_accept(), if rdma_create_qp() fails, there is useless
logic to try and call rdma_create_qp() again with reduced sge depths.
The assumption, I guess, was that perhaps the initial sge depths
chosen were too big.  However they initial depths are selected based
on the rdma device attribute max_sge returned from ib_query_device().
If rdma_create_qp() fails, it would not be because the max_send_sge and
max_recv_sge values passed in exceed the device's max.  So just remove
this code.

Signed-off-by: Steve Wise 
Signed-off-by: J. Bruce Fields

xprtrdma: Handle additional connection events

2014-07-31T20:22:59+00:00

Commit 38ca83a5 added RDMA_CM_EVENT_TIMEWAIT_EXIT. But that status
is relevant only for consumers that re-use their QPs on new
connections. xprtrdma creates a fresh QP on reconnection, so that
event should be explicitly ignored.

Squelch the alarming "unexpected CM event" message.

Signed-off-by: Chuck Lever 
Tested-by: Steve Wise 
Tested-by: Shirley Ma 
Tested-by: Devesh Sharma 
Signed-off-by: Anna Schumaker

xprtrdma: Remove RPCRDMA_PERSISTENT_REGISTRATION macro

2014-07-31T20:22:59+00:00

Clean up.

RPCRDMA_PERSISTENT_REGISTRATION was a compile-time switch between
RPCRDMA_REGISTER mode and RPCRDMA_ALLPHYSICAL mode.  Since
RPCRDMA_REGISTER has been removed, there's no need for the extra
conditional compilation.

Signed-off-by: Chuck Lever 
Tested-by: Steve Wise 
Tested-by: Shirley Ma 
Tested-by: Devesh Sharma 
Signed-off-by: Anna Schumaker

xprtrdma: Make rpcrdma_ep_disconnect() return void

2014-07-31T20:22:58+00:00

Clean up: The return code is used only for dprintk's that are
already redundant.

Signed-off-by: Chuck Lever 
Tested-by: Steve Wise 
Tested-by: Shirley Ma 
Tested-by: Devesh Sharma 
Signed-off-by: Anna Schumaker

xprtrdma: Schedule reply tasklet once per upcall

2014-07-31T20:22:58+00:00

Minor optimization: grab rpcrdma_tk_lock_g and disable hard IRQs
just once after clearing the receive completion queue.

Signed-off-by: Chuck Lever 
Tested-by: Steve Wise 
Tested-by: Shirley Ma 
Tested-by: Devesh Sharma 
Signed-off-by: Anna Schumaker

xprtrdma: Allocate each struct rpcrdma_mw separately

2014-07-31T20:22:57+00:00

Currently rpcrdma_buffer_create() allocates struct rpcrdma_mw's as
a single contiguous area of memory. It amounts to quite a bit of
memory, and there's no requirement for these to be carved from a
single piece of contiguous memory.

Signed-off-by: Chuck Lever 
Tested-by: Steve Wise 
Tested-by: Shirley Ma 
Tested-by: Devesh Sharma 
Signed-off-by: Anna Schumaker

xprtrdma: Rename frmr_wr

2014-07-31T20:22:57+00:00

Clean up: Name frmr_wr after the opcode of the Work Request,
consistent with the send and local invalidation paths.

Signed-off-by: Chuck Lever 
Tested-by: Steve Wise 
Tested-by: Shirley Ma 
Tested-by: Devesh Sharma 
Signed-off-by: Anna Schumaker

xprtrdma: Disable completions for LOCAL_INV Work Requests

2014-07-31T20:22:57+00:00

Instead of relying on a completion to change the state of an FRMR
to FRMR_IS_INVALID, set it in advance. If an error occurs, a completion
will fire anyway and mark the FRMR FRMR_IS_STALE.

Signed-off-by: Chuck Lever 
Tested-by: Steve Wise 
Tested-by: Shirley Ma 
Tested-by: Devesh Sharma 
Signed-off-by: Anna Schumaker

xprtrdma: Disable completions for FAST_REG_MR Work Requests

2014-07-31T20:22:56+00:00

Instead of relying on a completion to change the state of an FRMR
to FRMR_IS_VALID, set it in advance. If an error occurs, a completion
will fire anyway and mark the FRMR FRMR_IS_STALE.

Signed-off-by: Chuck Lever 
Tested-by: Steve Wise 
Tested-by: Shirley Ma 
Tested-by: Devesh Sharma 
Signed-off-by: Anna Schumaker