Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1735757

Re: [tip:x86/urgent] x86/mm/32: Load a sane CR3 before cpu_init() on secondary CPUs

From Pavel Machek <pavel@ucw.cz>
Newsgroups linux.kernel
Subject Re: [tip:x86/urgent] x86/mm/32: Load a sane CR3 before cpu_init() on secondary CPUs
Date 2017-09-20 15:00 +0200
Message-ID <urMLD-38Q-17@gated-at.bofh.it> (permalink)
References <uqKiS-1NC-15@gated-at.bofh.it> <uqMul-36u-7@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


[Multipart message — attachments visible in raw view] - view raw

On Sun 2017-09-17 11:19:53, tip-bot for Andy Lutomirski wrote:
> Commit-ID:  4ba55e65f471d011d3ba2ac2022180ea0877d68e
> Gitweb:     http://git.kernel.org/tip/4ba55e65f471d011d3ba2ac2022180ea0877d68e
> Author:     Andy Lutomirski <luto@kernel.org>
> AuthorDate: Sun, 17 Sep 2017 09:03:51 -0700
> Committer:  Ingo Molnar <mingo@kernel.org>
> CommitDate: Sun, 17 Sep 2017 18:59:09 +0200
> 
> x86/mm/32: Load a sane CR3 before cpu_init() on secondary CPUs
> 
> For unknown historical reasons (i.e. Borislav doesn't recall),
> 32-bit kernels invoke cpu_init() on secondary CPUs with
> initial_page_table loaded into CR3.  Then they set
> current->active_mm to &init_mm and call enter_lazy_tlb() before
> fixing CR3.  This means that the x86 TLB code gets invoked while CR3
> is inconsistent, and, with the improved PCID sanity checks I added,
> we warn.
> 
> Fix it by loading swapper_pg_dir (i.e. init_mm.pgd) earlier.
> 
> Reported-by: Paul Menzel <pmenzel@molgen.mpg.de>
> Reported-by: Pavel Machek <pavel@ucw.cz>

4.14.0-rc1-next-20170919 does not produce the warning during bootup.

									Pavel

Tested-by: Pavel Machek <pavel@ucw.cz>


-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH 0/4] More PCID fixes Andy Lutomirski <luto@kernel.org> - 2017-09-17 18:10 +0200
  [PATCH 3/4] x86/mm/32: Move setup_clear_cpu_cap(X86_FEATURE_PCID) earlier Andy Lutomirski <luto@kernel.org> - 2017-09-17 18:10 +0200
    [tip:x86/urgent] x86/mm/32: Move  setup_clear_cpu_cap(X86_FEATURE_PCID) earlier tip-bot for Andy Lutomirski <tipbot@zytor.com> - 2017-09-17 20:30 +0200
  [PATCH 4/4] x86/mm/32: Load a sane CR3 before cpu_init() on secondary CPUs Andy Lutomirski <luto@kernel.org> - 2017-09-17 18:10 +0200
    [tip:x86/urgent] x86/mm/32: Load a sane CR3 before cpu_init() on  secondary CPUs tip-bot for Andy Lutomirski <tipbot@zytor.com> - 2017-09-17 20:30 +0200
      Re: [tip:x86/urgent] x86/mm/32: Load a sane CR3 before cpu_init() on  secondary CPUs Paul Menzel <pmenzel@molgen.mpg.de> - 2017-09-18 10:20 +0200
      Re: [tip:x86/urgent] x86/mm/32: Load a sane CR3 before cpu_init() on  secondary CPUs Pavel Machek <pavel@ucw.cz> - 2017-09-20 15:00 +0200
  [PATCH 2/4] x86/mm/64: Stop using CR3.PCID == 0 in ASID-aware code Andy Lutomirski <luto@kernel.org> - 2017-09-17 18:10 +0200
    [tip:x86/urgent] x86/mm/64: Stop using CR3.PCID == 0 in ASID-aware  code tip-bot for Andy Lutomirski <tipbot@zytor.com> - 2017-09-17 20:30 +0200
  [PATCH 1/4] x86/mm: Factor out CR3-building code Andy Lutomirski <luto@kernel.org> - 2017-09-17 18:10 +0200
    [tip:x86/urgent] x86/mm: Factor out CR3-building code tip-bot for Andy Lutomirski <tipbot@zytor.com> - 2017-09-17 20:30 +0200

csiph-web