diff options
| author | Jeff Layton <[email protected]> | 2017-05-25 06:39:52 -0400 |
|---|---|---|
| committer | David Sterba <[email protected]> | 2017-06-19 18:26:01 +0200 |
| commit | 3189ff778630d9ce8a9f2fda9f8ae4510cceb154 (patch) | |
| tree | 3a54d9e4db65217606a47189b0c11ced8094e8d1 /tools/perf/scripts/python | |
| parent | 118c701e20ad6edfb35fd83ec6989991714b1d89 (diff) | |
btrfs: btrfs_wait_tree_block_writeback can be void return
Nothing checks its return value.
Is it safe to skip checking return value of btrfs_wait_tree_block_writeback?
Liu Bo: I think yes, it's used in walk_log_tree which is called in two
places, free_log_tree and log replay. For free_log_tree, it waits for
any running writeback of the extent buffer under freeing to finish in
case we need to access the eb pointer from page->private, and it's OK to
not check the return value, while for log replay, it's doesn't wait
because wc->wait is not set. So neither cares about the writeback error.
Signed-off-by: Jeff Layton <[email protected]>
Reviewed-by: Jan Kara <[email protected]>
Reviewed-by: Liu Bo <[email protected]>
[ added more explanation to changelog, from Liu Bo ]
Signed-off-by: David Sterba <[email protected]>
Diffstat (limited to 'tools/perf/scripts/python')
0 files changed, 0 insertions, 0 deletions