diff options
| author |   David Sterba <dsterba@suse.com> | 2017-08-18 18:16:25 +0200 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com> | 2017-08-22 13:22:05 +0200 |
| commit | db95c876c568cef951fbbd4c0118cb5386e4bb99 (patch) | |
| tree | f222af5b05e1c3e0a11e1bd252604568138151f2 | |
| parent | btrfs: remove unnecessary memory barrier in btrfs_direct_IO (diff) | |
| download | linux-dev-db95c876c568cef951fbbd4c0118cb5386e4bb99.tar.xz linux-dev-db95c876c568cef951fbbd4c0118cb5386e4bb99.zip | |
btrfs: submit superblock io with REQ_META and REQ_PRIO
The superblock is also metadata of the filesystem so the relevant IO
should be tagged as such. We also tag it as high priority, as it's the
last block committed for metadata from a given transaction. Any delays
would effectively block the whole transaction, also blocking any other
operation holding the device_list_mutex.
Reviewed-by: Josef Bacik <jbacik@fb.com>
Reviewed-by: Liu Bo <bo.li.liu@oracle.com>
Signed-off-by: David Sterba <dsterba@suse.com>
| -rw-r--r-- | fs/btrfs/disk-io.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/fs/btrfs/disk-io.c b/fs/btrfs/disk-io.c index 90b967ae46d0..27d458640536 100644 --- a/fs/btrfs/disk-io.c +++ b/fs/btrfs/disk-io.c @@ -3433,9 +3433,10 @@ static int write_dev_supers(struct btrfs_device *device, */ if (i == 0) { ret = btrfsic_submit_bh(REQ_OP_WRITE, - REQ_SYNC | REQ_FUA, bh); + REQ_SYNC | REQ_FUA | REQ_META | REQ_PRIO, bh); } else { - ret = btrfsic_submit_bh(REQ_OP_WRITE, REQ_SYNC, bh); + ret = btrfsic_submit_bh(REQ_OP_WRITE, + REQ_SYNC | REQ_META | REQ_PRIO, bh); } if (ret) errors++; |