Re: [PATCH 00/16] rcu: Some minor fixes and cleanups

Paul E. McKenney
2014-08-27T03:20:26Z

On Tue, Aug 26, 2014 at 09:10:10PM -0400, Pranith Kumar wrote:
> On Wed, Jul 23, 2014 at 10:45 AM, Paul E. McKenney
> <paulmck@linux.vnet.ibm.com> wrote:
> > On Wed, Jul 23, 2014 at 01:09:37AM -0400, Pranith Kumar wrote:
> >> Hi Paul,
> >>
> >> This is a series of minor fixes and cleanup patches which I found while studying
> >> the code. All my previous pending (but not rejected ;) patches are superseded by
> >> this series, expect the rcutorture snprintf changes. I am still waiting for you
> >> to decide on that one :)
> >>
> >> These changes have been tested by the kvm rcutorture test setup. Some tests give
> >> me stall warnings, but otherwise have SUCCESS messages in the logs.
> >
> > For patches 1, 3, 5, 8, 12, and 13, once you get a Reviewed-by from one
> > of the co-maintainers or designated reviewers, I will queue them.
> > The other patches I have responded to.
>
> Hi Paul, just a reminder so that these don't get forgotten :)

Hello, Pranith, haven't forgotten them, but also haven't seen any
reviews.

Thanx, Paul

> >> Pranith Kumar (16):
> >> rcu: Use rcu_num_nodes instead of NUM_RCU_NODES
> >> rcu: Check return value for cpumask allocation
> >> rcu: Fix comment for gp_state field values
> >> rcu: Remove redundant check for an online CPU
> >> rcu: Add noreturn attribute to boost kthread
> >> rcu: Clear gp_flags only when actually starting new gp
> >> rcu: Save and restore irq flags in rcu_gp_cleanup()
> >> rcu: Clean up rcu_spawn_one_boost_kthread()
> >> rcu: Remove redundant check for online cpu
> >> rcu: Check for RCU_FLAG_GP_INIT bit in gp_flags for spurious wakeup
> >> rcu: Check for spurious wakeup using return value
> >> rcu: Rename rcu_spawn_gp_kthread() to rcu_spawn_kthreads()
> >> rcu: Spawn nocb kthreads from rcu_prepare_kthreads()
> >> rcu: Remove redundant checks for rcu_scheduler_fully_active
> >> rcu: Check for a nocb cpu before trying to spawn nocb threads
> >> rcu: kvm.sh: Fix error when you pass --cpus argument
> >>
> >> kernel/rcu/tree.c | 42 ++++++++++++++-------------
> >> kernel/rcu/tree.h | 4 +--
> >> kernel/rcu/tree_plugin.h | 40 +++++++++++++------------
> >> tools/testing/selftests/rcutorture/bin/kvm.sh | 4 +--
> >> 4 files changed, 47 insertions(+), 43 deletions(-)
> >>
> >> --
> >> 2.0.0.rc2
> >>
> >
>
>
>
> --
> Pranith
>