From nobody Fri Dec 19 04:04:59 2025 Received: from mail.loongson.cn (mail.loongson.cn [114.242.206.163]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 249D613D26B for ; Mon, 5 Aug 2024 03:27:03 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=114.242.206.163 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1722828427; cv=none; b=VViqTm46Hyc5xsJLJNim+KGYYMIIaIJDDwxQ3D7YyaHHSfZfcFxh/gq46HjunJw3sjsJ1kUADsCmep0o1v2PG4iCe6jLZQTJMPjuiRpWu68jj2kQZksBz8tkKRpMFiKq3ZRG6VW2x2TxggCEuLSwDjTEsH2wYXZs1urZPkvyC24= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1722828427; c=relaxed/simple; bh=hl0dbmk20s8D35tLoREPfCShfffkpWB3hKqyL5SB2wM=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=gIu1Q2c7Dxx5zcVu7RhT4lgKKsJdRMyk6laGnZVM3I2FHt7ILo4yWu4C3tUyrJeces07lmS+LLZjJKAn5n2yWSjJ6KJOHOpjSsWENBBnGrJIPRNkGkgeaFFXCpB23LJNfpPL+zT4K1lfXtYTq8osoIDNSKHH4g/aWaZT1oNiQaM= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=loongson.cn; spf=pass smtp.mailfrom=loongson.cn; arc=none smtp.client-ip=114.242.206.163 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=loongson.cn Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=loongson.cn Received: from loongson.cn (unknown [113.200.148.30]) by gateway (Coremail) with SMTP id _____8Dxi+qGRrBmG9AHAA--.26907S3; Mon, 05 Aug 2024 11:27:02 +0800 (CST) Received: from linux.localdomain (unknown [113.200.148.30]) by front1 (Coremail) with SMTP id qMiowMDxkeGFRrBmgeADAA--.20473S3; Mon, 05 Aug 2024 11:27:02 +0800 (CST) From: Tiezhu Yang To: Josh Poimboeuf , Peter Zijlstra , Huacai Chen Cc: loongarch@lists.linux.dev, linux-kernel@vger.kernel.org Subject: [PATCH v2 1/3] objtool/LoongArch: Decode secondary stack instruction Date: Mon, 5 Aug 2024 11:26:58 +0800 Message-ID: <20240805032700.16038-2-yangtiezhu@loongson.cn> X-Mailer: git-send-email 2.42.0 In-Reply-To: <20240805032700.16038-1-yangtiezhu@loongson.cn> References: <20240805032700.16038-1-yangtiezhu@loongson.cn> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-CM-TRANSID: qMiowMDxkeGFRrBmgeADAA--.20473S3 X-CM-SenderInfo: p1dqw3xlh2x3gn0dqz5rrqw2lrqou0/ X-Coremail-Antispam: 1Uk129KBj93XoWxXr13XF18tw1DZw4DKr4kAFc_yoW5Cr18pa nrZ3s5Jr4rWr1SyrnrJ3y5urW3Grs3WryIqFsxG34IkrWaqr95XF1IgFy0yF1kGw4rKFyI 9r4fZw17WF1UAagCm3ZEXasCq-sJn29KB7ZKAUJUUUUr529EdanIXcx71UUUUU7KY7ZEXa sCq-sGcSsGvfJ3Ic02F40EFcxC0VAKzVAqx4xG6I80ebIjqfuFe4nvWSU5nxnvy29KBjDU 0xBIdaVrnRJUUUvEb4IE77IF4wAFF20E14v26r1j6r4UM7CY07I20VC2zVCF04k26cxKx2 IYs7xG6rWj6s0DM7CIcVAFz4kK6r1Y6r17M28lY4IEw2IIxxk0rwA2F7IY1VAKz4vEj48v e4kI8wA2z4x0Y4vE2Ix0cI8IcVAFwI0_JFI_Gr1l84ACjcxK6xIIjxv20xvEc7CjxVAFwI 0_Gr0_Cr1l84ACjcxK6I8E87Iv67AKxVW8JVWxJwA2z4x0Y4vEx4A2jsIEc7CjxVAFwI0_ Gr0_Gr1UM2kKe7AKxVWUXVWUAwAS0I0E0xvYzxvE52x082IY62kv0487Mc804VCY07AIYI kI8VC2zVCFFI0UMc02F40EFcxC0VAKzVAqx4xG6I80ewAv7VC0I7IYx2IY67AKxVWUAVWU twAv7VC2z280aVAFwI0_Jr0_Gr1lOx8S6xCaFVCjc4AY6r1j6r4UM4x0Y48IcxkI7VAKI4 8JMxAIw28IcxkI7VAKI48JMxC20s026xCaFVCjc4AY6r1j6r4UMxCIbckI1I0E14v26r1Y 6r17MI8I3I0E5I8CrVAFwI0_Jr0_Jr4lx2IqxVCjr7xvwVAFwI0_JrI_JrWlx4CE17CEb7 AF67AKxVWUAVWUtwCIc40Y0x0EwIxGrwCI42IY6xIIjxv20xvE14v26r1I6r4UMIIF0xvE 2Ix0cI8IcVCY1x0267AKxVWUJVW8JwCI42IY6xAIw20EY4v20xvaj40_Jr0_JF4lIxAIcV C2z280aVAFwI0_Jr0_Gr1lIxAIcVC2z280aVCY1x0267AKxVWUJVW8JbIYCTnIWIevJa73 UjIFyTuYvjxUc0eHDUUUU Content-Type: text/plain; charset="utf-8" After commit a0f7085f6a63 ("LoongArch: Add RANDOMIZE_KSTACK_OFFSET support"), there is a new instruction "sub.d $sp, $sp, $t0" for the secondary stack in do_syscall(), then there exists a objtool warning "do_syscall+0x11c: return with modified stack frame" and there is no handle_syscall() which is the previous frame of do_syscall() in the call trace when executing the command "echo l > /proc/sysrq-trigger". In order to solve the above issues, the first step is to decode the secondary stack instruction "sub.d $sp, $sp, $t0". Cc: stable@vger.kernel.org # 6.9+ Signed-off-by: Tiezhu Yang --- tools/arch/loongarch/include/asm/inst.h | 12 ++++++++++++ tools/objtool/arch/loongarch/decode.c | 21 +++++++++++++++++++++ 2 files changed, 33 insertions(+) diff --git a/tools/arch/loongarch/include/asm/inst.h b/tools/arch/loongarch= /include/asm/inst.h index c25b5853181d..935b596642e2 100644 --- a/tools/arch/loongarch/include/asm/inst.h +++ b/tools/arch/loongarch/include/asm/inst.h @@ -51,6 +51,10 @@ enum reg2i16_op { bgeu_op =3D 0x1b, }; =20 +enum reg3_op { + subd_op =3D 0x23, +}; + struct reg0i15_format { unsigned int immediate : 15; unsigned int opcode : 17; @@ -96,6 +100,13 @@ struct reg2i16_format { unsigned int opcode : 6; }; =20 +struct reg3_format { + unsigned int rd : 5; + unsigned int rj : 5; + unsigned int rk : 5; + unsigned int opcode : 17; +}; + union loongarch_instruction { unsigned int word; struct reg0i15_format reg0i15_format; @@ -105,6 +116,7 @@ union loongarch_instruction { struct reg2i12_format reg2i12_format; struct reg2i14_format reg2i14_format; struct reg2i16_format reg2i16_format; + struct reg3_format reg3_format; }; =20 #define LOONGARCH_INSN_SIZE sizeof(union loongarch_instruction) diff --git a/tools/objtool/arch/loongarch/decode.c b/tools/objtool/arch/loo= ngarch/decode.c index aee479d2191c..db4dd05cdb49 100644 --- a/tools/objtool/arch/loongarch/decode.c +++ b/tools/objtool/arch/loongarch/decode.c @@ -272,6 +272,25 @@ static bool decode_insn_reg2i16_fomat(union loongarch_= instruction inst, return true; } =20 +static bool decode_insn_reg3_fomat(union loongarch_instruction inst, + struct instruction *insn) +{ + switch (inst.reg3_format.opcode) { + case subd_op: + if ((inst.reg3_format.rd =3D=3D CFI_SP) && (inst.reg3_format.rj =3D=3D C= FI_SP)) { + /* + * sub.d sp,sp,t0 + * this is a rare case for the secondary stack. + */ + } + break; + default: + return false; + } + + return true; +} + int arch_decode_instruction(struct objtool_file *file, const struct sectio= n *sec, unsigned long offset, unsigned int maxlen, struct instruction *insn) @@ -303,6 +322,8 @@ int arch_decode_instruction(struct objtool_file *file, = const struct section *sec return 0; if (decode_insn_reg2i16_fomat(inst, insn)) return 0; + if (decode_insn_reg3_fomat(inst, insn)) + return 0; =20 if (inst.word =3D=3D 0) insn->type =3D INSN_NOP; --=20 2.42.0 From nobody Fri Dec 19 04:04:59 2025 Received: from mail.loongson.cn (mail.loongson.cn [114.242.206.163]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 181A1139D00 for ; Mon, 5 Aug 2024 03:27:04 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=114.242.206.163 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1722828427; cv=none; b=M2dv3oYCLFl56/J2AClDzVPZ3CSt5Bb0m6ue+RS8pmgnSFS3KtsfeeCcxk1X3DHfyDt88VSwL2nYCKzxE0KASogcISsrAjcZdqv2n+QT4uWfCRcCKfVcNXwS95g5663pa2VloiZRix4/qWOUA1BZ8SXFPLBwZGamKSG5dXFWgG8= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1722828427; c=relaxed/simple; bh=vZ3sMeD8V5KlT5OtrGoiGlkT57DiyJVyAvfiqHFT+t0=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=NHCATmQGX2CuUGeCjlBxHotssfN9rgQu5MvuC7WlO6Btrza9a5idPC0EfnG81/pya0DOaumvkU3lgpgM+2MPEmgxdBtqY2chyCy+CX5deVqlhOjvLHniX09uUn3Qmgyt/xK2nIs0Z2xoWVNvol9OJnfFDfnmpDQyNkkiI2nn9Ss= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=loongson.cn; spf=pass smtp.mailfrom=loongson.cn; arc=none smtp.client-ip=114.242.206.163 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=loongson.cn Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=loongson.cn Received: from loongson.cn (unknown [113.200.148.30]) by gateway (Coremail) with SMTP id _____8CxruuHRrBmH9AHAA--.26735S3; Mon, 05 Aug 2024 11:27:03 +0800 (CST) Received: from linux.localdomain (unknown [113.200.148.30]) by front1 (Coremail) with SMTP id qMiowMDxkeGFRrBmgeADAA--.20473S4; Mon, 05 Aug 2024 11:27:02 +0800 (CST) From: Tiezhu Yang To: Josh Poimboeuf , Peter Zijlstra , Huacai Chen Cc: loongarch@lists.linux.dev, linux-kernel@vger.kernel.org Subject: [PATCH v2 2/3] objtool: Handle secondary stack related instructions Date: Mon, 5 Aug 2024 11:26:59 +0800 Message-ID: <20240805032700.16038-3-yangtiezhu@loongson.cn> X-Mailer: git-send-email 2.42.0 In-Reply-To: <20240805032700.16038-1-yangtiezhu@loongson.cn> References: <20240805032700.16038-1-yangtiezhu@loongson.cn> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-CM-TRANSID: qMiowMDxkeGFRrBmgeADAA--.20473S4 X-CM-SenderInfo: p1dqw3xlh2x3gn0dqz5rrqw2lrqou0/ X-Coremail-Antispam: 1Uk129KBj93XoWxXr13XF18tw1DZw4DuF4rJFc_yoW7JrykpF ZrZay5GrW8Wrn3ZwnFqa1UWrZ8Aws7WrW0qFnxGw1xArWaqr93WF1Sv3WYyFnYqws5W3yI qrW3Zry2yF4j9agCm3ZEXasCq-sJn29KB7ZKAUJUUUUr529EdanIXcx71UUUUU7KY7ZEXa sCq-sGcSsGvfJ3Ic02F40EFcxC0VAKzVAqx4xG6I80ebIjqfuFe4nvWSU5nxnvy29KBjDU 0xBIdaVrnRJUUU9jb4IE77IF4wAFF20E14v26r1j6r4UM7CY07I20VC2zVCF04k26cxKx2 IYs7xG6rWj6s0DM7CIcVAFz4kK6r1Y6r17M28lY4IEw2IIxxk0rwA2F7IY1VAKz4vEj48v e4kI8wA2z4x0Y4vE2Ix0cI8IcVAFwI0_Gr0_Xr1l84ACjcxK6xIIjxv20xvEc7CjxVAFwI 0_Gr0_Cr1l84ACjcxK6I8E87Iv67AKxVW8JVWxJwA2z4x0Y4vEx4A2jsIEc7CjxVAFwI0_ Gr0_Gr1UM2kKe7AKxVWUXVWUAwAS0I0E0xvYzxvE52x082IY62kv0487Mc804VCY07AIYI kI8VC2zVCFFI0UMc02F40EFcxC0VAKzVAqx4xG6I80ewAv7VC0I7IYx2IY67AKxVWUtVWr XwAv7VC2z280aVAFwI0_Gr0_Cr1lOx8S6xCaFVCjc4AY6r1j6r4UM4x0Y48IcxkI7VAKI4 8JMxAIw28IcxkI7VAKI48JMxC20s026xCaFVCjc4AY6r1j6r4UMxCIbckI1I0E14v26r1Y 6r17MI8I3I0E5I8CrVAFwI0_Jr0_Jr4lx2IqxVCjr7xvwVAFwI0_JrI_JrWlx4CE17CEb7 AF67AKxVWUAVWUtwCIc40Y0x0EwIxGrwCI42IY6xIIjxv20xvE14v26r4j6ryUMIIF0xvE 2Ix0cI8IcVCY1x0267AKxVW8JVWxJwCI42IY6xAIw20EY4v20xvaj40_Jr0_JF4lIxAIcV C2z280aVAFwI0_Gr0_Cr1lIxAIcVC2z280aVCY1x0267AKxVW8JVW8JrUvcSsGvfC2Kfnx nUUI43ZEXa7IU8EeHDUUUUU== Content-Type: text/plain; charset="utf-8" After commit a0f7085f6a63 ("LoongArch: Add RANDOMIZE_KSTACK_OFFSET support"), there is a new instruction "sub.d $sp, $sp, $t0" for the secondary stack in do_syscall(), then there exists a objtool warning "do_syscall+0x11c: return with modified stack frame" and there is no handle_syscall() which is the previous frame of do_syscall() in the call trace when executing the command "echo l > /proc/sysrq-trigger". objdump shows something like this: 0000000000000000 : 0: 02ff8063 addi.d $sp, $sp, -32 4: 29c04076 st.d $fp, $sp, 16 8: 29c02077 st.d $s0, $sp, 8 c: 29c06061 st.d $ra, $sp, 24 10: 02c08076 addi.d $fp, $sp, 32 ... 74: 0011b063 sub.d $sp, $sp, $t0 ... a8: 4c000181 jirl $ra, $t0, 0 ... dc: 02ff82c3 addi.d $sp, $fp, -32 e0: 28c06061 ld.d $ra, $sp, 24 e4: 28c04076 ld.d $fp, $sp, 16 e8: 28c02077 ld.d $s0, $sp, 8 ec: 02c08063 addi.d $sp, $sp, 32 f0: 4c000020 jirl $zero, $ra, 0 The instruction "sub.d $sp, $sp, $t0" changes the stack bottom and the new stack size is a random value, in order to find the return address of do_syscall() which is stored in the original stack frame after executing "jirl $ra, $t0, 0", it should use fp which points to the original stack top. This is a rare case, add a member "secondary_stack" in the struct symbol as a label to avoid affecting the current normal case, then set it as true only if there exists the secondary stack instruction "sub.d $sp, $sp, $t0", at last check this label for the other special secondary stack instructions about fp to change the cfa base and cfa offset during the period of secondary stack in update_cfi_state(). Tested with the following two configs: (1) CONFIG_RANDOMIZE_KSTACK_OFFSET=3Dy && CONFIG_RANDOMIZE_KSTACK_OFFSET_DEFAULT=3Dn (2) CONFIG_RANDOMIZE_KSTACK_OFFSET=3Dy && CONFIG_RANDOMIZE_KSTACK_OFFSET_DEFAULT=3Dy Cc: stable@vger.kernel.org # 6.9+ Signed-off-by: Tiezhu Yang --- tools/objtool/arch/loongarch/decode.c | 8 +++++++- tools/objtool/check.c | 22 ++++++++++++++++++++++ tools/objtool/include/objtool/elf.h | 1 + 3 files changed, 30 insertions(+), 1 deletion(-) diff --git a/tools/objtool/arch/loongarch/decode.c b/tools/objtool/arch/loo= ngarch/decode.c index db4dd05cdb49..4085714ffd18 100644 --- a/tools/objtool/arch/loongarch/decode.c +++ b/tools/objtool/arch/loongarch/decode.c @@ -122,7 +122,7 @@ static bool decode_insn_reg2i12_fomat(union loongarch_i= nstruction inst, switch (inst.reg2i12_format.opcode) { case addid_op: if ((inst.reg2i12_format.rd =3D=3D CFI_SP) || (inst.reg2i12_format.rj = =3D=3D CFI_SP)) { - /* addi.d sp,sp,si12 or addi.d fp,sp,si12 */ + /* addi.d sp,sp,si12 or addi.d fp,sp,si12 or addi.d sp,fp,si12 */ insn->immediate =3D sign_extend64(inst.reg2i12_format.immediate, 11); ADD_OP(op) { op->src.type =3D OP_SRC_ADD; @@ -275,6 +275,8 @@ static bool decode_insn_reg2i16_fomat(union loongarch_i= nstruction inst, static bool decode_insn_reg3_fomat(union loongarch_instruction inst, struct instruction *insn) { + struct symbol *func; + switch (inst.reg3_format.opcode) { case subd_op: if ((inst.reg3_format.rd =3D=3D CFI_SP) && (inst.reg3_format.rj =3D=3D C= FI_SP)) { @@ -282,6 +284,10 @@ static bool decode_insn_reg3_fomat(union loongarch_ins= truction inst, * sub.d sp,sp,t0 * this is a rare case for the secondary stack. */ + func =3D find_func_containing(insn->sec, insn->offset); + if (!func) + return false; + func->secondary_stack =3D true; } break; default: diff --git a/tools/objtool/check.c b/tools/objtool/check.c index 01237d167223..c7b9942fee29 100644 --- a/tools/objtool/check.c +++ b/tools/objtool/check.c @@ -2993,6 +2993,28 @@ static int update_cfi_state(struct instruction *insn, break; } =20 + if (op->dest.reg =3D=3D CFI_BP && op->src.reg =3D=3D CFI_SP) { + /* addi.d fp,sp,imm for the secondary stack on LoongArch */ + if (cfa->base =3D=3D CFI_SP && cfa->offset =3D=3D op->src.offset) { + if (insn->sym->secondary_stack) { + cfa->base =3D CFI_BP; + cfa->offset =3D 0; + } + } + break; + } + + if (op->dest.reg =3D=3D CFI_SP && op->src.reg =3D=3D CFI_BP) { + /* addi.d sp,fp,imm for the secondary stack on LoongArch */ + if (cfa->base =3D=3D CFI_FP && cfa->offset =3D=3D 0) { + if (insn->sym->secondary_stack) { + cfa->base =3D CFI_SP; + cfa->offset =3D -op->src.offset; + } + } + break; + } + if (op->dest.reg =3D=3D CFI_SP && op->src.reg =3D=3D CFI_BP) { =20 /* lea disp(%rbp), %rsp */ diff --git a/tools/objtool/include/objtool/elf.h b/tools/objtool/include/ob= jtool/elf.h index 2b8a69de4db8..586916e0d441 100644 --- a/tools/objtool/include/objtool/elf.h +++ b/tools/objtool/include/objtool/elf.h @@ -68,6 +68,7 @@ struct symbol { u8 warned : 1; u8 embedded_insn : 1; u8 local_label : 1; + u8 secondary_stack : 1; struct list_head pv_target; struct reloc *relocs; }; --=20 2.42.0 From nobody Fri Dec 19 04:04:59 2025 Received: from mail.loongson.cn (mail.loongson.cn [114.242.206.163]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 34C5C13D521 for ; Mon, 5 Aug 2024 03:27:04 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=114.242.206.163 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1722828428; cv=none; b=jFwKxfW9APLhG3ryDp5Lnxkfe2QIqj3PCOHhyNEWNXB13dpyyd7OpUpMFfycFTrWuvIUPF+gi6vqYcbJ3PN9gMfFYWBvuOwUANqOTus+CO4VB35mEz9O/l+J4J5XSrTMQ4lU4OKAoO+GkFmPE9IlxVjxfsBv1Zy2MCT3NgvwJ8A= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1722828428; c=relaxed/simple; bh=dQdc4zvokUMe37PQ1p3xMiSz2t9xWE7Hfgtf9NLWvBE=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=f2aNJvAQyK9GzC2oSSOR6rd99IZa48poAqyshIlMI3mnnehVFC0alSnstg63qpoRxHZVgoXS6UnNsEjmOW7BrZUWgq92nuC24alx54mJQmKb9XxPyHHO23am8TvjfnWOyUXuPV3UuRTh+HdRZYgMW98fNWy+tl+/ZbRiFdjR0o0= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=loongson.cn; spf=pass smtp.mailfrom=loongson.cn; arc=none smtp.client-ip=114.242.206.163 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=loongson.cn Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=loongson.cn Received: from loongson.cn (unknown [113.200.148.30]) by gateway (Coremail) with SMTP id _____8Dxh+mHRrBmI9AHAA--.25346S3; Mon, 05 Aug 2024 11:27:03 +0800 (CST) Received: from linux.localdomain (unknown [113.200.148.30]) by front1 (Coremail) with SMTP id qMiowMDxkeGFRrBmgeADAA--.20473S5; Mon, 05 Aug 2024 11:27:03 +0800 (CST) From: Tiezhu Yang To: Josh Poimboeuf , Peter Zijlstra , Huacai Chen Cc: loongarch@lists.linux.dev, linux-kernel@vger.kernel.org Subject: [PATCH v2 3/3] LoongArch: Remove STACK_FRAME_NON_STANDARD(do_syscall) Date: Mon, 5 Aug 2024 11:27:00 +0800 Message-ID: <20240805032700.16038-4-yangtiezhu@loongson.cn> X-Mailer: git-send-email 2.42.0 In-Reply-To: <20240805032700.16038-1-yangtiezhu@loongson.cn> References: <20240805032700.16038-1-yangtiezhu@loongson.cn> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-CM-TRANSID: qMiowMDxkeGFRrBmgeADAA--.20473S5 X-CM-SenderInfo: p1dqw3xlh2x3gn0dqz5rrqw2lrqou0/ X-Coremail-Antispam: 1Uk129KBj9xXoW7JFykWr15JFyDtw4fWr48uFX_yoWDWrg_WF 1xKw1UWrWrtF42v3Z5t3W5uF4xZw1kJFsakFn2gry7Ka43JrZ8JFyYv345CF1q9aykWrZ5 urWkX3s0y34FyosvyTuYvTs0mTUanT9S1TB71UUUUUJqnTZGkaVYY2UrUUUUj1kv1TuYvT s0mT0YCTnIWjqI5I8CrVACY4xI64kE6c02F40Ex7xfYxn0WfASr-VFAUDa7-sFnT9fnUUI cSsGvfJTRUUUb38YFVCjjxCrM7AC8VAFwI0_Jr0_Gr1l1xkIjI8I6I8E6xAIw20EY4v20x vaj40_Wr0E3s1l1IIY67AEw4v_Jrv_JF1l8cAvFVAK0II2c7xJM28CjxkF64kEwVA0rcxS w2x7M28EF7xvwVC0I7IYx2IY67AKxVW8JVW5JwA2z4x0Y4vE2Ix0cI8IcVCY1x0267AKxV W8JVWxJwA2z4x0Y4vEx4A2jsIE14v26r4j6F4UM28EF7xvwVC2z280aVCY1x0267AKxVW8 JVW8Jr1ln4kS14v26r1Y6r17M2AIxVAIcxkEcVAq07x20xvEncxIr21l57IF6xkI12xvs2 x26I8E6xACxx1l5I8CrVACY4xI64kE6c02F40Ex7xfMcIj6xIIjxv20xvE14v26r1q6rW5 McIj6I8E87Iv67AKxVW8JVWxJwAm72CE4IkC6x0Yz7v_Jr0_Gr1lF7xvr2IYc2Ij64vIr4 1l42xK82IYc2Ij64vIr41l4I8I3I0E4IkC6x0Yz7v_Jr0_Gr1l4IxYO2xFxVAFwI0_Jrv_ JF1lx2IqxVAqx4xG67AKxVWUJVWUGwC20s026x8GjcxK67AKxVWUGVWUWwC2zVAF1VAY17 CE14v26r126r1DMIIYrxkI7VAKI48JMIIF0xvE2Ix0cI8IcVAFwI0_Gr0_Xr1lIxAIcVC0 I7IYx2IY6xkF7I0E14v26r4j6F4UMIIF0xvE42xK8VAvwI8IcIk0rVWUJVWUCwCI42IY6I 8E87Iv67AKxVW8JVWxJwCI42IY6I8E87Iv6xkF7I0E14v26r4j6r4UJbIYCTnIWIevJa73 UjIFyTuYvjxU4AhLUUUUU Content-Type: text/plain; charset="utf-8" For now, it can remove STACK_FRAME_NON_STANDARD(do_syscall) because there is no objtool warning "do_syscall+0x11c: return with modified stack frame", then there is handle_syscall() which is the previous frame of do_syscall() in the call trace when executing the command "echo l > /proc/sysrq-trigger". Fixes: a0f7085f6a63 ("LoongArch: Add RANDOMIZE_KSTACK_OFFSET support") Signed-off-by: Tiezhu Yang --- arch/loongarch/kernel/syscall.c | 4 ---- 1 file changed, 4 deletions(-) diff --git a/arch/loongarch/kernel/syscall.c b/arch/loongarch/kernel/syscal= l.c index ba5d0930a74f..168bd97540f8 100644 --- a/arch/loongarch/kernel/syscall.c +++ b/arch/loongarch/kernel/syscall.c @@ -79,7 +79,3 @@ void noinstr __no_stack_protector do_syscall(struct pt_re= gs *regs) =20 syscall_exit_to_user_mode(regs); } - -#ifdef CONFIG_RANDOMIZE_KSTACK_OFFSET -STACK_FRAME_NON_STANDARD(do_syscall); -#endif --=20 2.42.0