btrfs: scrub: inline helper scrub_free_wr_ctx
authorDavid Sterba <dsterba@suse.com>
Tue, 16 May 2017 17:10:26 +0000 (19:10 +0200)
committerDavid Sterba <dsterba@suse.com>
Mon, 19 Jun 2017 16:26:00 +0000 (18:26 +0200)
The helper scrub_free_wr_ctx is used only once and fits into
scrub_free_ctx as it continues sctx shutdown, no need to keep it
separate.

Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/scrub.c

index 2ab36973ad7ed81513f3bf7ed4023580b826f15f..c2d4f25417f2e9029bc9e16ded55241d56a38503 100644 (file)
@@ -289,7 +289,6 @@ static void scrub_remap_extent(struct btrfs_fs_info *fs_info,
                               u64 *extent_physical,
                               struct btrfs_device **extent_dev,
                               int *extent_mirror_num);
-static void scrub_free_wr_ctx(struct scrub_wr_ctx *wr_ctx);
 static int scrub_add_page_to_wr_bio(struct scrub_ctx *sctx,
                                    struct scrub_page *spage);
 static void scrub_wr_submit(struct scrub_ctx *sctx);
@@ -640,7 +639,10 @@ static noinline_for_stack void scrub_free_ctx(struct scrub_ctx *sctx)
        if (!sctx)
                return;
 
-       scrub_free_wr_ctx(&sctx->wr_ctx);
+       mutex_lock(&sctx->wr_ctx.wr_lock);
+       kfree(sctx->wr_ctx.wr_curr_bio);
+       sctx->wr_ctx.wr_curr_bio = NULL;
+       mutex_unlock(&sctx->wr_ctx.wr_lock);
 
        /* this can happen when scrub is cancelled */
        if (sctx->curr != -1) {
@@ -4337,14 +4339,6 @@ static void scrub_remap_extent(struct btrfs_fs_info *fs_info,
        btrfs_put_bbio(bbio);
 }
 
-static void scrub_free_wr_ctx(struct scrub_wr_ctx *wr_ctx)
-{
-       mutex_lock(&wr_ctx->wr_lock);
-       kfree(wr_ctx->wr_curr_bio);
-       wr_ctx->wr_curr_bio = NULL;
-       mutex_unlock(&wr_ctx->wr_lock);
-}
-
 static int copy_nocow_pages(struct scrub_ctx *sctx, u64 logical, u64 len,
                            int mirror_num, u64 physical_for_dev_replace)
 {