diff options
| author | Xuanqiang Luo <luoxuanqiang@kylinos.cn> | 2025-10-15 15:05:21 +0800 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com> | 2025-11-24 22:03:02 +0100 |
| commit | 225e747ea57781198b44cb65373d076865c51a7a (patch) | |
| tree | 78cdc7bb0ab075417cf89be52d4b619559fcdcd7 | |
| parent | a232ff90d14657c8637c6e94b606bb5d700a2ecb (diff) | |
btrfs: remove redundant refcount check in btrfs_put_transaction()
Eric Dumazet removed the redundant refcount check for sk_refcnt, I
noticed a similar issue in btrfs_put_transaction().
refcount_dec_and_test() already checks for a zero refcount and
complains, making the preceding WARN_ON redundant. This is a leftover
from the atomic_t times.
Signed-off-by: Xuanqiang Luo <luoxuanqiang@kylinos.cn>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
| -rw-r--r-- | fs/btrfs/transaction.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/fs/btrfs/transaction.c b/fs/btrfs/transaction.c index 6607e354eae5..907f2d047b44 100644 --- a/fs/btrfs/transaction.c +++ b/fs/btrfs/transaction.c @@ -138,7 +138,6 @@ static const unsigned int btrfs_blocked_trans_types[TRANS_STATE_MAX] = { void btrfs_put_transaction(struct btrfs_transaction *transaction) { - WARN_ON(refcount_read(&transaction->use_count) == 0); if (refcount_dec_and_test(&transaction->use_count)) { BUG_ON(!list_empty(&transaction->list)); WARN_ON(!xa_empty(&transaction->delayed_refs.head_refs)); |