Received: by 2002:a05:7412:98c1:b0:fa:551:50a7 with SMTP id kc1csp1370412rdb; Sun, 7 Jan 2024 17:09:27 -0800 (PST) X-Google-Smtp-Source: AGHT+IH6k3+dWy10LnPG3bKhwb+6bvW0aMWPv4+ZoEKrDlbCAEb/1u88vt9Juq8CpAdj96+33lrQ X-Received: by 2002:a05:6a00:174c:b0:6da:56cb:d8c8 with SMTP id j12-20020a056a00174c00b006da56cbd8c8mr3586694pfc.30.1704676167350; Sun, 07 Jan 2024 17:09:27 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1704676167; cv=none; d=google.com; s=arc-20160816; b=yAzfEn7SwAmN+K3poxtGjgW0NXqFfioKMtW9sYV4EC5Q3HyJ33EMEZEj71rBx99aP8 qSejqbxraAHsq3wXHXjbVGNA+1TcZiqYgrMbENtrgGAJqxtDD1u0ppJYlTpbytbkaoL2 jt3yE+PBurBAZoStjBBOSpNgS25Ke2lg5OvuOdK1OL1xZ0ZuBxlyeNS0/Im51H/A5PNt GmFQ9EhN0bZoBlOVm5Ew99QUQ9JCTLdKeUlHiZ7DHbVPP+2bRqJiA0O234Sk9pKL4iVi 84HP3KidsgACyEfzvZBoGk8SjB3GKGpyGJ9Bw8XjZEg6KXkMpK40B3ISwZ2D33uTGZxI MRlg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:references:in-reply-to:message-id :subject:cc:to:from:date:dkim-signature; bh=gumuc1EyzcJAx5DcmfxeMProzBlgUg12PTuRSijVBbA=; fh=A6sdv3mz5lxPGfTcvgsaiZqXngyXo9Y/lbW+fYRjcAE=; b=GpV3Zo+lx6edc/SEE8F2EVPd3wbr5c5i27eIy7Sjde6rH4QUk2VG3IErp8Edx/GxYt lW8obRFyvKBJ1AUGNPV3LFmKtzAVJl0rQAN3IepfccriOgK/EVi2jT4da3Q2sEKjVrX4 qNdQUptg9laV0U4EhZxySIhacZ1PuQvMHaeO5I4fV/MREtU/NYvmMDehTEFdLORAZon0 L4pTkI+10WbIp0IxtvqLamGQQmVZ9364DLk5Nqzs+gPCewOW+Ln8WR1/zQL37TUmtROG tWSGxh6qtseYvqN24uOzphLlYc2eCq2Zs5ZnzwqO0BHvnLX8d7T5SiRsfvTK3OvvkZio hUKA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=Pvmclpv7; spf=pass (google.com: domain of linux-kernel+bounces-19035-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-19035-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org. [2604:1380:45e3:2400::1]) by mx.google.com with ESMTPS id f4-20020a056a00228400b006d8ec0156ebsi5204073pfe.3.2024.01.07.17.09.26 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 07 Jan 2024 17:09:27 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-19035-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=Pvmclpv7; spf=pass (google.com: domain of linux-kernel+bounces-19035-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-19035-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sv.mirrors.kernel.org (Postfix) with ESMTPS id B806B281B1E for ; Mon, 8 Jan 2024 01:09:26 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 5E9A910F5; Mon, 8 Jan 2024 01:09:18 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="Pvmclpv7" X-Original-To: linux-kernel@vger.kernel.org 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 7F1A07E2; Mon, 8 Jan 2024 01:09:16 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 266F0C433C7; Mon, 8 Jan 2024 01:09:11 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1704676156; bh=slheypH8giDLGy7JTmdKyLyiTx2fJiDCXnZ7hL7YVpc=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=Pvmclpv7xnNZn53S3IENCpmZ7klxDcWJuGM5xD/y3/jrxzjT1oQ4MhnL0TAc/P1Ss dYw6dSoCBRN2hexogmxB3h1Uic2TCkWJbcxGf8/t+j9rKt80Sa70onhhHqEJ0H4EoH zmM1eIdCZZCGbFCAZBFFyNUc+S3Y34dQNyYNHPAMDemzV0TccBvfWsxjfMOaTmGJyZ O9A5MxVGC2BJhBlHEpL7EJ7tLFgMOdWjDKLNxV+mXfHFQE9TvscfEznGilGA1qwOoJ ifJBHcIHJDuNMRq8/9rM/wLkbjD3eqPpWDfet3PRz4iwDtChb/VQOEzg5x1HqX5ciM ZIW8U6BgIUbeA== Date: Mon, 8 Jan 2024 10:09:09 +0900 From: Masami Hiramatsu (Google) To: Mark Rutland Cc: Alexei Starovoitov , Steven Rostedt , Florent Revest , linux-trace-kernel@vger.kernel.org, LKML , Martin KaFai Lau , bpf , Sven Schnelle , Alexei Starovoitov , Jiri Olsa , Arnaldo Carvalho de Melo , Daniel Borkmann , Alan Maguire , Peter Zijlstra , Thomas Gleixner , Guo Ren Subject: Re: [PATCH v5 22/34] tracing: Rename ftrace_regs_return_value to ftrace_regs_get_return_value Message-Id: <20240108100909.e7cb718db83950328d4814a3@kernel.org> In-Reply-To: References: <170290509018.220107.1347127510564358608.stgit@devnote2> <170290535934.220107.9998902467249003656.stgit@devnote2> X-Mailer: Sylpheed 3.7.0 (GTK+ 2.24.33; x86_64-pc-linux-gnu) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit On Fri, 5 Jan 2024 17:14:06 +0000 Mark Rutland wrote: > On Mon, Dec 18, 2023 at 10:15:59PM +0900, Masami Hiramatsu (Google) wrote: > > From: Masami Hiramatsu (Google) > > > > Rename ftrace_regs_return_value to ftrace_regs_get_return_value as same as > > other ftrace_regs_get/set_* APIs. > > > > Signed-off-by: Masami Hiramatsu (Google) > > Acked-by: Mark Rutland > > Since this is a trivial cleanup, it might make sense to move this to the start > of the series, so that it can be queued even if the later parts need more work. Thanks for your Ack! and OK, let me move this to the start of this series. > > Mark. > > > --- > > Changes in v3: > > - Newly added. > > --- > > arch/loongarch/include/asm/ftrace.h | 2 +- > > arch/powerpc/include/asm/ftrace.h | 2 +- > > arch/s390/include/asm/ftrace.h | 2 +- > > arch/x86/include/asm/ftrace.h | 2 +- > > include/linux/ftrace.h | 2 +- > > kernel/trace/fgraph.c | 2 +- > > 6 files changed, 6 insertions(+), 6 deletions(-) > > > > diff --git a/arch/loongarch/include/asm/ftrace.h b/arch/loongarch/include/asm/ftrace.h > > index a11996eb5892..a9c3d0f2f941 100644 > > --- a/arch/loongarch/include/asm/ftrace.h > > +++ b/arch/loongarch/include/asm/ftrace.h > > @@ -70,7 +70,7 @@ ftrace_regs_set_instruction_pointer(struct ftrace_regs *fregs, unsigned long ip) > > regs_get_kernel_argument(&(fregs)->regs, n) > > #define ftrace_regs_get_stack_pointer(fregs) \ > > kernel_stack_pointer(&(fregs)->regs) > > -#define ftrace_regs_return_value(fregs) \ > > +#define ftrace_regs_get_return_value(fregs) \ > > regs_return_value(&(fregs)->regs) > > #define ftrace_regs_set_return_value(fregs, ret) \ > > regs_set_return_value(&(fregs)->regs, ret) > > diff --git a/arch/powerpc/include/asm/ftrace.h b/arch/powerpc/include/asm/ftrace.h > > index 9e5a39b6a311..7e138e0e3baf 100644 > > --- a/arch/powerpc/include/asm/ftrace.h > > +++ b/arch/powerpc/include/asm/ftrace.h > > @@ -69,7 +69,7 @@ ftrace_regs_get_instruction_pointer(struct ftrace_regs *fregs) > > regs_get_kernel_argument(&(fregs)->regs, n) > > #define ftrace_regs_get_stack_pointer(fregs) \ > > kernel_stack_pointer(&(fregs)->regs) > > -#define ftrace_regs_return_value(fregs) \ > > +#define ftrace_regs_get_return_value(fregs) \ > > regs_return_value(&(fregs)->regs) > > #define ftrace_regs_set_return_value(fregs, ret) \ > > regs_set_return_value(&(fregs)->regs, ret) > > diff --git a/arch/s390/include/asm/ftrace.h b/arch/s390/include/asm/ftrace.h > > index 5a82b08f03cd..01e775c98425 100644 > > --- a/arch/s390/include/asm/ftrace.h > > +++ b/arch/s390/include/asm/ftrace.h > > @@ -88,7 +88,7 @@ ftrace_regs_set_instruction_pointer(struct ftrace_regs *fregs, > > regs_get_kernel_argument(&(fregs)->regs, n) > > #define ftrace_regs_get_stack_pointer(fregs) \ > > kernel_stack_pointer(&(fregs)->regs) > > -#define ftrace_regs_return_value(fregs) \ > > +#define ftrace_regs_get_return_value(fregs) \ > > regs_return_value(&(fregs)->regs) > > #define ftrace_regs_set_return_value(fregs, ret) \ > > regs_set_return_value(&(fregs)->regs, ret) > > diff --git a/arch/x86/include/asm/ftrace.h b/arch/x86/include/asm/ftrace.h > > index 0b306c82855d..a061f8832b20 100644 > > --- a/arch/x86/include/asm/ftrace.h > > +++ b/arch/x86/include/asm/ftrace.h > > @@ -64,7 +64,7 @@ arch_ftrace_get_regs(struct ftrace_regs *fregs) > > regs_get_kernel_argument(&(fregs)->regs, n) > > #define ftrace_regs_get_stack_pointer(fregs) \ > > kernel_stack_pointer(&(fregs)->regs) > > -#define ftrace_regs_return_value(fregs) \ > > +#define ftrace_regs_get_return_value(fregs) \ > > regs_return_value(&(fregs)->regs) > > #define ftrace_regs_set_return_value(fregs, ret) \ > > regs_set_return_value(&(fregs)->regs, ret) > > diff --git a/include/linux/ftrace.h b/include/linux/ftrace.h > > index 79875a00c02b..da2a23f5a9ed 100644 > > --- a/include/linux/ftrace.h > > +++ b/include/linux/ftrace.h > > @@ -187,7 +187,7 @@ static __always_inline bool ftrace_regs_has_args(struct ftrace_regs *fregs) > > regs_get_kernel_argument(ftrace_get_regs(fregs), n) > > #define ftrace_regs_get_stack_pointer(fregs) \ > > kernel_stack_pointer(ftrace_get_regs(fregs)) > > -#define ftrace_regs_return_value(fregs) \ > > +#define ftrace_regs_get_return_value(fregs) \ > > regs_return_value(ftrace_get_regs(fregs)) > > #define ftrace_regs_set_return_value(fregs, ret) \ > > regs_set_return_value(ftrace_get_regs(fregs), ret) > > diff --git a/kernel/trace/fgraph.c b/kernel/trace/fgraph.c > > index 088432b695a6..9a60acaacc96 100644 > > --- a/kernel/trace/fgraph.c > > +++ b/kernel/trace/fgraph.c > > @@ -783,7 +783,7 @@ static void fgraph_call_retfunc(struct ftrace_regs *fregs, > > trace.rettime = trace_clock_local(); > > #ifdef CONFIG_FUNCTION_GRAPH_RETVAL > > if (fregs) > > - trace.retval = ftrace_regs_return_value(fregs); > > + trace.retval = ftrace_regs_get_return_value(fregs); > > else > > trace.retval = fgraph_ret_regs_return_value(ret_regs); > > #endif > > -- Masami Hiramatsu (Google)