diff options
| author | Chris Mason <clm@fb.com> | 2016-12-09 16:56:33 +0300 |
|---|---|---|
| committer | Chris Mason <clm@fb.com> | 2016-12-09 17:00:28 +0300 |
| commit | e5d6b12fe14e89ea1c494585c47b1dfb31d71183 (patch) | |
| tree | fe1b35f4766e8e49a5d0a910281faa208cdc3513 /fs/btrfs/ctree.h | |
| parent | 34441361c4f52a5f6e41d8de8e5debbeb415dbf0 (diff) | |
| download | linux-e5d6b12fe14e89ea1c494585c47b1dfb31d71183.tar.xz | |
Btrfs: don't WARN() in btrfs_transaction_abort() for IO errors
btrfs_transaction_abort() has a WARN() to help us nail down whatever
problem lead to the abort. But most of the time, we're aborting for EIO,
and the warning just adds noise.
Signed-off-by: Chris Mason <clm@fb.com>
Diffstat (limited to 'fs/btrfs/ctree.h')
| -rw-r--r-- | fs/btrfs/ctree.h | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/fs/btrfs/ctree.h b/fs/btrfs/ctree.h index fc1864acb368..50bcfb80d33a 100644 --- a/fs/btrfs/ctree.h +++ b/fs/btrfs/ctree.h @@ -3445,9 +3445,14 @@ do { \ /* Report first abort since mount */ \ if (!test_and_set_bit(BTRFS_FS_STATE_TRANS_ABORTED, \ &((trans)->fs_info->fs_state))) { \ - WARN(1, KERN_DEBUG \ - "BTRFS: Transaction aborted (error %d)\n", \ - (errno)); \ + if ((errno) != -EIO) { \ + WARN(1, KERN_DEBUG \ + "BTRFS: Transaction aborted (error %d)\n", \ + (errno)); \ + } else { \ + pr_debug("BTRFS: Transaction aborted (error %d)\n", \ + (errno)); \ + } \ } \ __btrfs_abort_transaction((trans), __func__, \ __LINE__, (errno)); \ |