Commit c2294e0f authored by Laurent Dufour's avatar Laurent Dufour Committed by Michael Ellerman

powerpc/mm: Move mmap_sem unlock up from do_sigbus

Move mmap_sem releasing in the do_sigbus()'s unique caller : mm_fault_error()

No functional changes.
Reviewed-by: default avatarAneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>
Signed-off-by: default avatarLaurent Dufour <ldufour@linux.vnet.ibm.com>
Signed-off-by: default avatarMichael Ellerman <mpe@ellerman.id.au>
parent 20f13b95
...@@ -120,8 +120,6 @@ static int do_sigbus(struct pt_regs *regs, unsigned long address, ...@@ -120,8 +120,6 @@ static int do_sigbus(struct pt_regs *regs, unsigned long address,
siginfo_t info; siginfo_t info;
unsigned int lsb = 0; unsigned int lsb = 0;
up_read(&current->mm->mmap_sem);
if (!user_mode(regs)) if (!user_mode(regs))
return MM_FAULT_ERR(SIGBUS); return MM_FAULT_ERR(SIGBUS);
...@@ -185,8 +183,10 @@ static int mm_fault_error(struct pt_regs *regs, unsigned long addr, int fault) ...@@ -185,8 +183,10 @@ static int mm_fault_error(struct pt_regs *regs, unsigned long addr, int fault)
return MM_FAULT_RETURN; return MM_FAULT_RETURN;
} }
if (fault & (VM_FAULT_SIGBUS|VM_FAULT_HWPOISON|VM_FAULT_HWPOISON_LARGE)) if (fault & (VM_FAULT_SIGBUS|VM_FAULT_HWPOISON|VM_FAULT_HWPOISON_LARGE)) {
up_read(&current->mm->mmap_sem);
return do_sigbus(regs, addr, fault); return do_sigbus(regs, addr, fault);
}
/* We don't understand the fault code, this is fatal */ /* We don't understand the fault code, this is fatal */
BUG(); BUG();
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment