diff options
author | Uros Bizjak <ubizjak@gmail.com> | 2022-08-09 18:36:33 +0200 |
---|---|---|
committer | David Sterba <dsterba@suse.com> | 2022-09-26 12:27:55 +0200 |
commit | e5677f056010298d3a8d43f710f8b445dd73462d (patch) | |
tree | 50fbd1389650c103540c84998d1c3e556d49b44d /fs/verity | |
parent | btrfs: scrub: remove impossible sanity checks (diff) | |
download | linux-e5677f056010298d3a8d43f710f8b445dd73462d.tar.xz linux-e5677f056010298d3a8d43f710f8b445dd73462d.zip |
btrfs: use atomic_try_cmpxchg in free_extent_buffer
Use `atomic_try_cmpxchg(ptr, &old, new)` instead of
`atomic_cmpxchg(ptr, old, new) == old` in free_extent_buffer. This
has two benefits:
- The x86 cmpxchg instruction returns success in the ZF flag, so this
change saves a compare after cmpxchg, as well as a related move
instruction in the front of cmpxchg.
- atomic_try_cmpxchg implicitly assigns the *ptr value to &old when
cmpxchg fails, enabling further code simplifications.
This patch has no functional change.
Reviewed-by: Boris Burkov <boris@bur.io>
Signed-off-by: Uros Bizjak <ubizjak@gmail.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/verity')
0 files changed, 0 insertions, 0 deletions