On March 17, 2020 6:48:20 PM EDT, "Paul E. McKenney" <paulmck@xxxxxxxxxx> wrote: >On Tue, Mar 17, 2020 at 06:30:51PM -0400, Joel Fernandes wrote: >> On Tue, Mar 17, 2020 at 5:45 PM Paul E. McKenney <paulmck@xxxxxxxxxx> >wrote: >> > >> > On Tue, Mar 17, 2020 at 02:08:22PM -0700, Paul E. McKenney wrote: >> > > On Mon, Mar 16, 2020 at 12:32:26PM -0400, Joel Fernandes (Google) >wrote: >> > > > This allows us to increase memory pressure dynamically using a >new >> > > > rcuperf boot command line parameter called 'rcumult'. >> > > > >> > > > Signed-off-by: Joel Fernandes (Google) <joel@xxxxxxxxxxxxxxxxx> >> > > >> > > Applied for testing and review, thank you! >> > >> > But testing did not go far: >> > >> > kernel/rcu/tree.c: In function ‘kfree_rcu_shrink_count’: >> > kernel/rcu/tree.c:3120:16: warning: unused variable ‘flags’ >[-Wunused-variable] >> > unsigned long flags, count = 0; >> >> I fixed the warning already but did not resend since it was just the >> one unused variable warning. The patches are otherwise good to apply. >> Sorry, and I can resend it soon if you are not reapplying right now. > >So remove "flags, " and all is well? Yes, that's right. I dropped the lock but forgot to remove it. >If so, I can just as easily fix that as take a new series. But next >time, please give a fella a warning. ;-) Will do, my bad. Thank you ;-) - Joel > > Thanx, Paul -- Sent from my Android device with K-9 Mail. Please excuse my brevity.
