lkml.org 
[lkml]   [2020]   [Jul]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 1/2] rcu/tree: Add a warning if CPU being onlined did not report QS already
    Date
    Add a warning if CPU being onlined did not report QS already. This is to
    simplify the code in the CPU onlining path and also to make clear about
    where QS is reported. The act of QS reporting in CPU onlining path is
    is likely unnecessary as shown by code reading and testing with
    rcutorture's TREE03 and hotplug parameters.

    Cc: Paul E. McKenney <paulmck@kernel.org>
    Cc: Neeraj Upadhyay <neeraju@codeaurora.org>
    Suggested-by: Paul E. McKenney <paulmck@kernel.org>
    Signed-off-by: Joel Fernandes (Google) <joel@joelfernandes.org>

    ---
    kernel/rcu/tree.c | 14 +++++++++++++-
    1 file changed, 13 insertions(+), 1 deletion(-)

    diff --git a/kernel/rcu/tree.c b/kernel/rcu/tree.c
    index 65e1b5e92319..1e51962b565b 100644
    --- a/kernel/rcu/tree.c
    +++ b/kernel/rcu/tree.c
    @@ -3996,7 +3996,19 @@ void rcu_cpu_starting(unsigned int cpu)
    rcu_gpnum_ovf(rnp, rdp); /* Offline-induced counter wrap? */
    rdp->rcu_onl_gp_seq = READ_ONCE(rcu_state.gp_seq);
    rdp->rcu_onl_gp_flags = READ_ONCE(rcu_state.gp_flags);
    - if (rnp->qsmask & mask) { /* RCU waiting on incoming CPU? */
    +
    + /*
    + * Delete QS reporting from here, by June 2021, if warning does not
    + * fire. Let us make the rules for reporting QS for an offline CPUs
    + * more explicit. The CPU onlining path does not need to report QS for
    + * an offline CPU. Either the QS should have reported during CPU
    + * offlining, or during rcu_gp_init() if it detected a race with either
    + * CPU offlining or task unblocking on previously offlined CPUs. Note
    + * that the FQS loop also does not report QS for an offline CPU any
    + * longer (unless it splats due to an offline CPU blocking the GP for
    + * too long).
    + */
    + if (WARN_ON_ONCE(rnp->qsmask & mask)) { /* RCU waiting on incoming CPU? */
    rcu_disable_urgency_upon_qs(rdp);
    /* Report QS -after- changing ->qsmaskinitnext! */
    rcu_report_qs_rnp(mask, rnp, rnp->gp_seq, flags);
    --
    2.28.0.rc0.142.g3c755180ce-goog
    \
     
     \ /
      Last update: 2020-07-30 05:02    [W:4.291 / U:0.596 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site