# Re: \[PATCH v7 04/10\] btrfs: fix check\_data\_csum() error message for direct I/O

- From: Josef Bacik \<josef@toxicpanda.com\>
- Date: 2021-01-25 21:01:39 
- List: linux-api
- Page: https://ratatoskr.run/linux-api/2021/01/198320
- Thread: https://ratatoskr.run/linux-fsdevel/2021/01/198194/t.md

```text
On 1/22/21 3:46 PM, Omar Sandoval wrote:
> From: Omar Sandoval <redacted>
>
> Commit 1dae796aabf6 ("btrfs: inode: sink parameter start and len to
> check_data_csum()") replaced the start parameter to check_data_csum()
> with page_offset(), but page_offset() is not meaningful for direct I/O
> pages. Bring back the start parameter.
>
> Fixes: 265d4ac03fdf ("btrfs: sink parameter start and len to check_data_csum")
> Signed-off-by: Omar Sandoval <redacted>

Reviewed-by: Josef Bacik <josef@toxicpanda.com>

Thanks,

Josef
```