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 64603CD37AA for ; Tue, 3 Sep 2024 20:08: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:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type: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=yzBjoP8ModVh2OhanaMnnHK44jzlGcKA6Bm5SD2D2CU=; b=K7JlztSdVI/DqYgZZI8U5uQDJy 9excwuWTqEnlFKl5lNI7zxPOHfWdmhMxu60WzENpdiPRHiHA6PTnjlkYCuEXxH/KRnYTHltClJ6jn Cqwy3XWnd9C7+ACPuDkfGXr+Ed6e0p9igbwNp+ouup5lGwoWWyNglbLkaQwpN6BYVTEQHSpeQXzaj NjvMLQqNAHqOUMPnc0V/s1dpRnPhQZqYhjxusDKivrjXSs3AQ4Lg1Vet1OmcfaDIK/wgAF2eeohKN kBzfwOeMFQn1mo63eqaodoYwrCaPbtH9dMczBjvtebYmzjnrqBnOu+8mW5RhetC6Jli31xT4hBq+z q3Qw8BhA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1slZof-00000001hWB-2bN4; Tue, 03 Sep 2024 20:07:53 +0000 Received: from dfw.source.kernel.org ([139.178.84.217]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1slZnk-00000001hNl-0Yx3 for linux-arm-kernel@lists.infradead.org; Tue, 03 Sep 2024 20:06:57 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id 71BE65C55CD; Tue, 3 Sep 2024 20:06:51 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 6E0C8C4CEC5; Tue, 3 Sep 2024 20:06:53 +0000 (UTC) Date: Tue, 3 Sep 2024 16:07:51 -0400 From: Steven Rostedt To: Mark Rutland Cc: Will Deacon , Puranjay Mohan , Catalin Marinas , "Madhavan T. Venkataraman" , Kalesh Singh , chenqiwu , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, puranjay12@gmail.com Subject: Re: [PATCH] arm64: stacktrace: fix the usage of ftrace_graph_ret_addr() Message-ID: <20240903160751.702f1f91@gandalf.local.home> In-Reply-To: References: <20240618162342.28275-1-puranjay@kernel.org> <20240619124318.GA3410@willie-the-truck> <20240624161741.2a16d904@gandalf.local.home> X-Mailer: Claws Mail 3.20.0git84 (GTK+ 2.24.33; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240903_130656_272618_F2C349BB X-CRM114-Status: GOOD ( 23.44 ) 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: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Thu, 29 Aug 2024 18:54:01 +0100 Mark Rutland wrote: > Hi Steve, > > On Mon, Jun 24, 2024 at 04:17:41PM -0400, Steven Rostedt wrote: > > On Wed, 19 Jun 2024 13:43:18 +0100 > > Will Deacon wrote: > > > > > > Catalin, Will, do you have any preference? > > > > > > I think it makes most sense if this patch travels together with > > > 29c1c24a2707 ("function_graph: Fix up ftrace_graph_ret_addr()"), so that > > > would be via Steve's tree. In which case: > > > > That makes sense to me. I'll go around pulling in all the updates to the > > arch code here (with the respective acks). > > Are you still planning to pick this up? On next-20240829 the ftrace > tests blow up (as below) due to having the core ftrace change to > ftrace_graph_ret_addr() but lacking the arm64 fix. > > Splat on next-20240829: Ah, I didn't check to see if it was sent to linux-trace-kernel, so it wasn't in patchwork, hence I forgot :-p I can add it this week. -- Steve > > [2] Basic test for tracers > ------------[ cut here ]------------ > WARNING: CPU: 0 PID: 198 at arch/arm64/kernel/stacktrace.c:112 arch_stack_walk+0x1d0/0x350 > Modules linked in: > CPU: 0 UID: 0 PID: 198 Comm: ftracetest Not tainted 6.11.0-rc5-next-20240829 #1 > Hardware name: linux,dummy-virt (DT) > pstate: 604000c5 (nZCv daIF +PAN -UAO -TCO -DIT -SSBS BTYPE=--) > pc : arch_stack_walk+0x1d0/0x350 > lr : arch_stack_walk+0x1c4/0x350 > sp : ffff800080003e10 > x29: ffff800080003e10 x28: ffff800080003fe0 x27: ffff800080003eb8 > x26: ffff000003813600 x25: ffff800080004000 x24: ffff80008002ebd8 > x23: ffff800080003ed8 x22: ffff800080023c78 x21: ffff80008002ebd8 > x20: ffff800080003f40 x19: ffff800080003f30 x18: 0000000082000000 > x17: 0000000000000000 x16: ffff800080000000 x15: 00003d0900000000 > x14: 00000000000c3500 x13: 0000000000000000 x12: 0001f4e457e39c1c > x11: ffff800082f06ca8 x10: ffff800080003f30 x9 : ffff80008002ebd8 > x8 : ffff800080003db8 x7 : 7fffffffffffffff x6 : 0000000006dd7418 > x5 : 0000000000000023 x4 : ffff800080003e88 x3 : ffff800080003fe0 > x2 : ffff80008002ebd8 x1 : 0000000000000000 x0 : ffff80008002ebd8 > Call trace: > arch_stack_walk+0x1d0/0x350 > return_address+0x40/0x80 > trace_hardirqs_on+0x8c/0x190 > handle_softirqs+0xf0/0x400 > ---[ end trace 0000000000000000 ]--- > > Mark.