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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id B7A0AC433EF for ; Fri, 22 Apr 2022 17:31:09 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=PHNnWwUeI2Pw+4UNk0/01GXvdzfeVvCUjJEucv7tIqY=; b=GPdHloOlKIaqPx yt8BoM2XvzaH3ng3ffsD5us84RW9jZNYjs9K1fWV9sd1t3wXbXsakRjOobL+jTGtiNH6S++QZEt9w hq1N0ka9z0DLxxE1PmUSNrqrdQ4hGQKOsMHQqYvdBjPHh+vIXG6B1jG7KZ7g37bEVG9MkH+3S2bc7 0LWp0MRNDwvXgpVlhWA2thNPGCDEPSTxtHTn8n8H7Jhk03sSp7rvNXNJM+Z2d9BRJ8K1V7mahDyKV CosQrHO+VGvUHyELGUCwxWBV2OWHzPawTkR7AV1aEhlcsnXkWpEGsKPanjS/2bBmxbKOKHsjuMS/W LXg9xhHlkhKdXCIpThnQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nhx6a-001mcc-2Y; Fri, 22 Apr 2022 17:30:04 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nhx4h-001lh4-GX for linux-arm-kernel@lists.infradead.org; Fri, 22 Apr 2022 17:28:10 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 816BA1063; Fri, 22 Apr 2022 10:28:00 -0700 (PDT) Received: from lakrids (usa-sjc-imap-foss1.foss.arm.com [10.121.207.14]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id EF4C63F73B; Fri, 22 Apr 2022 10:27:58 -0700 (PDT) Date: Fri, 22 Apr 2022 18:27:53 +0100 From: Mark Rutland To: Steven Rostedt Cc: Wang ShaoBo , cj.chengjian@huawei.com, huawei.libin@huawei.com, xiexiuqi@huawei.com, liwei391@huawei.com, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, catalin.marinas@arm.com, will@kernel.org, zengshun.wu@outlook.com, Masami Hiramatsu Subject: Re: [RFC PATCH -next v2 3/4] arm64/ftrace: support dynamically allocated trampolines Message-ID: References: <20220316100132.244849-1-bobo.shaobowang@huawei.com> <20220316100132.244849-4-bobo.shaobowang@huawei.com> <20220421100639.03c0d123@gandalf.local.home> <20220421114201.21228eeb@gandalf.local.home> <20220421130648.56b21951@gandalf.local.home> <20220422114541.34d71ad9@gandalf.local.home> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20220422114541.34d71ad9@gandalf.local.home> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220422_102807_704035_288C1BC0 X-CRM114-Status: GOOD ( 17.94 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Fri, Apr 22, 2022 at 11:45:41AM -0400, Steven Rostedt wrote: > On Fri, 22 Apr 2022 11:12:39 +0100 > Mark Rutland wrote: > > > As an aside, I'd also love to remove the REGS/!REGs distinction, and always > > save a minimum amount of state (like ARGS, but never saving a full pt_regs), > > since on arm64 the extra state stored for the REGS case isn't useful (and we > > can't reliably capture all of the pt_regs state anyway, so bits of it are made > > up or not filled in). > > Note, the reason for the addition of REGS was a requirement of kprobes. > Because before ftrace, kprobes would be triggered at the start of a > function by a breakpoint that would load in all the regs. And for backward > compatibility, Masami wanted to make sure that kprobes coming from ftrace > had all the regs just like it had when coming from a breakpoint. > > IIUC, kprobes is the only reason we have the "regs" variant (all other use > cases could get by with the ARGS version). I see. FWIW, we don't have KPROBES_ON_FTRACE on arm64. Also, the same problems apply to KRETPROBES: the synthetic `pstate` value is bogus and we don't fill in other bits of the regs (e.g. the PMR value), so it's not a "real" pt_regs, and things like interrupts_enabled(regs) won't work correctly. In addition, as KRETPROBES only hooks function entry/exit and x9-x17 + x19-x28 are meaningless at those times, no-one's going to care what they contain anyway. The state we can correctly snapshot (and that would be useful) is the same as ARGS. It'd be nice if KRETPROBES could just use ARGS, but a standard KPROBE that traps could provide regs (since it actually gets "real" regs, and within a function the other GPRs could be important). Thanks, Mark. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel 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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id AFF4CC43217 for ; Fri, 22 Apr 2022 18:00:19 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234155AbiDVSBi (ORCPT ); Fri, 22 Apr 2022 14:01:38 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:58170 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235962AbiDVR7U (ORCPT ); Fri, 22 Apr 2022 13:59:20 -0400 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id 9F859F9573 for ; Fri, 22 Apr 2022 10:56:26 -0700 (PDT) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 816BA1063; Fri, 22 Apr 2022 10:28:00 -0700 (PDT) Received: from lakrids (usa-sjc-imap-foss1.foss.arm.com [10.121.207.14]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id EF4C63F73B; Fri, 22 Apr 2022 10:27:58 -0700 (PDT) Date: Fri, 22 Apr 2022 18:27:53 +0100 From: Mark Rutland To: Steven Rostedt Cc: Wang ShaoBo , cj.chengjian@huawei.com, huawei.libin@huawei.com, xiexiuqi@huawei.com, liwei391@huawei.com, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, catalin.marinas@arm.com, will@kernel.org, zengshun.wu@outlook.com, Masami Hiramatsu Subject: Re: [RFC PATCH -next v2 3/4] arm64/ftrace: support dynamically allocated trampolines Message-ID: References: <20220316100132.244849-1-bobo.shaobowang@huawei.com> <20220316100132.244849-4-bobo.shaobowang@huawei.com> <20220421100639.03c0d123@gandalf.local.home> <20220421114201.21228eeb@gandalf.local.home> <20220421130648.56b21951@gandalf.local.home> <20220422114541.34d71ad9@gandalf.local.home> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20220422114541.34d71ad9@gandalf.local.home> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Apr 22, 2022 at 11:45:41AM -0400, Steven Rostedt wrote: > On Fri, 22 Apr 2022 11:12:39 +0100 > Mark Rutland wrote: > > > As an aside, I'd also love to remove the REGS/!REGs distinction, and always > > save a minimum amount of state (like ARGS, but never saving a full pt_regs), > > since on arm64 the extra state stored for the REGS case isn't useful (and we > > can't reliably capture all of the pt_regs state anyway, so bits of it are made > > up or not filled in). > > Note, the reason for the addition of REGS was a requirement of kprobes. > Because before ftrace, kprobes would be triggered at the start of a > function by a breakpoint that would load in all the regs. And for backward > compatibility, Masami wanted to make sure that kprobes coming from ftrace > had all the regs just like it had when coming from a breakpoint. > > IIUC, kprobes is the only reason we have the "regs" variant (all other use > cases could get by with the ARGS version). I see. FWIW, we don't have KPROBES_ON_FTRACE on arm64. Also, the same problems apply to KRETPROBES: the synthetic `pstate` value is bogus and we don't fill in other bits of the regs (e.g. the PMR value), so it's not a "real" pt_regs, and things like interrupts_enabled(regs) won't work correctly. In addition, as KRETPROBES only hooks function entry/exit and x9-x17 + x19-x28 are meaningless at those times, no-one's going to care what they contain anyway. The state we can correctly snapshot (and that would be useful) is the same as ARGS. It'd be nice if KRETPROBES could just use ARGS, but a standard KPROBE that traps could provide regs (since it actually gets "real" regs, and within a function the other GPRs could be important). Thanks, Mark.