diff options
| author | Catalin Marinas <catalin.marinas@arm.com> | 2012-10-24 16:34:02 +0100 |
|---|---|---|
| committer | Zach Pfeffer <zach.pfeffer@linaro.org> | 2012-10-24 10:52:35 -0500 |
| commit | d0162d1a6d5289ab7c2e405137aa4907ffb3b74f (patch) | |
| tree | cddec8c8a018e5e399a69861001ec3236de3b79e | |
| parent | 5d956e6184c98da7d121ba702be0f0311a2119ec (diff) | |
| download | linux-aarch64-linaro-aarch64+linux-2.6-armdroid-d0162d1a6d5289ab7c2e405137aa4907ffb3b74f.tar.gz | |
arm64: Make the user fault reporting more specificlinaro-aarch64+linux-2.6-armdroid-d0162d1a6d5289ab7c2e405137aa4907ffb3b74f
For user space faults the kernel reports "unhandled page fault" and it
gives the ESR value. With this patch the error message looked up in the
fault info array to give a better description.
Signed-off-by: Catalin Marinas <catalin.marinas@arm.com>
| -rw-r--r-- | arch/arm64/mm/fault.c | 13 |
1 files changed, 11 insertions, 2 deletions
diff --git a/arch/arm64/mm/fault.c b/arch/arm64/mm/fault.c index 1909a69983c..afadae6682e 100644 --- a/arch/arm64/mm/fault.c +++ b/arch/arm64/mm/fault.c @@ -36,6 +36,8 @@ #include <asm/pgtable.h> #include <asm/tlbflush.h> +static const char *fault_name(unsigned int esr); + /* * Dump out the page tables associated with 'addr' in mm 'mm'. */ @@ -112,8 +114,9 @@ static void __do_user_fault(struct task_struct *tsk, unsigned long addr, struct siginfo si; if (show_unhandled_signals) { - pr_info("%s[%d]: unhandled page fault (%d) at 0x%08lx, code 0x%03x\n", - tsk->comm, task_pid_nr(tsk), sig, addr, esr); + pr_info("%s[%d]: unhandled %s (%d) at 0x%08lx, esr 0x%03x\n", + tsk->comm, task_pid_nr(tsk), fault_name(esr), sig, + addr, esr); show_pte(tsk->mm, addr); show_regs(regs); } @@ -450,6 +453,12 @@ static struct fault_info { { do_bad, SIGBUS, 0, "unknown 63" }, }; +static const char *fault_name(unsigned int esr) +{ + const struct fault_info *inf = fault_info + (esr & 63); + return inf->name; +} + /* * Dispatch a data abort to the relevant handler. */ |