From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 349BD2F24 for ; Wed, 12 Apr 2023 08:51:05 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id B1C89C433EF; Wed, 12 Apr 2023 08:51:04 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1681289465; bh=u/C0e+FNWyklVXkajKVmeQ0tBKqzZ0YudeDwsWd4CWk=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=Oh3K2VWxjm/E1rV2Spy5Of3N/evSRHjWuKfIdFBbSZ5fDisJXmgl0RO+gcs8kczZ3 eXoueT6Si38A5bY5ug9q3nTSxHFJbi1tgy1IZJDdFWu+il58VBmSRLbwJqlu2M3pjf l5L8mxlYJXx06MWi/GTt+iNIXdNkoRqYbF3be5c0= From: Greg Kroah-Hartman To: stable@vger.kernel.org Cc: Greg Kroah-Hartman , patches@lists.linux.dev, Masami Hiramatsu , Mark Rutland , John Keeping , "Steven Rostedt (Google)" Subject: [PATCH 6.2 109/173] ftrace: Mark get_lock_parent_ip() __always_inline Date: Wed, 12 Apr 2023 10:33:55 +0200 Message-Id: <20230412082842.479349866@linuxfoundation.org> X-Mailer: git-send-email 2.40.0 In-Reply-To: <20230412082838.125271466@linuxfoundation.org> References: <20230412082838.125271466@linuxfoundation.org> User-Agent: quilt/0.67 Precedence: bulk X-Mailing-List: patches@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit From: John Keeping commit ea65b41807a26495ff2a73dd8b1bab2751940887 upstream. If the compiler decides not to inline this function then preemption tracing will always show an IP inside the preemption disabling path and never the function actually calling preempt_{enable,disable}. Link: https://lore.kernel.org/linux-trace-kernel/20230327173647.1690849-1-john@metanate.com Cc: Masami Hiramatsu Cc: Mark Rutland Cc: stable@vger.kernel.org Fixes: f904f58263e1d ("sched/debug: Fix preempt_disable_ip recording for preempt_disable()") Signed-off-by: John Keeping Signed-off-by: Steven Rostedt (Google) Signed-off-by: Greg Kroah-Hartman --- include/linux/ftrace.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- a/include/linux/ftrace.h +++ b/include/linux/ftrace.h @@ -970,7 +970,7 @@ static inline void __ftrace_enabled_rest #define CALLER_ADDR5 ((unsigned long)ftrace_return_address(5)) #define CALLER_ADDR6 ((unsigned long)ftrace_return_address(6)) -static inline unsigned long get_lock_parent_ip(void) +static __always_inline unsigned long get_lock_parent_ip(void) { unsigned long addr = CALLER_ADDR0;