diff options
author | Qu Wenruo <wqu@suse.com> | 2020-10-21 14:24:54 +0800 |
---|---|---|
committer | David Sterba <dsterba@suse.com> | 2020-12-08 15:53:55 +0100 |
commit | 265d4ac03fdf75e84002d5749854c77d7240aa81 (patch) | |
tree | e8657fcb52ad5514a27108dd9fbc0add19862c7e /fs/btrfs/tests/btrfs-tests.c | |
parent | 8896a08d8ea95809adbc3742cdf9c7575f66c960 (diff) |
btrfs: sink parameter start and len to check_data_csum
For check_data_csum(), the page we're using is directly from the inode
mapping, thus it has valid page_offset().
We can use (page_offset() + pg_off) to replace @start parameter
completely, while the @len should always be sectorsize.
Since we're here, also add some comment, as there are quite some
confusion in words like start/offset, without explaining whether it's
file_offset or logical bytenr.
This should not affect the existing behavior, as for current sectorsize
== PAGE_SIZE case, @pgoff should always be 0, and len is always
PAGE_SIZE (or sectorsize from the dio read path).
Reviewed-by: Goldwyn Rodrigues <rgoldwyn@suse.com>
Signed-off-by: Qu Wenruo <wqu@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/tests/btrfs-tests.c')
0 files changed, 0 insertions, 0 deletions