btrfs: clean up transaction abort messages

The transaction abort stacktrace is printed only once per module
lifetime, but we'd like to see it each time it happens per mounted
filesystem.  Introduce a fs_state flag that records it.

Tweak the messages around abort:
* add error number to the first abort
* print the exact negative errno from btrfs_decode_error
* clean up btrfs_decode_error and callers
* no dots at the end of the messages

Signed-off-by: David Sterba <dsterba@suse.cz>
Signed-off-by: Josef Bacik <jbacik@fusionio.com>

1 files changed, 2 insertions, 3 deletions

diff --git a/fs/btrfs/transaction.c b/fs/btrfs/transaction.c
index 50767bbaad6c..6c0a72ab6de0 100644
--- a/fs/btrfs/transaction.c
+++ b/fs/btrfs/transaction.c
@@ -1808,7 +1808,7 @@ int btrfs_commit_transaction(struct btrfs_trans_handle *trans,
         ret = btrfs_write_and_wait_transaction(trans, root);
         if (ret) {
                 btrfs_error(root->fs_info, ret,
-                            "Error while writing out transaction.");
+                            "Error while writing out transaction");
                 mutex_unlock(&root->fs_info->tree_log_mutex);
                 goto cleanup_transaction;
         }
@@ -1864,8 +1864,7 @@ cleanup_transaction:
                 btrfs_qgroup_free(root, trans->qgroup_reserved);
                 trans->qgroup_reserved = 0;
         }
-        btrfs_printk(root->fs_info, "Skipping commit of aborted transaction.\n");
-//      WARN_ON(1);
+        btrfs_printk(root->fs_info, "Skipping commit of aborted transaction\n");
         if (current->journal_info == trans)
                 current->journal_info = NULL;
         cleanup_transaction(trans, root, ret);