From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 9702CC54E67 for ; Wed, 20 Mar 2024 04:08:29 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 0D6446B0099; Wed, 20 Mar 2024 00:08:29 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 085416B009A; Wed, 20 Mar 2024 00:08:29 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id E8F1C6B009B; Wed, 20 Mar 2024 00:08:28 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0011.hostedemail.com [216.40.44.11]) by kanga.kvack.org (Postfix) with ESMTP id D6DA36B0099 for ; Wed, 20 Mar 2024 00:08:28 -0400 (EDT) Received: from smtpin01.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay03.hostedemail.com (Postfix) with ESMTP id 7771EA025D for ; Wed, 20 Mar 2024 04:08:28 +0000 (UTC) X-FDA: 81916085496.01.A685BA8 Received: from szxga04-in.huawei.com (szxga04-in.huawei.com [45.249.212.190]) by imf27.hostedemail.com (Postfix) with ESMTP id 6EE5F40006 for ; Wed, 20 Mar 2024 04:08:23 +0000 (UTC) Authentication-Results: imf27.hostedemail.com; dkim=none; dmarc=pass (policy=quarantine) header.from=huawei.com; spf=pass (imf27.hostedemail.com: domain of xiaojiangfeng@huawei.com designates 45.249.212.190 as permitted sender) smtp.mailfrom=xiaojiangfeng@huawei.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1710907705; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=J2KCcdmB1zLBV67Qz/LPi/LQ88LoUtxdg29+8o7YCaA=; b=jEjpHIsgWHYD/kLINLr4LrWQacRH4HsXYVGbOlOL01Mpju9hYk1P0z+Auv8oHjzbqyULsX 5qhURzphqcrU8XgAiua68GSgECt55LXlt5AebdRglH1yBiKOerrHxl9wFr806ZI6zPDxHu 30ZIzfukmeER/HmvJhFEL49AxpZZv9Y= ARC-Authentication-Results: i=1; imf27.hostedemail.com; dkim=none; dmarc=pass (policy=quarantine) header.from=huawei.com; spf=pass (imf27.hostedemail.com: domain of xiaojiangfeng@huawei.com designates 45.249.212.190 as permitted sender) smtp.mailfrom=xiaojiangfeng@huawei.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1710907705; a=rsa-sha256; cv=none; b=Ap9X1BlPNoKinspUFxAZQMN2NeNksSiQAG4rqQHYPrHXVidm4Xr8JfI554I9Qc+NHaCsam gt4x47P+kJvX0xOP8e5nwNmg2+19TP/OFk4gtalDZLUTq4JYDEmtVinJG0eUlLs58sOorM uZ5j3LBJKHaCNHzevHviIM6PD6/IXaA= Received: from mail.maildlp.com (unknown [172.19.163.44]) by szxga04-in.huawei.com (SkyGuard) with ESMTP id 4Tzw4z1lcYz1xs2C; Wed, 20 Mar 2024 12:06:27 +0800 (CST) Received: from canpemm500010.china.huawei.com (unknown [7.192.105.118]) by mail.maildlp.com (Postfix) with ESMTPS id C9E441405A2; Wed, 20 Mar 2024 12:08:19 +0800 (CST) Received: from huawei.com (10.67.189.167) by canpemm500010.china.huawei.com (7.192.105.118) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2507.35; Wed, 20 Mar 2024 12:08:19 +0800 From: Jiangfeng Xiao To: , , , , , , , CC: , , , , , , , , , , , , , , , , , Subject: [PATCH v2] ARM: unwind: improve unwinders for noreturn case Date: Wed, 20 Mar 2024 11:44:38 +0800 Message-ID: <1710906278-23851-1-git-send-email-xiaojiangfeng@huawei.com> X-Mailer: git-send-email 1.8.5.6 In-Reply-To: <1709516385-7778-1-git-send-email-xiaojiangfeng@huawei.com> References: <1709516385-7778-1-git-send-email-xiaojiangfeng@huawei.com> MIME-Version: 1.0 Content-Type: text/plain X-Originating-IP: [10.67.189.167] X-ClientProxiedBy: dggems705-chm.china.huawei.com (10.3.19.182) To canpemm500010.china.huawei.com (7.192.105.118) X-Rspamd-Queue-Id: 6EE5F40006 X-Rspam-User: X-Rspamd-Server: rspam04 X-Stat-Signature: yf7myhp3mnr6rgiaxdyrt1t8i7ud1d66 X-HE-Tag: 1710907703-503192 X-HE-Meta: U2FsdGVkX18XNKsAlGrJ7fjR9xej+BOz4GgEok9yuzV/hJFPkYv9q9QHE9uAGzBf91m9+jSqySvaVvoGS8ezgQ6omcELS9RXIBlK/3Uka8Z3rXKX0iHTuM+4yCBcrnNbOkmqwRRziTbu3TxuyKXaJ8283i6YMX7Tfadq3HQhC4PUMs0PJlRx7VY96B2mBh0paTDdcuaKKweZLMyrlJgxn5jIy4d4nGT1FLFCmItoyXTzCEKRpCRpTV1t7WDBZBpEOsYyr5cW9qqcfEsz4Br7Mk1dKqzESfO7jkvAemktpPdyNTiqDMOrzm2WNQIUlvi0QhA7WTny/WfidKJHYgUy2AtYCOcBaJbk85bIS3B3NS4i7/+m2vaZLzEoINhtIJMxtzzbfb1Xjd6FOeaKJm6OG9sSoKNH4vbJLcS7QnW+cXVPh/WDBz2xl9kdUDniZGRX2U2pwTznDG57i7k88bb6XPVi1AoFFSVlwkv1G0n38t1cSPBsWBubgZi2MAnbsvQCvLnBFg8JFIPg37hXs6j5v1cfQ1oslYfaeVflsr9MNdt5cHRBQRtg96D3mi43tk7fWtdM24DsQLoDkNtXxhk3S4u8pOTT1OYnuWZUWTX2DMUsC6+zWxqpzuAy9xMiGMh5mQbl90opjuCqyC2GOlJDlIISUvTzbxxF8r7Q3sBjLrbkSjSdOPGl0FgtUU4iXJBWIuKwbx6RZqCjaxfaMo+bTeBXicdwdgLIw0QebhxpsTZPjXi9HOU6tIh80kvolsZFWohlXIYlhWZ1RVQT4cZOw+8lZ45MXFvdpK4ZzpTOQau6d0yH6XGPz3dTObtU3cbTW5FlrhJlIJlIvTtpng3Svz/E5WW77QaDlkboAyhN3vgcndy/QFlGia+S7otunXQXZksNbgC3h8cTDyGLJIJN9SDIjg6qTcXEPEyBLTzGbhupBQmFuXFlcN+YUBGKliC9oFiybSeXurEeRAsGhct cvXdEfNh YqqgyuTmomRLQSquwn4QfNW+SAMpVyeeybS9G/w1Afi9lT9dv8KmWaaS9Ouh0dumixu0B5BVGTUVMIV1mogWkrRBjKq8FMOxLlHhAt+bXPEeYsgSUTUSZOg/oJ+LPwLFae2hvCtPGOkuxq4kVjhIyKT6b5Mcx6T1iDvfhFOAdnzSqIfpAkT3nUpluC/A0VbPzu+GtXA4vQc7Cwj0ODZDA2tc1HvXWpvyXc2CCrj81bZ4hNmZCmOs6tTLOYw== X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: This is an off-by-one bug which is common in unwinders, due to the fact that the address on the stack points to the return address rather than the call address. So, for example, when the last instruction of a function is a function call (e.g., to a noreturn function), it can cause the unwinder to incorrectly try to unwind from the function after the callee. foo: ... bl bar ... end of function and thus next function ... which results in LR pointing into the next function. Fixed this by subtracting 1 from frmae->pc in the call frame (but not exception frames) like ORC on x86 does. Refer to the unwind_next_frame function in the unwind_orc.c Suggested-by: Josh Poimboeuf Link: https://lkml.kernel.org/lkml/20240305175846.qnyiru7uaa7itqba@treble/ Signed-off-by: Jiangfeng Xiao --- ChangeLog v1->v2 - stay printk("%s...", loglvl, ...) --- arch/arm/include/asm/stacktrace.h | 4 ---- arch/arm/kernel/stacktrace.c | 2 -- arch/arm/kernel/traps.c | 4 ++-- arch/arm/kernel/unwind.c | 18 +++++++++++++++--- 4 files changed, 17 insertions(+), 11 deletions(-) diff --git a/arch/arm/include/asm/stacktrace.h b/arch/arm/include/asm/stacktrace.h index 360f0d2..07e4c16 100644 --- a/arch/arm/include/asm/stacktrace.h +++ b/arch/arm/include/asm/stacktrace.h @@ -21,9 +21,7 @@ struct stackframe { struct llist_node *kr_cur; struct task_struct *tsk; #endif -#ifdef CONFIG_UNWINDER_FRAME_POINTER bool ex_frame; -#endif }; static __always_inline @@ -37,9 +35,7 @@ void arm_get_current_stackframe(struct pt_regs *regs, struct stackframe *frame) frame->kr_cur = NULL; frame->tsk = current; #endif -#ifdef CONFIG_UNWINDER_FRAME_POINTER frame->ex_frame = in_entry_text(frame->pc); -#endif } extern int unwind_frame(struct stackframe *frame); diff --git a/arch/arm/kernel/stacktrace.c b/arch/arm/kernel/stacktrace.c index 620aa82..1abd4f9 100644 --- a/arch/arm/kernel/stacktrace.c +++ b/arch/arm/kernel/stacktrace.c @@ -154,9 +154,7 @@ static void start_stack_trace(struct stackframe *frame, struct task_struct *task frame->kr_cur = NULL; frame->tsk = task; #endif -#ifdef CONFIG_UNWINDER_FRAME_POINTER frame->ex_frame = in_entry_text(frame->pc); -#endif } void arch_stack_walk(stack_trace_consume_fn consume_entry, void *cookie, diff --git a/arch/arm/kernel/traps.c b/arch/arm/kernel/traps.c index 3bad79d..46a5b1e 100644 --- a/arch/arm/kernel/traps.c +++ b/arch/arm/kernel/traps.c @@ -84,10 +84,10 @@ void dump_backtrace_entry(unsigned long where, unsigned long from, printk("%sFunction entered at [<%08lx>] from [<%08lx>]\n", loglvl, where, from); #elif defined CONFIG_BACKTRACE_VERBOSE - printk("%s[<%08lx>] (%ps) from [<%08lx>] (%pS)\n", + printk("%s[<%08lx>] (%ps) from [<%08lx>] (%pB)\n", loglvl, where, (void *)where, from, (void *)from); #else - printk("%s %ps from %pS\n", loglvl, (void *)where, (void *)from); + printk("%s %ps from %pB\n", loglvl, (void *)where, (void *)from); #endif if (in_entry_text(from) && end <= ALIGN(frame, THREAD_SIZE)) diff --git a/arch/arm/kernel/unwind.c b/arch/arm/kernel/unwind.c index 9d21921..f2baf92 100644 --- a/arch/arm/kernel/unwind.c +++ b/arch/arm/kernel/unwind.c @@ -30,6 +30,7 @@ #include #include +#include #include #include #include @@ -416,8 +417,14 @@ int unwind_frame(struct stackframe *frame) pr_debug("%s(pc = %08lx lr = %08lx sp = %08lx)\n", __func__, frame->pc, frame->lr, frame->sp); - - idx = unwind_find_idx(frame->pc); + /* + * For a call frame (as opposed to a exception frame), when the last + * instruction of a function is a function call (e.g., to a noreturn + * function), it can cause the unwinder incorrectly try to unwind + * from the function after the callee, fixed this by subtracting 1 + * from frame->pc in the call frame like ORC on x86 does. + */ + idx = unwind_find_idx(frame->ex_frame ? frame->pc : frame->pc - 1); if (!idx) { if (frame->pc && kernel_text_address(frame->pc)) { if (in_module_plt(frame->pc) && frame->pc != frame->lr) { @@ -427,6 +434,7 @@ int unwind_frame(struct stackframe *frame) * the state of the stack or the register file */ frame->pc = frame->lr; + frame->ex_frame = in_entry_text(frame->pc); return URC_OK; } pr_warn("unwind: Index not found %08lx\n", frame->pc); @@ -454,6 +462,7 @@ int unwind_frame(struct stackframe *frame) if (frame->pc == frame->lr) return -URC_FAILURE; frame->pc = frame->lr; + frame->ex_frame = in_entry_text(frame->pc); return URC_OK; } else if ((idx->insn & 0x80000000) == 0) /* prel31 to the unwind table */ @@ -515,6 +524,7 @@ int unwind_frame(struct stackframe *frame) frame->lr = ctrl.vrs[LR]; frame->pc = ctrl.vrs[PC]; frame->lr_addr = ctrl.lr_addr; + frame->ex_frame = in_entry_text(frame->pc); return URC_OK; } @@ -544,6 +554,7 @@ void unwind_backtrace(struct pt_regs *regs, struct task_struct *tsk, */ here: frame.pc = (unsigned long)&&here; + frame.ex_frame = false; } else { /* task blocked in __switch_to */ frame.fp = thread_saved_fp(tsk); @@ -554,11 +565,12 @@ void unwind_backtrace(struct pt_regs *regs, struct task_struct *tsk, */ frame.lr = 0; frame.pc = thread_saved_pc(tsk); + frame.ex_frame = false; } while (1) { int urc; - unsigned long where = frame.pc; + unsigned long where = frame.ex_frame ? frame.pc : frame.pc - 1; urc = unwind_frame(&frame); if (urc < 0) -- 1.8.5.6