diff options
author | Heiko Carstens <hca@linux.ibm.com> | 2023-10-12 09:40:39 +0200 |
---|---|---|
committer | Vasily Gorbik <gor@linux.ibm.com> | 2023-10-23 18:21:22 +0200 |
commit | ae626f686351d220d5eab746dc3a19e90bb1de9a (patch) | |
tree | 17b123353d950fb313b2b061d3d511f1bd9d9e9a /arch/s390/mm | |
parent | 5c845de331d980d90c6d18028942812fa491fe69 (diff) | |
download | lwn-ae626f686351d220d5eab746dc3a19e90bb1de9a.tar.gz lwn-ae626f686351d220d5eab746dc3a19e90bb1de9a.zip |
s390/mm,fault: replace WARN_ON_ONCE() with unreachable()
do_secure_storage_access() contains a switch statements which handles
all possible return values from get_fault_type(). Therefore remove the
pointless default case error handling and replace it with unreachable().
Reviewed-by: Janosch Frank <frankja@linux.ibm.com>
Signed-off-by: Heiko Carstens <hca@linux.ibm.com>
Signed-off-by: Vasily Gorbik <gor@linux.ibm.com>
Diffstat (limited to 'arch/s390/mm')
-rw-r--r-- | arch/s390/mm/fault.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/arch/s390/mm/fault.c b/arch/s390/mm/fault.c index 4ab698988618..dd1dca10bd6b 100644 --- a/arch/s390/mm/fault.c +++ b/arch/s390/mm/fault.c @@ -618,8 +618,7 @@ void do_secure_storage_access(struct pt_regs *regs) BUG(); break; default: - do_fault_error(regs, VM_FAULT_BADMAP); - WARN_ON_ONCE(1); + unreachable(); } } NOKPROBE_SYMBOL(do_secure_storage_access); |