The macro allocate_dropping_locks accepts a parameter _trans,
but it was not used, rather the variable trans was directly used,
which may be a local variable inside a function that calls the macros.

Signed-off-by: Julian Sun <sunjunchao2...@gmail.com>
---
 fs/bcachefs/btree_iter.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/bcachefs/btree_iter.h b/fs/bcachefs/btree_iter.h
index 489004173d28..b4e71b590bc9 100644
--- a/fs/bcachefs/btree_iter.h
+++ b/fs/bcachefs/btree_iter.h
@@ -860,7 +860,7 @@ __bch2_btree_iter_peek_and_restart(struct btree_trans 
*trans,
        _ret = 0;                                                       \
        if (unlikely(!_p)) {                                            \
                _gfp = GFP_KERNEL;                                      \
-               _ret = drop_locks_do(trans, ((_p = _do), 0));           \
+               _ret = drop_locks_do(_trans, ((_p = _do), 0));          \
        }                                                               \
        _p;                                                             \
 })
-- 
2.39.2


Reply via email to