MIPS: Remove confusing else statement in __do_page_fault()
authorPaul Burton <paul.burton@imgtec.com>
Thu, 30 Mar 2017 21:27:02 +0000 (14:27 -0700)
committerRalf Baechle <ralf@linux-mips.org>
Wed, 12 Apr 2017 22:47:04 +0000 (00:47 +0200)
commit72f941af88a543e6a038c75b8467903b7812239c
tree610bba4faa1aafe67707498f4a1bbf60b9f1b39e
parente64889823d58305a5ddb2828ae0a988c59e87d7e
MIPS: Remove confusing else statement in __do_page_fault()

Commit 41c594ab65fc ("[MIPS] MT: Improved multithreading support.")
added an else case to an if statement in do_page_fault() (which has
since gained 2 leading underscores) for some unclear reason. If the
condition in the if statement evaluates true then we execute a goto &
branch elsewhere anyway, so the else has no effect. Combined with an #if
0 block with misleading indentation introduced in the same commit it
makes the code less clear than it could be.

Remove the unnecessary else statement & de-indent the printk within
the #if 0 block in order to make the code easier for humans to parse.

Signed-off-by: Paul Burton <paul.burton@imgtec.com>
Cc: linux-mips@linux-mips.org
Cc: trivial@kernel.org
Patchwork: https://patchwork.linux-mips.org/patch/15842/
Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
arch/mips/mm/fault.c