Chuck, While hunting for some bugs, I found some warts that the first two patches clean up. I also added some sanity checks when freeing delegations or queueing up a CB_RECALL. Your call on whether you want to accept those. These should apply cleanly to your for-next branch, but the last patch may need to be adjusted when Dai fixes up his patch. Cheers, Jeff Layton (4): nfsd: only fill out return pointer on success in nfsd4_lookup_stateid nfsd: fix comments about spinlock handling with delegations nfsd: make nfsd4_run_cb a bool return function nfsd: extra checks when freeing delegation stateids fs/nfsd/nfs4callback.c | 14 ++++++++++++-- fs/nfsd/nfs4state.c | 26 +++++++++++++++----------- fs/nfsd/state.h | 2 +- 3 files changed, 28 insertions(+), 14 deletions(-) -- 2.37.3