RE: [PATCH 00/11] nfs refcount conversions

From: Reshetova, Elena
Date: Mon Oct 23 2017 - 03:30:47 EST



> On Fri, Oct 20, 2017 at 12:53:27PM +0300, Elena Reshetova wrote:
> > This series, for nfs components, replaces atomic_t reference
> > counters with the new refcount_t type and API (see include/linux/refcount.h).
> > By doing this we prevent intentional or accidental
> > underflows or overflows that can led to use-after-free vulnerabilities.
> >
> > The patches are fully independent and can be cherry-picked separately.
> > If there are no objections to the patches, please merge them via respective trees.
> >
> > Rebased on top of linux-next.
> >
> > Elena Reshetova (11):
> > fs, nfsd: convert nfs4_stid.sc_count from atomic_t to refcount_t
> > fs, nfsd: convert nfs4_cntl_odstate.co_odcount from atomic_t to
> > refcount_t
> > fs, nfsd: convert nfs4_file.fi_ref from atomic_t to refcount_t
>
> Thanks, applying for 4.15.

Thank you very much!

>
> I haven't followed recent discussion on the refcount api, but I assume
> the consensus is to do this, and these particular conversions certainly
> look trivial enough.

refcount_api will continue for evolve for some time I think, we recently got
x86 specific FAST implementation done, some other archs might follow.
There is an ongoing discussion about the memory ordering, which we have to
decide which way we want things to be, but these are all independent things
from the actual conversions. So for conversions that are basic and rather trivial,
nothing should prevent doing them now.

Best Regards,
Elena.


>
> --b.