Comment by lukeh
Sure. It was an uninitialized variable in the retry loop below. Fixed now.
https://github.com/PADL/linux/commit/b83b9619eecc02c5e95a1d3...
Sure. It was an uninitialized variable in the retry loop below. Fixed now.
https://github.com/PADL/linux/commit/b83b9619eecc02c5e95a1d3...
I think it contains the bug (min_attempts), not the fix.
Compilers have been warning about unitialized variables for a very long time now. The claim was that the compiler missed that in a patch you reviewed for the kernel. The link posted does not show the review discussion nor a commit log showing that incremental fix, and so does not at all support the claim.
Look like valgrind would be able to catch that too.