Received: by 2002:a05:6358:d09b:b0:dc:cd0c:909e with SMTP id jc27csp6698233rwb; Tue, 15 Nov 2022 02:22:12 -0800 (PST) X-Google-Smtp-Source: AA0mqf6svH2zeotwpIyw9vqDpQ+8mWdHSt99Kg2xLVg8VK7ZsRHhiqvHFwbdeICijl6lEXhrHWs6 X-Received: by 2002:a17:902:cf03:b0:186:9295:2012 with SMTP id i3-20020a170902cf0300b0018692952012mr3365218plg.19.1668507732188; Tue, 15 Nov 2022 02:22:12 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1668507732; cv=none; d=google.com; s=arc-20160816; b=p8DdyQ1M7OhOIuabbiTjJHpJiiGuYbvSLBOjwMsi3csPqq1UCQNb9/zK+WGFQUfmRw qXFA+0SihEskFJYqe8Y7xI3Ge6d72VcDrjCG7AKUIhi0EW5nTcz0XK3hSQzAJ857swfc zIc8jwM3+odxtyItbkDaT2n8dEYUtVSSntodRiu2Mpit1N7a2othRjDb2tlT5Vtc7Nqi 6c2fMhQJ/ZsgOngnYi0uYq/kxst5Rc/gNb8oTExJBy6zjI60z8ZS6gLQltSdrp8Nv4qX F5HVC5tIpu0Y8zweKROVpXusM5eiht07LjGQICt/3sLsXCJKIAHwvzYBTd1AYmvc6xe7 ++/w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:content-language :in-reply-to:mime-version:user-agent:date:message-id:from:references :cc:to:subject; bh=bh9TsfmdMNyiD9Ks0Xpi/OJnH5SbrHttkVOEOkY+6KE=; b=pKTAViE/DPCaIzz4XMRd6PjROsHBzRkhTNqy3T6eAJvnFRPBPbthFXGm1I7TkR/HS9 UoxBxRF9MHxOIu9JQJHQusGQDYyTfEsurSk5r7wkoKnTuFhwTJWxy5+xcxC9WOYCBUDY mtoz7Ke78++n511cpwDUAwBBBuAppStUFt6oVNzJbECPNlOS+i0VEJtz2erwm1oU6y3R 3FAKg7SP0+WmSaqM4lEc49F4fxm2uMjY0OHBdqkM3FK/rVyVxnNQEgg08iKb23DkPDQn SbMYo2TnkO+BplqsEOCDE1cv3DNN5NZ3a/drHAFM5v+6KOWzjwdvGXQA29wHtH32lSAO pDyA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id 203-20020a6301d4000000b004769752ec8csi3508145pgb.511.2022.11.15.02.21.57; Tue, 15 Nov 2022 02:22:12 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S237077AbiKOJXA (ORCPT + 88 others); Tue, 15 Nov 2022 04:23:00 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:50932 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232789AbiKOJWz (ORCPT ); Tue, 15 Nov 2022 04:22:55 -0500 Received: from loongson.cn (mail.loongson.cn [114.242.206.163]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id C6D402622 for ; Tue, 15 Nov 2022 01:22:51 -0800 (PST) Received: from loongson.cn (unknown [113.200.148.30]) by gateway (Coremail) with SMTP id _____8AxHLZqWnNjoTUHAA--.9506S3; Tue, 15 Nov 2022 17:22:50 +0800 (CST) Received: from [10.130.0.63] (unknown [113.200.148.30]) by localhost.localdomain (Coremail) with SMTP id AQAAf8DxbuBpWnNjPH0TAA--.51957S3; Tue, 15 Nov 2022 17:22:50 +0800 (CST) Subject: Re: [PATCH v6 5/9] LoongArch/ftrace: Add DYNAMIC_FTRACE_WITH_REGS support To: Huacai Chen Cc: Steven Rostedt , Ingo Molnar , loongarch@lists.linux.dev, linux-kernel@vger.kernel.org, Jinyang He References: <20221115025527.13382-1-zhangqing@loongson.cn> <20221115025527.13382-6-zhangqing@loongson.cn> From: Qing Zhang Message-ID: <1288f936-7e43-592a-8061-d9df94573c4d@loongson.cn> Date: Tue, 15 Nov 2022 17:22:49 +0800 User-Agent: Mozilla/5.0 (X11; Linux mips64; rv:68.0) Gecko/20100101 Thunderbird/68.7.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit X-CM-TRANSID: AQAAf8DxbuBpWnNjPH0TAA--.51957S3 X-CM-SenderInfo: x2kd0wptlqwqxorr0wxvrqhubq/ X-Coremail-Antispam: 1Uk129KBjvJXoWxGF15tFyDXF1rKFyDGw13twb_yoWrKw1fpr y7AFn8GFW09FnF9rWSq34kurs8Jryvg3429asrJFyfGr4qqF15ZrW8Cr1DXFyIqw4xG3yI 9F1SkryYva13X37anT9S1TB71UUUUUJqnTZGkaVYY2UrUUUUj1kv1TuYvTs0mT0YCTnIWj qI5I8CrVACY4xI64kE6c02F40Ex7xfYxn0WfASr-VFAUDa7-sFnT9fnUUIcSsGvfJTRUUU bSkYFVCjjxCrM7AC8VAFwI0_Jr0_Gr1l1xkIjI8I6I8E6xAIw20EY4v20xvaj40_Wr0E3s 1l1IIY67AEw4v_Jrv_JF1l8cAvFVAK0II2c7xJM28CjxkF64kEwVA0rcxSw2x7M28EF7xv wVC0I7IYx2IY67AKxVWUCVW8JwA2z4x0Y4vE2Ix0cI8IcVCY1x0267AKxVW8JVWxJwA2z4 x0Y4vEx4A2jsIE14v26r4UJVWxJr1l84ACjcxK6I8E87Iv6xkF7I0E14v26r4UJVWxJr1l n4kS14v26r1Y6r17M2AIxVAIcxkEcVAq07x20xvEncxIr21l57IF6xkI12xvs2x26I8E6x ACxx1l5I8CrVACY4xI64kE6c02F40Ex7xfMcIj6xIIjxv20xvE14v26r1Y6r17McIj6I8E 87Iv67AKxVWxJVW8Jr1lOx8S6xCaFVCjc4AY6r1j6r4UM4x0Y48IcVAKI48JMxk0xIA0c2 IEe2xFo4CEbIxvr21l42xK82IYc2Ij64vIr41l4I8I3I0E4IkC6x0Yz7v_Jr0_Gr1l4IxY O2xFxVAFwI0_Jrv_JF1lx2IqxVAqx4xG67AKxVWUJVWUGwC20s026x8GjcxK67AKxVWUGV WUWwC2zVAF1VAY17CE14v26r126r1DMIIYrxkI7VAKI48JMIIF0xvE2Ix0cI8IcVAFwI0_ Jr0_JF4lIxAIcVC0I7IYx2IY6xkF7I0E14v26r1j6r4UMIIF0xvE42xK8VAvwI8IcIk0rV WUJVWUCwCI42IY6I8E87Iv67AKxVWxJVW8Jr1lIxAIcVC2z280aVCY1x0267AKxVW8JVW8 JrUvcSsGvfC2KfnxnUUI43ZEXa7IU8O18PUUUUU== X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,NICE_REPLY_A, SPF_HELO_PASS,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, Huacai On 2022/11/15 下午4:41, Huacai Chen wrote: > Hi, Qing, > > Patch5 and Patch6 are small and related, maybe they can be combined > with a name "LoongArch/ftrace: Add DYNAMIC_FTRACE_WITH_{ARGS,REGS} > support" I think they should be independent due to different use scenarios. Thanks, -Qing > > Huacai > > On Tue, Nov 15, 2022 at 10:55 AM Qing Zhang wrote: >> >> This patch implements DYNAMIC_FTRACE_WITH_REGS on LoongArch, which allows >> a traced function's arguments (and some other registers) to be captured >> into a struct pt_regs, allowing these to be inspected and modified. >> >> Co-developed-by: Jinyang He >> Signed-off-by: Jinyang He >> Signed-off-by: Qing Zhang >> --- >> arch/loongarch/Kconfig | 1 + >> arch/loongarch/include/asm/ftrace.h | 3 +++ >> arch/loongarch/kernel/ftrace_dyn.c | 17 ++++++++++++++ >> arch/loongarch/kernel/mcount-dyn.S | 36 +++++++++++++++++++++++++++-- >> 4 files changed, 55 insertions(+), 2 deletions(-) >> >> diff --git a/arch/loongarch/Kconfig b/arch/loongarch/Kconfig >> index 615ce62422b8..12e3e91a68ae 100644 >> --- a/arch/loongarch/Kconfig >> +++ b/arch/loongarch/Kconfig >> @@ -90,6 +90,7 @@ config LOONGARCH >> select HAVE_DEBUG_STACKOVERFLOW >> select HAVE_DMA_CONTIGUOUS >> select HAVE_DYNAMIC_FTRACE >> + select HAVE_DYNAMIC_FTRACE_WITH_REGS >> select HAVE_EBPF_JIT >> select HAVE_EXIT_THREAD >> select HAVE_FAST_GUP >> diff --git a/arch/loongarch/include/asm/ftrace.h b/arch/loongarch/include/asm/ftrace.h >> index 76ca58767f4d..a3f974a7a5ce 100644 >> --- a/arch/loongarch/include/asm/ftrace.h >> +++ b/arch/loongarch/include/asm/ftrace.h >> @@ -28,6 +28,9 @@ struct dyn_ftrace; >> int ftrace_init_nop(struct module *mod, struct dyn_ftrace *rec); >> #define ftrace_init_nop ftrace_init_nop >> >> +#ifdef CONFIG_DYNAMIC_FTRACE_WITH_REGS >> +#define ARCH_SUPPORTS_FTRACE_OPS 1 >> +#endif >> #endif /* CONFIG_DYNAMIC_FTRACE */ >> #endif /* __ASSEMBLY__ */ >> #endif /* CONFIG_FUNCTION_TRACER */ >> diff --git a/arch/loongarch/kernel/ftrace_dyn.c b/arch/loongarch/kernel/ftrace_dyn.c >> index 3fe791b6783e..ec3d951be50c 100644 >> --- a/arch/loongarch/kernel/ftrace_dyn.c >> +++ b/arch/loongarch/kernel/ftrace_dyn.c >> @@ -99,6 +99,23 @@ int ftrace_make_nop(struct module *mod, struct dyn_ftrace *rec, >> return ftrace_modify_code(pc, old, new, true); >> } >> >> +#ifdef CONFIG_DYNAMIC_FTRACE_WITH_REGS >> +int ftrace_modify_call(struct dyn_ftrace *rec, unsigned long old_addr, >> + unsigned long addr) >> +{ >> + unsigned long pc; >> + long offset; >> + u32 old, new; >> + >> + pc = rec->ip + LOONGARCH_INSN_SIZE; >> + >> + old = larch_insn_gen_bl(pc, old_addr); >> + new = larch_insn_gen_bl(pc, addr); >> + >> + return ftrace_modify_code(pc, old, new, true); >> +} >> +#endif /* CONFIG_DYNAMIC_FTRACE_WITH_REGS */ >> + >> void arch_ftrace_update_code(int command) >> { >> command |= FTRACE_MAY_SLEEP; >> diff --git a/arch/loongarch/kernel/mcount-dyn.S b/arch/loongarch/kernel/mcount-dyn.S >> index 0c12cc108e6f..02835186b463 100644 >> --- a/arch/loongarch/kernel/mcount-dyn.S >> +++ b/arch/loongarch/kernel/mcount-dyn.S >> @@ -27,7 +27,7 @@ >> * follows the LoongArch psABI well. >> */ >> >> - .macro ftrace_regs_entry >> + .macro ftrace_regs_entry allregs=0 >> PTR_ADDI sp, sp, -PT_SIZE >> /* Save trace function ra at PT_ERA */ >> PTR_S ra, sp, PT_ERA >> @@ -43,16 +43,48 @@ >> PTR_S a7, sp, PT_R11 >> PTR_S fp, sp, PT_R22 >> >> + .if \allregs >> + PTR_S t0, sp, PT_R12 >> + PTR_S t1, sp, PT_R13 >> + PTR_S t2, sp, PT_R14 >> + PTR_S t3, sp, PT_R15 >> + PTR_S t4, sp, PT_R16 >> + PTR_S t5, sp, PT_R17 >> + PTR_S t6, sp, PT_R18 >> + PTR_S t7, sp, PT_R19 >> + PTR_S t8, sp, PT_R20 >> + PTR_S s0, sp, PT_R23 >> + PTR_S s1, sp, PT_R24 >> + PTR_S s2, sp, PT_R25 >> + PTR_S s3, sp, PT_R26 >> + PTR_S s4, sp, PT_R27 >> + PTR_S s5, sp, PT_R28 >> + PTR_S s6, sp, PT_R29 >> + PTR_S s7, sp, PT_R30 >> + PTR_S s8, sp, PT_R31 >> + PTR_S tp, sp, PT_R2 >> + /* Clear it for later use as a flag sometimes. */ >> + PTR_S zero, sp, PT_R0 >> + PTR_S $r21, sp, PT_R21 >> + .endif >> + >> PTR_ADDI t8, sp, PT_SIZE >> PTR_S t8, sp, PT_R3 >> >> .endm >> >> SYM_CODE_START(ftrace_caller) >> - ftrace_regs_entry >> + ftrace_regs_entry allregs=0 >> b ftrace_common >> SYM_CODE_END(ftrace_caller) >> >> +#ifdef CONFIG_DYNAMIC_FTRACE_WITH_REGS >> +SYM_CODE_START(ftrace_regs_caller) >> + ftrace_regs_entry allregs=1 >> + b ftrace_common >> +SYM_CODE_END(ftrace_regs_caller) >> +#endif >> + >> SYM_CODE_START(ftrace_common) >> PTR_ADDI a0, ra, -8 /* arg0: ip */ >> move a1, t0 /* arg1: parent_ip */ >> -- >> 2.36.0 >> >>