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 7F1A07E2; Mon, 8 Jan 2024 01:09:16 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="Pvmclpv7" 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-trace-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)