Re: [PATCH v3 0/6] ceph: check the cephx mds auth access in client side

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Feb 6, 2024 at 2:50 PM Venky Shankar <vshankar@xxxxxxxxxx> wrote:
>
> On Tue, Dec 5, 2023 at 6:47 AM <xiubli@xxxxxxxxxx> wrote:
> >
> > From: Xiubo Li <xiubli@xxxxxxxxxx>
> >
> > The code are refered to the userspace libcephfs:
> > https://github.com/ceph/ceph/pull/48027.
> >
> >
> > V3:
> > - Fix https://tracker.ceph.com/issues/63141.
> >
> > V2:
> > - Fix memleak for built 'path'.
> >
> >
> > Xiubo Li (6):
> >   ceph: save the cap_auths in client when session being opened
> >   ceph: add ceph_mds_check_access() helper support
> >   ceph: check the cephx mds auth access for setattr
> >   ceph: check the cephx mds auth access for open
> >   ceph: check the cephx mds auth access for async dirop
> >   ceph: add CEPHFS_FEATURE_MDS_AUTH_CAPS_CHECK feature bit
> >
> >  fs/ceph/dir.c        |  28 +++++
> >  fs/ceph/file.c       |  61 +++++++++-
> >  fs/ceph/inode.c      |  46 ++++++--
> >  fs/ceph/mds_client.c | 265 ++++++++++++++++++++++++++++++++++++++++++-
> >  fs/ceph/mds_client.h |  28 ++++-
> >  5 files changed, 415 insertions(+), 13 deletions(-)
> >
> > --
> > 2.41.0
> >
>
> Tested-by: Venky Shankar <vshankar@xxxxxxxxxx>

cc Rishabh.

Revoking this in the meantime since I suspect this changeset to be causing

        https://tracker.ceph.com/issues/64172

since it happens with the testing kernel, is auth cap related and I
guess the changes were merged in the testing branch around the time we
started seeing this failure. I haven't seen the changes closely, but I
suspect the part where the last auth cap permission is followed might
be involved - not 100% sure though. Xiubo, please have a look.

>
> --
> Cheers,
> Venky



-- 
Cheers,
Venky






[Index of Archives]     [CEPH Users]     [Ceph Large]     [Ceph Dev]     [Information on CEPH]     [Linux BTRFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux