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 22FD6C53219 for ; Wed, 29 Jul 2026 18:08:26 +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=NwF6WahzMHm2895PJCZr64azvFZo3rJf0ZJJcnJOyTI=; b=ggSNnJnR6AZKebDoBqGu9K3ghL FvpTMqVFUaurlrW+5KXmL7aqRbQLnIMLbqlU2JiKgvd89T5tkGLOhWOzG3SMoabcIxM1PtCWCL7yp ctOjUBHBxuuw0drnjBwqO3H87UU4559X2i33WYAE2kirV0hzYUmqVoDF4KJbtrQjfkE/E4IIJsseh y4k563ty7Vj4sWMe/E22E9aKc9FI6ZF5BaVyNKGN8AzJEJVviH47/d6jqUc/QMkmde9TIAdYKdB+P X9cUTq6INneR8W6n3fhjW8ElGSZB0HOvT82TpaYnBoNECnwwZRN8/F3hZjC185wYwr4vfVW6hSewH l3GvWbCw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1wp8hW-00000008mWY-0FvC; Wed, 29 Jul 2026 18:08:18 +0000 Received: from fhigh-a3-smtp.messagingengine.com ([103.168.172.154]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1wp8hT-00000008mVY-0frt for linux-arm-kernel@lists.infradead.org; Wed, 29 Jul 2026 18:08:16 +0000 Received: from phl-compute-09.internal (phl-compute-09.internal [10.202.2.49]) by mailfhigh.phl.internal (Postfix) with ESMTP id D8C4014002E6; Wed, 29 Jul 2026 14:08:11 -0400 (EDT) Received: from phl-frontend-03 ([10.202.2.162]) by phl-compute-09.internal (MEProxy); Wed, 29 Jul 2026 14:08:11 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=rostedt.org; h= cc:cc:content-transfer-encoding:content-type:content-type:date :date:from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:subject:subject:to:to; s=fm3; t=1785348491; x=1785434891; bh=NwF6WahzMHm2895PJCZr64azvFZo3rJf0ZJJcnJOyTI=; b= JRdh59GvEw2YJjNl4b0sb2XXDN/Wb2PCc/JW2F/OV/a7kRmZ1FITXfoI7p0tHTaa PQfszlVejq3AKOGIYMQOVDwUGQBfYl9XxtIiJpAMGDFPb19lvoltUVFNTztlks+m Om3B8fS/cHK/74JHMylRgDm0kSQ/X/s2bPIacOm35jJTKdriFb5QGfpQJrY9FMp9 loU7WNOFAV1hd+pI6vt6xPrgbEzy/j9+0r++VSFSWVmBJrcYasOnQHuDBamNUDP+ Pj9vL0+thOKtr2e3HD437RT4jjVTaIqyy65XLoJKhVDrTKwWbImbXZejxFV5TPZj FC4mTzZlGLCfuGEtRglsfA== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-transfer-encoding :content-type:content-type:date:date:feedback-id:feedback-id :from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:subject:subject:to:to:x-me-proxy :x-me-sender:x-me-sender:x-sasl-enc; s=fm2; t=1785348491; x= 1785434891; bh=NwF6WahzMHm2895PJCZr64azvFZo3rJf0ZJJcnJOyTI=; b=b +GMsB8BaZzp9S8uTtN2iW19GRLMCZGSp3VyhrRheqXMvJBLUaMENric23PgprnJD nHSxBtYmlEnhyGrwozhx+cQ/0W5yS1t6L3oP2Dld7mD+CLhv+BShND6GcBiRYBaP 8Iy2MSEp9eAT+Q8oT76faaJF7Mh87fdOBkEJ1GnPYlvTL2Bf4GQIzP3uZfrHxaTe dogakftHkBPHR29e/nmeBME5RrgL7UndR6O031XBs9L1Sf6iF+ws0WLch4hGsprZ NEYtQI5v2663sQ4i5x6upgzj0Izl+uevzcFEWIYWJ3qRAimaR4lShiX0FQf/hoRD aSINvusbC8nowItbuWp/w== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: dmFkZTERBjXApnrZrwfYcYubtd8IizaaJqU0kamfvzZdOJhvnPPUf06TI5DFPDTIcatc28 2an4NFY/41OsegM7YIizOcF3moFF5tPsTqvOEWMQ49m/J9ujLJLevMme8PQF2WUY5QmBm6 nz5aape8gfrzo7xjNg52J0NJs/H4wY2qB2dO5GSMsEmMJIUk/rcHiWJhe20Z6NwiObz3Zq WST4S9B1DwsDaGGoeQe7/o/DEIO98FM6f2Pi93sG4xJiOjE4RmWz9v5f/bX5w5JPRan3cw 07qPOtAsZ2FWnKk+GUM3y2FQotAisp7eo1YGmOF5z85KHeLwJts5U1vJy5sA5C2K8HGZOk 3Bjp6FkzfsKLiass48861fo/4E0zzIvGq6ByRJTsHEwMciPmcsNZktiMu5OhwEmv57UMXo ZKc/YcbrEZqakZat4X2VOMt1S71EUwUqpjXyCW4MoE7F5GSFUp8vdZEFBwwXM/ghCGugAU BOZyy/vjfXooose5kT6eg2PhaoNf62OTrq2Cy55Rp22J22kjvzvz/vMBQW0s8QS5+IvJ9z m7rnUc8W0NlQn7g+fUcB7JlG/CT+z6MrN7Aqy8BEP5cx0sxrtyVKUChNUFRs3CeFp0UnHO 7pl2URwtEkDfDgPeNdo14bH/g924Y+OGcIPl2hCq/P/K4kryJG6+lvlEz+OA X-ME-Proxy: Feedback-ID: id06e481b:Fastmail Received: by mail.messagingengine.com (Postfix) with ESMTPA; Wed, 29 Jul 2026 14:08:10 -0400 (EDT) Date: Wed, 29 Jul 2026 14:08:46 -0400 From: Steven Rostedt To: Masami Hiramatsu Cc: Hongyan Xia , Will Deacon , Catalin Marinas , Dennis Zhou , Tejun Heo , Christoph Lameter , Oleg Nesterov , Naveen N Rao , "David S. Miller" , Nathan Chancellor , Nick Desaulniers , Bill Wendling , Justin Stitt , Jiazi Li , Pu Hu , "linux-kernel@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-mm@kvack.org" , "linux-trace-kernel@vger.kernel.org" , "llvm@lists.linux.dev" Subject: Re: [RFC PATCH 9/9] arm64/debug: Mark debug exception helpers __always_inline Message-ID: <20260729140846.3c18f437@gandalf.local.home> In-Reply-To: References: 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.9.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260729_110815_510332_12868779 X-CRM114-Status: GOOD ( 18.08 ) 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 Masami, On Mon, 27 Jul 2026 12:25:48 +0000 Hongyan Xia wrote: > From: Hongyan Xia > > Static inline should be enough to actually inline functions for most > compilers, but my Clang-19 somehow thinks it's better to outline them. > These tiny helpers then live in normal .text sections instead of > .noinstr sections, violating noinstr. > > Mark them __always_inline so the compiler can never outline them. > > Signed-off-by: Hongyan Xia > --- > arch/arm64/include/asm/esr.h | 2 +- > arch/arm64/include/asm/percpu.h | 2 +- > arch/arm64/include/asm/preempt.h | 4 ++-- > arch/arm64/include/asm/ptrace.h | 4 ++-- > arch/arm64/kernel/debug-monitors.c | 2 +- > include/linux/kprobes.h | 8 ++++---- Are you OK with this patch? If so, can you ack it? -- Steve > 6 files changed, 11 insertions(+), 11 deletions(-) > [..] > diff --git a/include/linux/kprobes.h b/include/linux/kprobes.h > index 8c4f3bb24429..5880445ed0f0 100644 > --- a/include/linux/kprobes.h > +++ b/include/linux/kprobes.h > @@ -384,17 +384,17 @@ static inline void kprobe_ftrace_kill(void) {} > struct kprobe *get_kprobe(void *addr); > > /* kprobe_running() will just return the current_kprobe on this CPU */ > -static inline struct kprobe *kprobe_running(void) > +static __always_inline struct kprobe *kprobe_running(void) > { > return __this_cpu_read(current_kprobe); > } > > -static inline void reset_current_kprobe(void) > +static __always_inline void reset_current_kprobe(void) > { > __this_cpu_write(current_kprobe, NULL); > } > > -static inline struct kprobe_ctlblk *get_kprobe_ctlblk(void) > +static __always_inline struct kprobe_ctlblk *get_kprobe_ctlblk(void) > { > return this_cpu_ptr(&kprobe_ctlblk); > } > @@ -449,7 +449,7 @@ static inline struct kprobe *get_kprobe(void *addr) > { > return NULL; > } > -static inline struct kprobe *kprobe_running(void) > +static __always_inline struct kprobe *kprobe_running(void) > { > return NULL; > }