Re: [PATCH] SUNRPC: Convert from atomic_t to refcount_t on rpc_clnt->cl_count

On Sat, 2021-07-17 at 18:18 +0800, Xiyu Yang wrote:

refcount_t type and corresponding API can protect refcounters from
accidental underflow and overflow and further use-after-free
situations.

Have you tested this patch? As far as I remember, the reason why we
never converted is that refcount_inc() gets upset and WARNs when you
bump a zero refcount, like we do very much on purpose in
rpc_free_auth(). Is that no longer the case?


-- 
Trond Myklebust
Linux NFS client maintainer, Hammerspace
trond.myklebust@hammerspace.com