• Qu Wenruo's avatar
    btrfs: sink parameter start and len to check_data_csum · 265d4ac0
    Qu Wenruo authored
    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: default avatarGoldwyn Rodrigues <rgoldwyn@suse.com>
    Signed-off-by: default avatarQu Wenruo <wqu@suse.com>
    Reviewed-by: default avatarDavid Sterba <dsterba@suse.com>
    Signed-off-by: default avatarDavid Sterba <dsterba@suse.com>
    265d4ac0
inode.c 285 KB