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 mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 0757CC433FE for ; Thu, 21 Oct 2021 14:27:50 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id C5AA86121F for ; Thu, 21 Oct 2021 14:27:49 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org C5AA86121F Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org 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:Mime-Version:References:In-Reply-To: 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=ZlAdrJJORmfz7wQT7VEjncAwlV4dEiz8+gdQQKpqYGM=; b=QZza/wCKqZ8EhV 54Yk6AWoYyzOZlnFcbAkSvLKKrxBlhcyKk0llb+ZyAGeAEIHmh4tNNJO5Z912qETCd6XBOZcVF9t/ 5hDGC4+b0puQbOSDFo6q0DzDWPeWpljxlEtT0zRf9cCtj+V/8PbsZPYl56OLdDKmM0KRmpRZZ9eN3 ez8G0de95Fp5Prj8Dtm4IbFo203o2ghgu9eE8ecc4QxA8cYQu61WMrlOcqJDCNE0K+upaGZPzDtrU Nnkm9KC1ytZk16V7moExvew3knTyZ7LR7DHC7u4L0yZmEP1EhSja/ZGIvaoUS7SW25JYMJsf9ZbgJ 89Af9z+P0xTBO0aPZzwQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mdZ1i-007pmz-Vb; Thu, 21 Oct 2021 14:26:39 +0000 Received: from mail.kernel.org ([198.145.29.99]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1mdZ1e-007plx-CV for linux-arm-kernel@lists.infradead.org; Thu, 21 Oct 2021 14:26:35 +0000 Received: by mail.kernel.org (Postfix) with ESMTPSA id 78DDF6120F; Thu, 21 Oct 2021 14:26:32 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1634826394; bh=jfdEw2NewFPjeRp4KTG29+/+u+3mkuWccUasLnsv9Ks=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=aQz24XFq3Wy0Wc6sePt3mOOLSc1LtLZOhlrd0Hyj2e5DET1jicrrO4d7SVxRjx8RM FDQiNoFuV9oGow5hg118avECfUIBUbYlBbmNvu1fHiY4lEXfqmI0t8jznxCuFsnC4u xXqlPW7XUAe/M2B+dt4iGU1TZAScFbke4C5KnC+RFS4KSXtp9wBbnAV+KyhHl+7ZwG 822JAWeYe6mmShnlNe+3w9a9E6zM3uNUFl3UmOfQcu9iGoarOnZK7apI/XtfbETq+t /CDQTwlqCjJFFU95yfPJmaMPAsbKfhQf7I/1N0C2wThamPXxadKR4cJ4TagwsYcwdh FiOCN6flg/crQ== Date: Thu, 21 Oct 2021 23:26:30 +0900 From: Masami Hiramatsu To: Will Deacon Cc: Steven Rostedt , "Naveen N . Rao" , Ananth N Mavinakayanahalli , Ingo Molnar , linux-kernel@vger.kernel.org, Sven Schnelle , Catalin Marinas , Russell King , Nathan Chancellor , Nick Desaulniers , linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v3 6/9] arm64: Recover kretprobe modified return address in stacktrace Message-Id: <20211021232630.94bea4540670cdab5a7a63c5@kernel.org> In-Reply-To: <20211021101512.GA16485@willie-the-truck> References: <163477765570.264901.3851692300287671122.stgit@devnote2> <163477770935.264901.1772964361191833681.stgit@devnote2> <20211021101512.GA16485@willie-the-truck> X-Mailer: Sylpheed 3.7.0 (GTK+ 2.24.32; x86_64-pc-linux-gnu) Mime-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20211021_072634_509922_1C06C192 X-CRM114-Status: GOOD ( 25.75 ) 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 Thu, 21 Oct 2021 11:15:12 +0100 Will Deacon wrote: > On Thu, Oct 21, 2021 at 09:55:09AM +0900, Masami Hiramatsu wrote: > > Since the kretprobe replaces the function return address with > > the kretprobe_trampoline on the stack, stack unwinder shows it > > instead of the correct return address. > > > > This checks whether the next return address is the > > __kretprobe_trampoline(), and if so, try to find the correct > > return address from the kretprobe instance list. For this purpose > > this adds 'kr_cur' loop cursor to memorize the current kretprobe > > instance. > > > > With this fix, now arm64 can enable > > CONFIG_ARCH_CORRECT_STACKTRACE_ON_KRETPROBE, and pass the > > kprobe self tests. > > > > Signed-off-by: Masami Hiramatsu > > --- > > Changes in v2: > > - Add comment for kr_cur. > > - Make the kretprobe related code depends on CONFIG_KRETPROBES. > > - Initialize "kr_cur" directly in start_backtrace() instead > > of clearing "frame" data structure by memset(). > > --- > > arch/arm64/Kconfig | 1 + > > arch/arm64/include/asm/stacktrace.h | 4 ++++ > > arch/arm64/kernel/stacktrace.c | 7 +++++++ > > 3 files changed, 12 insertions(+) > > Acked-by: Will Deacon Thank you! > > I'm not sure how you're planning to merge this, so please let me know if > you want me to queue any of the arm64 bits. Ah, good question. Since this part depends on the first 3 patches and Steve's tracing tree, these should go through the tracing tree. Is that OK for you? (Or, wait for merging the current tracing tree and merge rest of them. but this will take a long time.) Thank you, -- Masami Hiramatsu _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel