Re: [PATCH 8/9] dm vdo: Remove unused functions

[Matthew Sakai]

On 10/2/24 9:15 PM, li...@treblig.org wrote:

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

get_data_vio_pool_active_discards()
get_data_vio_pool_discard_limit()
get_data_vio_pool_maximum_discards()
set_data_vio_pool_discard_limit()

are all unused since commit
a9da0fb6d8c6 ("dm vdo: remove all sysfs interfaces")

Remove them.

Signed-off-by: Dr. David Alan Gilbert <li...@treblig.org>

Reviewed-by: Matthew Sakai <msa...@redhat.com>

---
  drivers/md/dm-vdo/data-vio.c | 29 -----------------------------
  drivers/md/dm-vdo/data-vio.h |  5 -----
  2 files changed, 34 deletions(-)

diff --git a/drivers/md/dm-vdo/data-vio.c b/drivers/md/dm-vdo/data-vio.c
index 0d502f6a86ad..2b489e37538e 100644
--- a/drivers/md/dm-vdo/data-vio.c
+++ b/drivers/md/dm-vdo/data-vio.c
@@ -1074,35 +1074,6 @@ void dump_data_vio_pool(struct data_vio_pool *pool, bool 
dump_vios)
        spin_unlock(&pool->lock);
  }

-data_vio_count_t get_data_vio_pool_active_discards(struct data_vio_pool *pool)

-{
-       return READ_ONCE(pool->discard_limiter.busy);
-}
-
-data_vio_count_t get_data_vio_pool_discard_limit(struct data_vio_pool *pool)
-{
-       return READ_ONCE(pool->discard_limiter.limit);
-}
-
-data_vio_count_t get_data_vio_pool_maximum_discards(struct data_vio_pool *pool)
-{
-       return READ_ONCE(pool->discard_limiter.max_busy);
-}
-
-int set_data_vio_pool_discard_limit(struct data_vio_pool *pool, 
data_vio_count_t limit)
-{
-       if (get_data_vio_pool_request_limit(pool) < limit) {
-               // The discard limit may not be higher than the data_vio limit.
-               return -EINVAL;
-       }
-
-       spin_lock(&pool->lock);
-       pool->discard_limiter.limit = limit;
-       spin_unlock(&pool->lock);
-
-       return VDO_SUCCESS;
-}
-
  data_vio_count_t get_data_vio_pool_active_requests(struct data_vio_pool *pool)
  {
        return READ_ONCE(pool->limiter.busy);
diff --git a/drivers/md/dm-vdo/data-vio.h b/drivers/md/dm-vdo/data-vio.h
index 25926b6cd98b..067b983bb291 100644
--- a/drivers/md/dm-vdo/data-vio.h
+++ b/drivers/md/dm-vdo/data-vio.h
@@ -336,11 +336,6 @@ void drain_data_vio_pool(struct data_vio_pool *pool, 
struct vdo_completion *comp
  void resume_data_vio_pool(struct data_vio_pool *pool, struct vdo_completion 
*completion);

void dump_data_vio_pool(struct data_vio_pool *pool, bool dump_vios);

-data_vio_count_t get_data_vio_pool_active_discards(struct data_vio_pool *pool);
-data_vio_count_t get_data_vio_pool_discard_limit(struct data_vio_pool *pool);
-data_vio_count_t get_data_vio_pool_maximum_discards(struct data_vio_pool 
*pool);
-int __must_check set_data_vio_pool_discard_limit(struct data_vio_pool *pool,
-                                                data_vio_count_t limit);
  data_vio_count_t get_data_vio_pool_active_requests(struct data_vio_pool 
*pool);
  data_vio_count_t get_data_vio_pool_request_limit(struct data_vio_pool *pool);
  data_vio_count_t get_data_vio_pool_maximum_requests(struct data_vio_pool 
*pool);