[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CA+i-1C31TrceZiizC_tng_cc-zcvKsfXLAZD_XDftXnp9B2Tdw@mail.gmail.com>
Date: Wed, 12 Feb 2025 14:32:59 +0100
From: Brendan Jackman <jackmanb@...gle.com>
To: Rik van Riel <riel@...riel.com>
Cc: x86@...nel.org, linux-kernel@...r.kernel.org, bp@...en8.de,
peterz@...radead.org, dave.hansen@...ux.intel.com, zhengqi.arch@...edance.com,
nadav.amit@...il.com, thomas.lendacky@....com, kernel-team@...a.com,
linux-mm@...ck.org, akpm@...ux-foundation.org, jannh@...gle.com,
mhklinux@...look.com, andrew.cooper3@...rix.com,
Manali Shukla <Manali.Shukla@....com>
Subject: Re: [PATCH v10 10/12] x86/mm: do targeted broadcast flushing from
tlbbatch code
On Tue, 11 Feb 2025 at 22:09, Rik van Riel <riel@...riel.com> wrote:
> @@ -798,6 +829,8 @@ void switch_mm_irqs_off(struct mm_struct *unused, struct mm_struct *next,
> if (IS_ENABLED(CONFIG_PROVE_LOCKING))
> WARN_ON_ONCE(!irqs_disabled());
>
> + tlbsync();
> +
Reminder to add comments or move this:
https://lore.kernel.org/linux-kernel/e8a156d4c8f5db07cf03b55fb81c75d523cac680.camel@surriel.com/
I actually sent another reply which is not yet visible in Lore
continuing to advocate for doing this in arch_start_context_switch() -
I won't die on that hill but I do think we need either that or the
commentary.
Powered by blists - more mailing lists