diff options
| author | Miao Xie <miaox@cn.fujitsu.com> | 2013-01-15 06:27:25 (GMT) |
|---|---|---|
| committer | Josef Bacik <jbacik@fusionio.com> | 2013-01-24 17:51:23 (GMT) |
| commit | 8d25a086eb104297e3ba1fdd180b04cfaaa84797 (patch) | |
| tree | 075e3a0e731b08a418ad65dfd4e0bb53818a40c1 /fs/btrfs/super.c | |
| parent | e58dd74bccb4317e39e4b675bf9c6cd133608fac (diff) | |
| download | linux-8d25a086eb104297e3ba1fdd180b04cfaaa84797.tar.xz | |
Btrfs: Add ACCESS_ONCE() to transaction->abort accesses
We may access and update transaction->aborted on the different CPUs without
lock, so we need ACCESS_ONCE() wrapper to prevent the compiler from creating
unsolicited accesses and make sure we can get the right value.
Signed-off-by: Miao Xie <miaox@cn.fujitsu.com>
Signed-off-by: Josef Bacik <jbacik@fusionio.com>
Diffstat (limited to 'fs/btrfs/super.c')
| -rw-r--r-- | fs/btrfs/super.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/btrfs/super.c b/fs/btrfs/super.c index 99545df..d8982e9 100644 --- a/fs/btrfs/super.c +++ b/fs/btrfs/super.c @@ -267,7 +267,7 @@ void __btrfs_abort_transaction(struct btrfs_trans_handle *trans, function, line, errstr); return; } - trans->transaction->aborted = errno; + ACCESS_ONCE(trans->transaction->aborted) = errno; __btrfs_std_error(root->fs_info, function, line, errno, NULL); } /* |