2015-06-17 01:00:48

by Stephen Boyd

Subject: [PATCH] scripts/tags.sh: Include trace_*_rcuidle() in tags

Every tracepoint creates two functions, the usual one 'trace_*()'
and the rcuidle one 'trace_*_rcuidle()'. Add regex for the
rcuidle variant so that we can jump to the tracepoints that use
rcuidle.

Cc: Steven Rostedt <[email protected]>
Signed-off-by: Stephen Boyd <[email protected]>
---
scripts/tags.sh | 4 ++++
1 file changed, 4 insertions(+)

diff --git a/scripts/tags.sh b/scripts/tags.sh
index cdb491d84503..896bb303f735 100755
--- a/scripts/tags.sh
+++ b/scripts/tags.sh
@@ -170,7 +170,9 @@ exuberant()
--regex-c='/^SYSCALL_DEFINE[[:digit:]]?\(([^,)]*).*/sys_\1/' \
--regex-c='/^COMPAT_SYSCALL_DEFINE[[:digit:]]?\(([^,)]*).*/compat_sys_\1/' \
--regex-c++='/^TRACE_EVENT\(([^,)]*).*/trace_\1/' \
+ --regex-c++='/^TRACE_EVENT\(([^,)]*).*/trace_\1_rcuidle/' \
--regex-c++='/^DEFINE_EVENT\([^,)]*, *([^,)]*).*/trace_\1/' \
+ --regex-c++='/^DEFINE_EVENT\([^,)]*, *([^,)]*).*/trace_\1_rcuidle/' \
--regex-c++='/PAGEFLAG\(([^,)]*).*/Page\1/' \
--regex-c++='/PAGEFLAG\(([^,)]*).*/SetPage\1/' \
--regex-c++='/PAGEFLAG\(([^,)]*).*/ClearPage\1/' \
@@ -233,7 +235,9 @@ emacs()
--regex='/^SYSCALL_DEFINE[0-9]?(\([^,)]*\).*/sys_\1/' \
--regex='/^COMPAT_SYSCALL_DEFINE[0-9]?(\([^,)]*\).*/compat_sys_\1/' \
--regex='/^TRACE_EVENT(\([^,)]*\).*/trace_\1/' \
+ --regex='/^TRACE_EVENT(\([^,)]*\).*/trace_\1_rcuidle/' \
--regex='/^DEFINE_EVENT([^,)]*, *\([^,)]*\).*/trace_\1/' \
+ --regex='/^DEFINE_EVENT([^,)]*, *\([^,)]*\).*/trace_\1_rcuidle/' \
--regex='/PAGEFLAG(\([^,)]*\).*/Page\1/' \
--regex='/PAGEFLAG(\([^,)]*\).*/SetPage\1/' \
--regex='/PAGEFLAG(\([^,)]*\).*/ClearPage\1/' \
--
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project

2015-07-28 00:17:09

by Stephen Boyd

Subject: Re: [PATCH] scripts/tags.sh: Include trace_*_rcuidle() in tags

On 06/16/2015 06:00 PM, Stephen Boyd wrote:
> Every tracepoint creates two functions, the usual one 'trace_*()'
> and the rcuidle one 'trace_*_rcuidle()'. Add regex for the
> rcuidle variant so that we can jump to the tracepoints that use
> rcuidle.
>
> Cc: Steven Rostedt <[email protected]>
> Signed-off-by: Stephen Boyd <[email protected]>
> ---

Can this be applied?

> scripts/tags.sh | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/scripts/tags.sh b/scripts/tags.sh
> index cdb491d84503..896bb303f735 100755
> --- a/scripts/tags.sh
> +++ b/scripts/tags.sh
> @@ -170,7 +170,9 @@ exuberant()
> --regex-c='/^SYSCALL_DEFINE[[:digit:]]?\(([^,)]*).*/sys_\1/' \
> --regex-c='/^COMPAT_SYSCALL_DEFINE[[:digit:]]?\(([^,)]*).*/compat_sys_\1/' \
> --regex-c++='/^TRACE_EVENT\(([^,)]*).*/trace_\1/' \
> + --regex-c++='/^TRACE_EVENT\(([^,)]*).*/trace_\1_rcuidle/' \
> --regex-c++='/^DEFINE_EVENT\([^,)]*, *([^,)]*).*/trace_\1/' \
> + --regex-c++='/^DEFINE_EVENT\([^,)]*, *([^,)]*).*/trace_\1_rcuidle/' \
> --regex-c++='/PAGEFLAG\(([^,)]*).*/Page\1/' \
> --regex-c++='/PAGEFLAG\(([^,)]*).*/SetPage\1/' \
> --regex-c++='/PAGEFLAG\(([^,)]*).*/ClearPage\1/' \
> @@ -233,7 +235,9 @@ emacs()
> --regex='/^SYSCALL_DEFINE[0-9]?(\([^,)]*\).*/sys_\1/' \
> --regex='/^COMPAT_SYSCALL_DEFINE[0-9]?(\([^,)]*\).*/compat_sys_\1/' \
> --regex='/^TRACE_EVENT(\([^,)]*\).*/trace_\1/' \
> + --regex='/^TRACE_EVENT(\([^,)]*\).*/trace_\1_rcuidle/' \
> --regex='/^DEFINE_EVENT([^,)]*, *\([^,)]*\).*/trace_\1/' \
> + --regex='/^DEFINE_EVENT([^,)]*, *\([^,)]*\).*/trace_\1_rcuidle/' \
> --regex='/PAGEFLAG(\([^,)]*\).*/Page\1/' \
> --regex='/PAGEFLAG(\([^,)]*\).*/SetPage\1/' \
> --regex='/PAGEFLAG(\([^,)]*\).*/ClearPage\1/' \


--
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project

2015-08-19 13:33:45

by Michal Marek

Subject: Re: [PATCH] scripts/tags.sh: Include trace_*_rcuidle() in tags

On 2015-07-28 02:17, Stephen Boyd wrote:
> On 06/16/2015 06:00 PM, Stephen Boyd wrote:
>> Every tracepoint creates two functions, the usual one 'trace_*()'
>> and the rcuidle one 'trace_*_rcuidle()'. Add regex for the
>> rcuidle variant so that we can jump to the tracepoints that use
>> rcuidle.
>>
>> Cc: Steven Rostedt <[email protected]>
>> Signed-off-by: Stephen Boyd <[email protected]>
>> ---
>
> Can this be applied?

Applied to kbuild.git#misc now.

Michal