Hi Kent,
On Fri, Nov 15, 2024 at 03:35:16PM -0500, Kent Overstreet wrote:
> bcachefs: Kill bch2_get_next_backpointer()
As pointed out by the kbuild test robot [1], this change introduces several
-Wuninitialized warnings:
fs/bcachefs/move.c:814:36: error: variable 'dirty_sectors' is uninitialized
when used here [-Werror,-Wuninitialized]
814 | trace_evacuate_bucket(c, &bucket, dirty_sectors, bucket_size,
fragmentation, ret);
| ^~~~~~~~~~~~~
fs/bcachefs/move.c:677:24: note: initialize the variable 'dirty_sectors' to
silence this warning
677 | unsigned dirty_sectors, bucket_size;
| ^
| = 0
fs/bcachefs/move.c:814:51: error: variable 'bucket_size' is uninitialized
when used here [-Werror,-Wuninitialized]
814 | trace_evacuate_bucket(c, &bucket, dirty_sectors, bucket_size,
fragmentation, ret);
| ^~~~~~~~~~~
fs/bcachefs/move.c:677:37: note: initialize the variable 'bucket_size' to
silence this warning
677 | unsigned dirty_sectors, bucket_size;
| ^
| = 0
fs/bcachefs/move.c:814:64: error: variable 'fragmentation' is uninitialized
when used here [-Werror,-Wuninitialized]
814 | trace_evacuate_bucket(c, &bucket, dirty_sectors, bucket_size,
fragmentation, ret);
|
^~~~~~~~~~~~~
fs/bcachefs/move.c:678:19: note: initialize the variable 'fragmentation' to
silence this warning
678 | u64 fragmentation;
| ^
| = 0
3 errors generated.
I see you were already sent a change [2] to fix this but the fix should
go with the problematic change atomically. Build failures during the
merge window suck because they potentially hide other issues. I have a
further comment on that patch that I will send shortly.
[1]: https://lore.kernel.org/[email protected]/
[2]: https://lore.kernel.org/[email protected]/
Cheers,
Nathan
