With moving crypt_free_buffer_pages() before crypt_alloc_buffer(), we
don't need an extra declaration anymore.

Signed-off-by: Yang Shi <shy828...@gmail.com>
---
 drivers/md/dm-crypt.c | 23 +++++++++++------------
 1 file changed, 11 insertions(+), 12 deletions(-)

diff --git a/drivers/md/dm-crypt.c b/drivers/md/dm-crypt.c
index 2653516bcdef..73069f200cc5 100644
--- a/drivers/md/dm-crypt.c
+++ b/drivers/md/dm-crypt.c
@@ -1639,7 +1639,17 @@ static blk_status_t crypt_convert(struct crypt_config 
*cc,
        return 0;
 }
 
-static void crypt_free_buffer_pages(struct crypt_config *cc, struct bio 
*clone);
+
+static void crypt_free_buffer_pages(struct crypt_config *cc, struct bio *clone)
+{
+       struct bio_vec *bv;
+       struct bvec_iter_all iter_all;
+
+       bio_for_each_segment_all(bv, clone, iter_all) {
+               BUG_ON(!bv->bv_page);
+               mempool_free(bv->bv_page, &cc->page_pool);
+       }
+}
 
 /*
  * Generate a new unfragmented bio with the given size
@@ -1707,17 +1717,6 @@ static struct bio *crypt_alloc_buffer(struct dm_crypt_io 
*io, unsigned size)
        return clone;
 }
 
-static void crypt_free_buffer_pages(struct crypt_config *cc, struct bio *clone)
-{
-       struct bio_vec *bv;
-       struct bvec_iter_all iter_all;
-
-       bio_for_each_segment_all(bv, clone, iter_all) {
-               BUG_ON(!bv->bv_page);
-               mempool_free(bv->bv_page, &cc->page_pool);
-       }
-}
-
 static void crypt_io_init(struct dm_crypt_io *io, struct crypt_config *cc,
                          struct bio *bio, sector_t sector)
 {
-- 
2.39.0

--
dm-devel mailing list
dm-devel@redhat.com
https://listman.redhat.com/mailman/listinfo/dm-devel

Reply via email to