Re: [PATCH] sysctl: move local variable in proc_do_large_bitmap() to proper scope

2020-11-13 Thread Luis Chamberlain
On Mon, Nov 09, 2020 at 05:26:37PM -0800, Tom Rix wrote: > > On 11/8/20 11:11 PM, Lukas Bulwahn wrote: > > make clang-analyzer caught my attention with: > > > > kernel/sysctl.c:1511:4: warning: Value stored to 'first' is never read \ > > [clang-analyzer-deadcode.DeadStores] > >

Re: [PATCH] sysctl: move local variable in proc_do_large_bitmap() to proper scope

2020-11-09 Thread Tom Rix
On 11/8/20 11:11 PM, Lukas Bulwahn wrote: > make clang-analyzer caught my attention with: > > kernel/sysctl.c:1511:4: warning: Value stored to 'first' is never read \ > [clang-analyzer-deadcode.DeadStores] > first = 0; > ^ > > Commit

Re: [PATCH] sysctl: move local variable in proc_do_large_bitmap() to proper scope

2020-11-09 Thread Nathan Chancellor
On Mon, Nov 09, 2020 at 08:11:07AM +0100, Lukas Bulwahn wrote: > make clang-analyzer caught my attention with: > > kernel/sysctl.c:1511:4: warning: Value stored to 'first' is never read \ > [clang-analyzer-deadcode.DeadStores] > first = 0; >

[PATCH] sysctl: move local variable in proc_do_large_bitmap() to proper scope

2020-11-08 Thread Lukas Bulwahn
make clang-analyzer caught my attention with: kernel/sysctl.c:1511:4: warning: Value stored to 'first' is never read \ [clang-analyzer-deadcode.DeadStores] first = 0; ^ Commit 9f977fb7ae9d ("sysctl: add proc_do_large_bitmap") introduced