diff options
| author |   Liu Bo <bo.li.liu@oracle.com> | 2017-09-07 11:22:20 -0600 |
|---|---|---|
| committer |   David Sterba <dsterba@suse.com> | 2017-10-30 12:27:59 +0100 |
| commit | f851689b5ae3eb8e1b4d397eaf2805ccfd88bf77 (patch) | |
| tree | f03b7128699fb5449ac8b2c3b6e5a5dd0ea5cbf9 /fs/btrfs/ctree.h | |
| parent | btrfs: tree-checker: Enhance output for check_extent_data_item (diff) | |
| download | linux-dev-f851689b5ae3eb8e1b4d397eaf2805ccfd88bf77.tar.xz linux-dev-f851689b5ae3eb8e1b4d397eaf2805ccfd88bf77.zip | |
Btrfs: remove nr_async_bios
This was intended to congest higher layers to not send bios, but as
1) the congested bit has been taken by writeback
Async bios come from buffered writes and DIO writes.
For DIO writes, we want to submit them ASAP, while for buffered writes,
writeback uses balance_dirty_pages() to throttle how much dirty pages we
can have.
2) and no one is waiting for %nr_async_bios down to zero,
Historically, it was introduced along with changes which let
checksumming workload spread accross different cpus. And at that time,
pdflush was used instead of per-bdi flushing, perhaps pdflush did not
have the necessary information for writeback to do throttling.
We can safely remove them now.
Signed-off-by: Liu Bo <bo.li.liu@oracle.com>
[ additional explanation from mails, removed unused variable 'limit' ]
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/ctree.h')
| -rw-r--r-- | fs/btrfs/ctree.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/fs/btrfs/ctree.h b/fs/btrfs/ctree.h index 16b3537f31d4..bc1b6a033700 100644 --- a/fs/btrfs/ctree.h +++ b/fs/btrfs/ctree.h @@ -880,7 +880,6 @@ struct btrfs_fs_info { atomic_t nr_async_submits; atomic_t async_submit_draining; - atomic_t nr_async_bios; atomic_t async_delalloc_pages; atomic_t open_ioctl_trans; |