diff options
| author | Miao Xie <miaox@cn.fujitsu.com> | 2012-09-06 04:04:44 -0600 |
|---|---|---|
| committer | Chris Mason <chris.mason@fusionio.com> | 2012-10-01 15:19:13 -0400 |
| commit | 69ce977a179750915e04fcc12bfbe33e6c8f5132 (patch) | |
| tree | c353f29055286be143052b6b08daa72152489bc2 | |
| parent | 2ecb79239bcd04c9d410f4cdce16adb6840b19da (diff) | |
| download | blackbird-op-linux-69ce977a179750915e04fcc12bfbe33e6c8f5132.tar.gz blackbird-op-linux-69ce977a179750915e04fcc12bfbe33e6c8f5132.zip | |
Btrfs: output more information when aborting a unused transaction handle
Though we dump the stack information when aborting a unused transaction
handle, we don't know the correct place where we decide to abort the
transaction handle if one function has several place where the transaction
abort function is invoked and jumps to the same place after this call.
And beside that we also don't know the reason why we jump to abort
the current handle. So I modify the transaction abort function and make
it output the function name, line and error information.
Signed-off-by: Miao Xie <miaox@cn.fujitsu.com>
| -rw-r--r-- | fs/btrfs/super.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/fs/btrfs/super.c b/fs/btrfs/super.c index 06ff1dd0f9b7..867e8e799dea 100644 --- a/fs/btrfs/super.c +++ b/fs/btrfs/super.c @@ -248,7 +248,13 @@ void __btrfs_abort_transaction(struct btrfs_trans_handle *trans, /* Nothing used. The other threads that have joined this * transaction may be able to continue. */ if (!trans->blocks_used) { - btrfs_printk(root->fs_info, "Aborting unused transaction.\n"); + char nbuf[16]; + const char *errstr; + + errstr = btrfs_decode_error(root->fs_info, errno, nbuf); + btrfs_printk(root->fs_info, + "%s:%d: Aborting unused transaction(%s).\n", + function, line, errstr); return; } trans->transaction->aborted = errno; |
