On 7/15/20 6:18 PM, John Paul Adrian Glaubitz wrote:
> Found the culprit:
> 
> c5b27a889da92f4a969d61df77bd4f79ffce57c9 is the first bad commit
> commit c5b27a889da92f4a969d61df77bd4f79ffce57c9
> Author: Peter Zijlstra <pet...@infradead.org>
> Date:   Tue Sep 4 14:45:04 2018 +0200
> 
>     sh/tlb: Convert SH to generic mmu_gather
>     
>     Generic mmu_gather provides everything SH needs (range tracking and
>     cache coherency).
>     
>     No change in behavior intended.
>     
>     Signed-off-by: Peter Zijlstra (Intel) <pet...@infradead.org>
>     Cc: Andrew Morton <a...@linux-foundation.org>
>     Cc: Andy Lutomirski <l...@kernel.org>
>     Cc: Aneesh Kumar K.V <aneesh.ku...@linux.vnet.ibm.com>
>     Cc: Borislav Petkov <b...@alien8.de>
>     Cc: Dave Hansen <dave.han...@linux.intel.com>
>     Cc: H. Peter Anvin <h...@zytor.com>
>     Cc: Linus Torvalds <torva...@linux-foundation.org>
>     Cc: Nick Piggin <npig...@gmail.com>
>     Cc: Peter Zijlstra <pet...@infradead.org>
>     Cc: Rich Felker <dal...@libc.org>
>     Cc: Rik van Riel <r...@surriel.com>
>     Cc: Thomas Gleixner <t...@linutronix.de>
>     Cc: Will Deacon <will.dea...@arm.com>
>     Cc: Yoshinori Sato <ys...@users.sourceforge.jp>
>     Signed-off-by: Ingo Molnar <mi...@kernel.org>
> 
>  arch/sh/include/asm/pgalloc.h |   9 +++
>  arch/sh/include/asm/tlb.h     | 130 
> +-----------------------------------------
>  2 files changed, 10 insertions(+), 129 deletions(-)
> 
> CC'ing the author (Peter Zijlstra <pet...@infradead.org>).
This is fixed by [1]:

sh/tlb: Fix PGTABLE_LEVELS > 2

Geert reported that his SH7722-based Migo-R board failed to boot after
commit:

  c5b27a889da9 ("sh/tlb: Convert SH to generic mmu_gather")

That commit fell victim to copying the wrong pattern --
__pmd_free_tlb() used to be implemented with pmd_free().

Fixes: c5b27a889da9 ("sh/tlb: Convert SH to generic mmu_gather")
Reported-by: Geert Uytterhoeven <ge...@linux-m68k.org>
Signed-off-by: Peter Zijlstra (Intel) <pet...@infradead.org>
Reviewed-by: Geert Uytterhoeven <geert+rene...@glider.be>
Tested-by: Geert Uytterhoeven <geert+rene...@glider.be>

I'm now running a current kernel on my SH-7785LCR with no issues.

Adrian

> [1] 
> https://git.kernel.org/pub/scm/linux/kernel/git/peterz/queue.git/commit/?h=mm/tlb&id=1f19a4bf8dcb42efc278940fb81c288c346b3eda

-- 
 .''`.  John Paul Adrian Glaubitz
: :' :  Debian Developer - glaub...@debian.org
`. `'   Freie Universitaet Berlin - glaub...@physik.fu-berlin.de
  `-    GPG: 62FF 8A75 84E0 2956 9546  0006 7426 3B37 F5B5 F913

Reply via email to