From: "Dr. David Alan Gilbert" <linux@treblig.org>

Hi,
  This series is a set of deadcoding in fs/ceph and net/ceph.
It's strictly function deletion so should have no change
in behaviour.

(get_maintainer was suggesting the netdev team as well
as ceph? Is that correct???)

Build & boot tested on x86-64.

Dave

Dr. David Alan Gilbert (5):
  libceph: Remove unused ceph_pagelist functions
  libceph: Remove unused pagevec functions
  libceph: Remove unused ceph_osdc_watch_check
  libceph: Remove unused ceph_crypto_key_encode
  ceph: Remove fs/ceph deadcode

 fs/ceph/caps.c                  | 14 ---------
 fs/ceph/mds_client.c            |  8 -----
 fs/ceph/mds_client.h            |  2 --
 fs/ceph/super.h                 |  1 -
 include/linux/ceph/libceph.h    |  6 ----
 include/linux/ceph/osd_client.h |  2 --
 include/linux/ceph/pagelist.h   | 12 --------
 net/ceph/crypto.c               | 12 --------
 net/ceph/crypto.h               |  1 -
 net/ceph/osd_client.c           | 34 ---------------------
 net/ceph/pagelist.c             | 38 ------------------------
 net/ceph/pagevec.c              | 52 ---------------------------------
 12 files changed, 182 deletions(-)

-- 
2.46.2

On Sun, Oct 6, 2024 at 3:19 AM <linux@treblig.org> wrote:
>
> From: "Dr. David Alan Gilbert" <linux@treblig.org>
>
> Hi,
>   This series is a set of deadcoding in fs/ceph and net/ceph.
> It's strictly function deletion so should have no change
> in behaviour.
>
> (get_maintainer was suggesting the netdev team as well
> as ceph? Is that correct???)

Hi David,

No, it's not correct.  It's probably caused by

F:    net/

entry in "NETWORKING [GENERAL]" section.  I don't recall
get_maintainer.pl doing that in the past, but I could be wrong.
Anyway, I'll send a patch to exclude net/ceph there.

Thanks,

                Ilya

* Ilya Dryomov (idryomov@gmail.com) wrote:
> On Sun, Oct 6, 2024 at 3:19 AM <linux@treblig.org> wrote:
> >
> > From: "Dr. David Alan Gilbert" <linux@treblig.org>
> >
> > Hi,
> >   This series is a set of deadcoding in fs/ceph and net/ceph.
> > It's strictly function deletion so should have no change
> > in behaviour.
> >
> > (get_maintainer was suggesting the netdev team as well
> > as ceph? Is that correct???)
> 
> Hi David,
> 
> No, it's not correct.  It's probably caused by
> 
> F:    net/
> 
> entry in "NETWORKING [GENERAL]" section.  I don't recall
> get_maintainer.pl doing that in the past, but I could be wrong.

THanks, I didn't think it was right.

> Anyway, I'll send a patch to exclude net/ceph there.

Thanks, and thanks for applying the cleanup.

Dave

> Thanks,
> 
>                 Ilya
> 
-- 
 -----Open up your eyes, open up your mind, open up your code -------   
/ Dr. David Alan Gilbert    |       Running GNU/Linux       | Happy  \ 
\        dave @ treblig.org |                               | In Hex /
 \ _________________________|_____ http://www.treblig.org   |_______/

On Sun, Oct 6, 2024 at 3:19 AM <linux@treblig.org> wrote:
>
> From: "Dr. David Alan Gilbert" <linux@treblig.org>
>
> Hi,
>   This series is a set of deadcoding in fs/ceph and net/ceph.
> It's strictly function deletion so should have no change
> in behaviour.
>
> (get_maintainer was suggesting the netdev team as well
> as ceph? Is that correct???)
>
> Build & boot tested on x86-64.
>
> Dave
>
> Dr. David Alan Gilbert (5):
>   libceph: Remove unused ceph_pagelist functions
>   libceph: Remove unused pagevec functions
>   libceph: Remove unused ceph_osdc_watch_check
>   libceph: Remove unused ceph_crypto_key_encode
>   ceph: Remove fs/ceph deadcode
>
>  fs/ceph/caps.c                  | 14 ---------
>  fs/ceph/mds_client.c            |  8 -----
>  fs/ceph/mds_client.h            |  2 --
>  fs/ceph/super.h                 |  1 -
>  include/linux/ceph/libceph.h    |  6 ----
>  include/linux/ceph/osd_client.h |  2 --
>  include/linux/ceph/pagelist.h   | 12 --------
>  net/ceph/crypto.c               | 12 --------
>  net/ceph/crypto.h               |  1 -
>  net/ceph/osd_client.c           | 34 ---------------------
>  net/ceph/pagelist.c             | 38 ------------------------
>  net/ceph/pagevec.c              | 52 ---------------------------------
>  12 files changed, 182 deletions(-)

Applied.

Thanks,

                Ilya