* Andrew Morton <a...@linux-foundation.org> [220716 21:03]: > On Thu, 14 Jul 2022 09:56:19 +0800 kernel test robot <l...@intel.com> wrote: > > > lib/maple_tree.c:1522:52: warning: Parameter 'gaps' can be declared with > > const [constParameter] > > lib/maple_tree.c:1871:21: warning: Array index 'split' is used before > > limits check. [arrayIndexThenCheck] > > lib/maple_tree.c:2033:55: warning: Parameter 'mas' can be declared with > > const [constParameter] > > lib/maple_tree.c:2426:8: warning: Redundant initialization for 'r_tmp'. The > > initialized value is overwritten before it is read. > > [redundantInitialization] > > lib/maple_tree.c:2427:8: warning: Redundant initialization for 'l_tmp'. The > > initialized value is overwritten before it is read. > > [redundantInitialization] > > lib/maple_tree.c:3160:22: warning: Found suspicious operator ',' > > [constStatement] > > lib/maple_tree.c:3208:11: warning: Size of pointer 'pivs' used instead of > > size of its data. [pointerSize] > > lib/maple_tree.c:326:2: warning: Assignment of function parameter has no > > effect outside the function. Did you forget dereferencing it? > > [uselessAssignmentPtrArg] > > lib/maple_tree.c:4266:15: warning: The if condition is the same as the > > previous if condition [duplicateCondition] > > lib/maple_tree.c:4302:23: warning: Boolean result is used in bitwise > > operation. Clarify expression with parentheses. [clarifyCondition] > > lib/maple_tree.c:694:59: warning: Parameter 'pivots' can be declared with > > const [constParameter] > > lib/test_printf.c:415:11: warning: Local variable 'addr' shadows outer > > function [shadowFunction] > > mm/highmem.c:737:13: warning: Uninitialized variable: pam->page [uninitvar] > > mm/migrate.c:355:53: warning: Parameter 'mapping' can be declared with > > const [constParameter] > > mm/migrate.c:875:7: warning: Redundant initialization for 'rc'. The > > initialized value is overwritten before it is read. > > [redundantInitialization] > > mm/mlock.c:230:20: warning: Using pointer that is a temporary. > > [danglingTemporaryLifetime] > > mm/slab.c:1635:24: warning: Uninitialized variables: slab.__page_flags, > > slab.__unused_1, slab.freelist, slab.units, slab.__unused_2, > > slab.__page_refcount [uninitvar] > > mm/slab.c:3289:7: warning: Redundant assignment of 'objp' to itself. > > [selfAssignment] > > mm/slab.c:3509:8: warning: Redundant assignment of 'p[i]' to itself. > > [selfAssignment] > > mm/slab.c:405:9: warning: Local variable 'slab_size' shadows outer function > > [shadowFunction] > > mm/vmstat.c:1409:53: warning: Parameter 'pos' can be declared with const > > [constParameter] > > mm/vmstat.c:1650:68: warning: Parameter 'zone' can be declared with const > > [constParameter] > > mm/zsmalloc.c:2019:15: warning: Uninitialized variables: zspage.huge, > > zspage.fullness, zspage.class, zspage.isolated, zspage.magic, zspage.inuse, > > zspage.freeobj, zspage.first_page, zspage.lock [uninitvar] > > mm/zsmalloc.c:2060:16: warning: Local variable 'obj_allocated' shadows > > outer function [shadowFunction] > > urgh, thanks, lots of stuff to go through here. > > Liam, I suggest we worry about the mapletree things at a later time ;)
Thanks. I'll dig into them on top of v11, which I just sent out.
