Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Thu, 19 May 2022 17:07:55 +0200
From:   Frederic Weisbecker <frederic@kernel.org>
To:     Nicolas Saenz Julienne <nsaenzju@redhat.com>
Cc:     LKML <linux-kernel@vger.kernel.org>,
        Peter Zijlstra <peterz@infradead.org>,
        Phil Auld <pauld@redhat.com>,
        Alex Belits <abelits@marvell.com>,
        Xiongfeng Wang <wangxiongfeng2@huawei.com>,
        Neeraj Upadhyay <quic_neeraju@quicinc.com>,
        Thomas Gleixner <tglx@linutronix.de>,
        Yu Liao <liaoyu15@huawei.com>,
        Boqun Feng <boqun.feng@gmail.com>,
        "Paul E . McKenney" <paulmck@kernel.org>,
        Marcelo Tosatti <mtosatti@redhat.com>,
        Paul Gortmaker <paul.gortmaker@windriver.com>,
        Uladzislau Rezki <uladzislau.rezki@sony.com>,
        Joel Fernandes <joel@joelfernandes.org>
Subject: Re: [PATCH 00/21] rcu/context-tracking: Merge RCU eqs-dynticks
 counter to context tracking v2
Message-ID: <20220519150755.GD618878@lothringen>
References: <20220503100051.2799723-1-frederic@kernel.org>
 <6690c4c7119839a784d0d96523e832a10e04fcca.camel@redhat.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <6690c4c7119839a784d0d96523e832a10e04fcca.camel@redhat.com>
Precedence: bulk

On Wed, May 18, 2022 at 05:33:02PM +0200, Nicolas Saenz Julienne wrote:
> On Tue, 2022-05-03 at 12:00 +0200, Frederic Weisbecker wrote:
> > This mixes up the RCU dynticks counter and the context tracking state
> > updates into a single atomic instruction. This may serve several
> > purposes:
> > 
> > 1) Improve CPU isolation with deferring some disturbances until sensitive
> >    userspace workload completes and goes to the kernel. This can take
> >    several forms, for example smp_call_function_housekeeping() or
> >    on_each_housekeeping_cpu() to enqueue and execute work on all
> >    housekeeping CPUs. Then an atomic operation on ct->state can defer
> >    the work on nohz_full CPUs until they run in kernel (or IPI them
> >    if they are in kernel mode), see this proposal by Peter:
> >    https://lore.kernel.org/all/20210929151723.162004989@infradead.org/#r
> > 
> > 2) Unearth sysidle (https://git.kernel.org/pub/scm/linux/kernel/git/paulmck/linux-rcu.git/commit/?h=sysidle.2017.05.11a&id=fe5ac724d81a3c7803e60c2232718f212f3f38d4)
> >    This feature allowed to shutdown the tick on the last housekeeping
> >    CPU once the rest of the system is fully idle. We needed some proper
> >    fully ordered context tracking for that.
> > 
> > Inspired by Peterz: https://lore.kernel.org/all/20210929151723.162004989@infradead.org
> > 
> > Passed 100 hours of TREE01.
> > 
> > ---
> 
> For the whole series:
> 
> Reviewed-by: Nicolas Saenz Julienne <nsaenzju@redhat.com>
> Tested-by: Nicolas Saenz Julienne <nsaenzju@redhat.com>

I hesitated to apply these but the series has changed a little, although
the fundamental bits remain.

I'll just wait for a confirmation from you on this v3.

Thanks!