bcachefs: fix ! vs ~ typo in __clear_bit_le64()

Dan Carpenter · Kent Overstreet · commit a6c4162d844d · 2024-04-05T14:42:37.000-04:00
The ! was obviously intended to be ~. As it is, this function does the equivalent to: "addr[bit / 64] = 0;". Fixes: 27fcec6 ("bcachefs: Clear recovery_passes_required as they complete without errors") Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org> Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>
diff --git a/fs/bcachefs/util.h b/fs/bcachefs/util.h
@@ -795,7 +795,7 @@ static inline void __set_bit_le64(size_t bit, __le64 *addr)
 
 static inline void __clear_bit_le64(size_t bit, __le64 *addr)
 {
-	addr[bit / 64] &= !cpu_to_le64(BIT_ULL(bit % 64));
+	addr[bit / 64] &= ~cpu_to_le64(BIT_ULL(bit % 64));
 }
 
 static inline bool test_bit_le64(size_t bit, __le64 *addr)

Original file line number	Diff line number	Diff line change
`@@ -795,7 +795,7 @@ static inline void __set_bit_le64(size_t bit, __le64 *addr)`
`795`	`795`
`796`	`796`	`static inline void __clear_bit_le64(size_t bit, __le64 *addr)`
`797`	`797`	`{`
`798`		`- addr[bit / 64] &= !cpu_to_le64(BIT_ULL(bit % 64));`
	`798`	`+ addr[bit / 64] &= ~cpu_to_le64(BIT_ULL(bit % 64));`
`799`	`799`	`}`
`800`	`800`
`801`	`801`	`static inline bool test_bit_le64(size_t bit, __le64 *addr)`