On Sun, Jan 16, 2022 at 4:45 PM Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx> wrote: > > Hi all, > > After merging the origin tree, today's linux-next build (powerpc > allnoconfig) produced this warning: > > kernel/sysctl.c:125:12: warning: 'one_thousand' defined but not used [-Wunused-variable] > 125 | static int one_thousand = 1000; > | ^~~~~~~~~~~~ > > Introduced by commit > > 39c65a94cd96 ("mm/pagealloc: sysctl: change watermark_scale_factor max limit to 30%") > > It seems to only be used when CONFIG_PERF_EVENTS is set. > > I didn't see it in linux-next because there is a patch later in the > akpm series (not yet in Linus' tree) that removes this variable. Sorry, I saw the warning before when backporting this patch but I didn't realize that the followup patches refactoring this code would not be merged. Posted a fix at: https://lore.kernel.org/all/20220117012317.21168-1-surenb@xxxxxxxxxx/ Thanks! > > -- > Cheers, > Stephen Rothwell