lkml.org 
[lkml]   [2017]   [Apr]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH 0/2] fs, ceph filesystem refcount conversions
Date

> On Fri, Mar 3, 2017 at 10:15 AM, Elena Reshetova
> <elena.reshetova@intel.com> wrote:
> > Now when new refcount_t type and API are finally merged
> > (see include/linux/refcount.h), the following
> > patches convert various refcounters in the ceph filesystem from atomic_t
> > to refcount_t. By doing this we prevent intentional or accidental
> > underflows or overflows that can led to use-after-free vulnerabilities.
> >
> > The below patches are fully independent and can be cherry-picked separately.
> > Since we convert all kernel subsystems in the same fashion, resulting
> > in about 300 patches, we have to group them for sending at least in some
> > fashion to be manageable. Please excuse the long cc list.
> >
> > Not runtime tested, since I am not sure how to do it for ceph.
> > However changes are pretty trivial in this case.
> >
> > Elena Reshetova (2):
> > fs, ceph: convert ceph_mds_session.s_ref from atomic_t to refcount_t
> > fs, ceph: convert ceph_cap_snap.nref from atomic_t to refcount_t
> >
> > fs/ceph/caps.c | 4 ++--
> > fs/ceph/mds_client.c | 18 +++++++++---------
> > fs/ceph/mds_client.h | 5 +++--
> > fs/ceph/snap.c | 2 +-
> > fs/ceph/super.h | 5 +++--
> > 5 files changed, 18 insertions(+), 16 deletions(-)
>
> I'll pull these into testing.

Just a check: how did testing went on this?
If everything is good, can you merge the changes?

Best Regards,
Elena.

>
> Thanks,
>
> Ilya
\
 
 \ /
  Last update: 2017-04-20 09:42    [W:0.557 / U:0.028 seconds]
©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site