Whamcloud - gitweb
LU-12511 fld: remove unused functions from fld_cache.c 37/37737/2
authorMr NeilBrown <neilb@suse.de>
Thu, 27 Feb 2020 02:35:51 +0000 (13:35 +1100)
committerOleg Drokin <green@whamcloud.com>
Tue, 24 Mar 2020 05:15:55 +0000 (05:15 +0000)
fld_cache_delete and fld_cache_entry_lookup{,nolock}
are never used, so they can be removed.
fld_cache_entry_delete is only used locally to fld_cache.c
so it can be made static.

Test-Parameters: trivial
Signed-off-by: Mr NeilBrown <neilb@suse.de>
Change-Id: Ied370420c348c95e14897187daaaec6708b5c96a
Reviewed-on: https://review.whamcloud.com/37737
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: James Simmons <jsimmons@infradead.org>
Reviewed-by: Shaun Tancheff <shaun.tancheff@hpe.com>
lustre/fld/fld_cache.c
lustre/fld/fld_internal.h

index 482fd10..c2e59f3 100644 (file)
@@ -101,8 +101,8 @@ void fld_cache_fini(struct fld_cache *cache)
 /**
  * delete given node from list.
  */
-void fld_cache_entry_delete(struct fld_cache *cache,
-                           struct fld_cache_entry *node)
+static void fld_cache_entry_delete(struct fld_cache *cache,
+                                  struct fld_cache_entry *node)
 {
        list_del(&node->fce_list);
        list_del(&node->fce_lru);
@@ -455,56 +455,6 @@ void fld_cache_delete_nolock(struct fld_cache *cache,
 }
 
 /**
- * Delete FLD entry in FLD cache.
- *
- */
-void fld_cache_delete(struct fld_cache *cache,
-                     const struct lu_seq_range *range)
-{
-       write_lock(&cache->fci_lock);
-       fld_cache_delete_nolock(cache, range);
-       write_unlock(&cache->fci_lock);
-}
-
-struct fld_cache_entry *
-fld_cache_entry_lookup_nolock(struct fld_cache *cache,
-                             const struct lu_seq_range *range)
-{
-       struct fld_cache_entry *flde;
-       struct fld_cache_entry *got = NULL;
-       struct list_head *head;
-
-       head = &cache->fci_entries_head;
-       list_for_each_entry(flde, head, fce_list) {
-               if (range->lsr_start == flde->fce_range.lsr_start ||
-                  (range->lsr_end == flde->fce_range.lsr_end &&
-                   range->lsr_flags == flde->fce_range.lsr_flags)) {
-                       got = flde;
-                       break;
-               }
-       }
-
-       RETURN(got);
-}
-
-/**
- * lookup \a seq sequence for range in fld cache.
- */
-struct fld_cache_entry *
-fld_cache_entry_lookup(struct fld_cache *cache,
-                      const struct lu_seq_range *range)
-{
-       struct fld_cache_entry *got = NULL;
-       ENTRY;
-
-       read_lock(&cache->fci_lock);
-       got = fld_cache_entry_lookup_nolock(cache, range);
-       read_unlock(&cache->fci_lock);
-
-       RETURN(got);
-}
-
-/**
  * lookup \a seq sequence for range in fld cache.
  */
 int fld_cache_lookup(struct fld_cache *cache,
index e326edd..1230b17 100644 (file)
@@ -197,24 +197,11 @@ struct fld_cache_entry
 
 int fld_cache_insert_nolock(struct fld_cache *cache,
                            struct fld_cache_entry *f_new);
-void fld_cache_delete(struct fld_cache *cache,
-                      const struct lu_seq_range *range);
 void fld_cache_delete_nolock(struct fld_cache *cache,
                             const struct lu_seq_range *range);
 int fld_cache_lookup(struct fld_cache *cache,
                     const u64 seq, struct lu_seq_range *range);
 
-struct fld_cache_entry *
-fld_cache_entry_lookup(struct fld_cache *cache,
-                      const struct lu_seq_range *range);
-
-void fld_cache_entry_delete(struct fld_cache *cache,
-                           struct fld_cache_entry *node);
-
-struct fld_cache_entry *
-fld_cache_entry_lookup_nolock(struct fld_cache *cache,
-                             const struct lu_seq_range *range);
-
 static inline const char *
 fld_target_name(const struct lu_fld_target *tar)
 {