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;
From:   Thomas Gleixner <tglx@linutronix.de>
To:     Nicholas Piggin <npiggin@gmail.com>, linux-kernel@vger.kernel.org,
        mm-commits@vger.kernel.org, torvalds@linux-foundation.org,
        peterz@infradead.org, akpm@linux-foundation.org
Subject: Re: +
 lazy-tlb-fix-hotplug-exit-race-with-mmu_lazy_tlb_shootdown.patch added to
 mm-hotfixes-unstable branch
In-Reply-To: <CTH8ZNDRMLEZ.2WS18IS2E38T7@wheely>
References: <20230525205253.E2FAEC433EF@smtp.kernel.org>
 <87v8fv86bh.ffs@tglx> <CTH8ZNDRMLEZ.2WS18IS2E38T7@wheely>
Date:   Tue, 20 Jun 2023 08:32:31 +0200
Message-ID: <87jzvyprsw.ffs@tglx>
MIME-Version: 1.0
Content-Type: text/plain
Precedence: bulk

On Tue, Jun 20 2023 at 16:02, Nicholas Piggin wrote:
> On Sun Jun 11, 2023 at 5:29 AM AEST, Thomas Gleixner wrote:
>> /*
>>  * Invoked on the outgoing CPU in context of the CPU hotplug thread
>>  * after ensuring that there are no user space tasks left on the CPU.
>>  *
>>  * If there is a lazy mm in use on the hotplug thread, drop it and
>>  * switch to init_mm.
>>  *
>>  * The reference count on init_mm is dropped in finish_cpu().
>>  */
>> static void sched_force_init_mm(void)
>> {
>>
>> No?
>
> It could be done in many places. Peter touched it last and it's
> been in the tree since prehistoric times.

That's an argument for slapping it into some randomly chosen place and
be done with it, right?

>> > +/*
>> > + * After the CPU is offline, double check that it was previously switched to
>> > + * init_mm. This call can be removed because the condition is caught in
>> > + * finish_cpu() as well.
>>
>> So why adding it in the first place?
>>
>> The changelog mumbles something about reducing churn, but I fail to see
>> that reduction. This adds 10 lines of pointless code and comments for
>> zero value.
>
> Not sure what you're talking about. The patch didn't add it. Removing it
> requires removing it from all archs, which is the churn.

Sure. That's left as an exercise for others, right?

Oh well.

	tglx