On Wed, Jan 31, 2018 at 08:03:10AM -0800, Andy Lutomirski wrote: > flush_tlb_single() and flush_tlb_one() sound almost identical, but > they really mean "flush one user translation" and "flush one kernel > translation". Rename them to flush_tlb_one_user() and > flush_tlb_one_kernel() to make the semantics more obvious. > > Cc: Peter Zijlstra <[email protected]> > Cc: Dave Hansen <[email protected]> > Cc: Borislav Petkov <[email protected]> > Cc: Kees Cook <[email protected]> > Cc: Hugh Dickins <[email protected]> > Cc: Brian Gerst <[email protected]> > Cc: Josh Poimboeuf <[email protected]> > Cc: Rik van Riel <[email protected]> > Cc: Boris Ostrovsky <[email protected]> > Cc: Juergen Gross <[email protected]> > Cc: Eduardo Valentin <[email protected]> > Cc: Will Deacon <[email protected]> > Cc: Linux-MM <[email protected]> > Signed-off-by: Andy Lutomirski <[email protected]> > --- > > I was looking at some PTI-related code, and the flush-one-address code > is unnecessarily hard to understand because the names of the helpers are > uninformative. This came up during PTI review, but no one got around to > doing it.
Right, got as far as making it consistent and putting a comment on :-) Acked-by: Peter Zijlstra (Intel) <[email protected]>
