diff options
| author | Anand Jain <anand.jain@oracle.com> | 2023-02-10 19:15:55 +0300 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com> | 2023-04-17 19:01:13 +0300 |
| commit | 19337f8ea3fc0ed177142bbaf8496cda26f1432e (patch) | |
| tree | f5f7289ddc588f2605a2fa7012814618655cb297 /fs/btrfs/file-item.c | |
| parent | da8269a3e9edb8c57cd5448e07a5157872ddd143 (diff) | |
| download | linux-19337f8ea3fc0ed177142bbaf8496cda26f1432e.tar.xz | |
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 <hch@lst.de>
Signed-off-by: Anand Jain <anand.jain@oracle.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/file-item.c')
| -rw-r--r-- | fs/btrfs/file-item.c | 8 |
1 files changed, 4 insertions, 4 deletions
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; |