As all changed_ functions need to return something, just return 0 directly here, as the verity status is passed via the context. Suggested-by: David Sterba <dsterba@xxxxxxxx> Reported-by: kernel test robot <lkp@xxxxxxxxx> Closes: https://lore.kernel.org/oe-kbuild-all/202410092305.WbyqspH8-lkp@xxxxxxxxx/ Signed-off-by: Christian Heusel <christian@xxxxxxxxx> --- fs/btrfs/send.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/fs/btrfs/send.c b/fs/btrfs/send.c index 7f48ba6c1c77a0862932bdeffdf7b350267ca544..3f7e100a63cd5e444f8cd76c24114a5855a86e61 100644 --- a/fs/btrfs/send.c +++ b/fs/btrfs/send.c @@ -7167,13 +7167,11 @@ static int changed_extent(struct send_ctx *sctx, static int changed_verity(struct send_ctx *sctx, enum btrfs_compare_tree_result result) { - int ret = 0; - if (!sctx->cur_inode_new_gen && !sctx->cur_inode_deleted) { if (result == BTRFS_COMPARE_TREE_NEW) sctx->cur_inode_needs_verity = true; } - return ret; + return 0; } static int dir_changed(struct send_ctx *sctx, u64 dir) --- base-commit: 9852d85ec9d492ebef56dc5f229416c925758edc change-id: 20241010-btrfs-return-cleanup-57ebbbf53cbe Best regards, -- Christian Heusel <christian@xxxxxxxxx>
