From 19337f8ea3fc0ed177142bbaf8496cda26f1432e Mon Sep 17 00:00:00 2001 From: Anand Jain Date: Sat, 11 Feb 2023 00:15:55 +0800 Subject: btrfs: switch search_file_offset_in_bio to return bool Function search_file_offset_in_bio() finds the file offset in the file_offset_ret, and we use the return value to indicate if it is successful, so use bool. Reviewed-by: Christoph Hellwig Signed-off-by: Anand Jain Reviewed-by: David Sterba Signed-off-by: David Sterba --- fs/btrfs/file-item.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'fs/btrfs/file-item.c') diff --git a/fs/btrfs/file-item.c b/fs/btrfs/file-item.c index 89e9415b8f06..fff09e5635e5 100644 --- a/fs/btrfs/file-item.c +++ b/fs/btrfs/file-item.c @@ -345,8 +345,8 @@ out: * * @inode is used to determine if the bvec page really belongs to @inode. * - * Return 0 if we can't find the file offset - * Return >0 if we find the file offset and restore it to @file_offset_ret + * Return false if we can't find the file offset + * Return true if we find the file offset and restore it to @file_offset_ret */ static int search_file_offset_in_bio(struct bio *bio, struct inode *inode, u64 disk_bytenr, u64 *file_offset_ret) @@ -354,7 +354,7 @@ static int search_file_offset_in_bio(struct bio *bio, struct inode *inode, struct bvec_iter iter; struct bio_vec bvec; u64 cur = bio->bi_iter.bi_sector << SECTOR_SHIFT; - int ret = 0; + bool ret = false; bio_for_each_segment(bvec, bio, iter) { struct page *page = bvec.bv_page; @@ -368,7 +368,7 @@ static int search_file_offset_in_bio(struct bio *bio, struct inode *inode, ASSERT(in_range(disk_bytenr, cur, bvec.bv_len)); if (page->mapping && page->mapping->host && page->mapping->host == inode) { - ret = 1; + ret = true; *file_offset_ret = page_offset(page) + bvec.bv_offset + disk_bytenr - cur; break; -- cgit v1.2.3