Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Tue, 30 Jun 2020 12:07:05 +1000
From:   Nicholas Piggin <npiggin@gmail.com>
Subject: Re: [PATCH] ring-buffer: speed up buffer resets by avoiding
 synchronize_rcu for each CPU
To:     Steven Rostedt <rostedt@goodmis.org>
Cc:     Anton Blanchard <anton@ozlabs.org>, linux-kernel@vger.kernel.org,
        Paul McKenney <paulmck@kernel.org>
References: <20200625053403.2386972-1-npiggin@gmail.com>
        <20200629181625.4b87a63a@oasis.local.home>
In-Reply-To: <20200629181625.4b87a63a@oasis.local.home>
MIME-Version: 1.0
Message-Id: <1593482725.5cl7vnne3j.astroid@bobo.none>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

Excerpts from Steven Rostedt's message of June 30, 2020 8:16 am:
> On Thu, 25 Jun 2020 15:34:03 +1000
> Nicholas Piggin <npiggin@gmail.com> wrote:
>=20
>> Batch these up so we disable all the per-cpu buffers first, then
>> synchronize_rcu() once, then reset each of the buffers. This brings
>> the time down to about 0.5s.
>=20
> After applying this patch, running tools/testing/selftests/ftracetest
> went from 5 minutes and 35 seconds to 5 minutes 5 seconds to complete
> on my 4 core (8 with hyperthreading) machine! That's almost a 10% drop!
>=20
> Thanks, I'm definitely applying this for the next merge window.

Cool, always good when a big system optimisation helps small ones as=20
well. Thanks all.

Thanks,
Nick