[PATCH] kbuild: add sys_* entries for syscalls in tags

[PATCH] kbuild: add sys_* entries for syscalls in tags

[Rabin Vincent]

Currently, it is no longer possible to use the tags file to jump to
system call function definitions with sys_foo because the definitions
are obscured by the use of the SYSCALL_DEFINE* macros.

This patch adds the appropriate option to ctags to make it see through
the macro.

Signed-off-by: Rabin Vincent <rabin@rab.in>
---
 scripts/tags.sh |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/scripts/tags.sh b/scripts/tags.sh
index fdbe78b..0d58f48 100755
--- a/scripts/tags.sh
+++ b/scripts/tags.sh
@@ -99,7 +99,8 @@ exuberant()
 	-I ____cacheline_internodealigned_in_smp                \
 	-I EXPORT_SYMBOL,EXPORT_SYMBOL_GPL                      \
 	--extra=+f --c-kinds=+px                                \
-	--regex-asm='/^ENTRY\(([^)]*)\).*/\1/'
+	--regex-asm='/^ENTRY\(([^)]*)\).*/\1/'                  \
+	--regex-c='/^SYSCALL_DEFINE[[:digit:]]?\(([^,)]*).*/sys_\1/'
 
 	all_kconfigs | xargs $1 -a                              \
 	--langdef=kconfig --language-force=kconfig              \
-- 
1.5.6.5

Re: [PATCH] kbuild: add sys_* entries for syscalls in tags

[Alexey Dobriyan]

On Sun, Jan 25, 2009 at 03:35:54PM +0530, Rabin Vincent wrote:
> Currently, it is no longer possible to use the tags file to jump to
> system call function definitions with sys_foo because the definitions
> are obscured by the use of the SYSCALL_DEFINE* macros.
> 
> This patch adds the appropriate option to ctags to make it see through
> the macro.
> 
> Signed-off-by: Rabin Vincent <rabin@rab.in>
> ---
>  scripts/tags.sh |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletions(-)
> 
> diff --git a/scripts/tags.sh b/scripts/tags.sh
> index fdbe78b..0d58f48 100755
> --- a/scripts/tags.sh
> +++ b/scripts/tags.sh
> @@ -99,7 +99,8 @@ exuberant()
>  	-I ____cacheline_internodealigned_in_smp                \
>  	-I EXPORT_SYMBOL,EXPORT_SYMBOL_GPL                      \
>  	--extra=+f --c-kinds=+px                                \
> -	--regex-asm='/^ENTRY\(([^)]*)\).*/\1/'
> +	--regex-asm='/^ENTRY\(([^)]*)\).*/\1/'                  \
> +	--regex-c='/^SYSCALL_DEFINE[[:digit:]]?\(([^,)]*).*/sys_\1/'

You should patch emacs() as well.

[PATCH v2] kbuild: add sys_* entries for syscalls in tags

[Rabin Vincent]

Currently, it is no longer possible to use the tags file to jump to
system call function definitions with sys_foo, because the definitions
are obscured by use of the SYSCALL_DEFINE* macros.

This patch adds the appropriate option to ctags to make it see through
the macro.  Also, it adds the ENTRY() work already done for Exuberant
to Emacs too.

Signed-off-by: Rabin Vincent <rabin@rab.in>
---
This version of the patch makes the modifications for emacs too.

 scripts/tags.sh |    7 +++++--
 1 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/scripts/tags.sh b/scripts/tags.sh
index fdbe78b..fe7c99d 100755
--- a/scripts/tags.sh
+++ b/scripts/tags.sh
@@ -99,7 +99,8 @@ exuberant()
 	-I ____cacheline_internodealigned_in_smp                \
 	-I EXPORT_SYMBOL,EXPORT_SYMBOL_GPL                      \
 	--extra=+f --c-kinds=+px                                \
-	--regex-asm='/^ENTRY\(([^)]*)\).*/\1/'
+	--regex-asm='/^ENTRY\(([^)]*)\).*/\1/'                  \
+	--regex-c='/^SYSCALL_DEFINE[[:digit:]]?\(([^,)]*).*/sys_\1/'
 
 	all_kconfigs | xargs $1 -a                              \
 	--langdef=kconfig --language-force=kconfig              \
@@ -117,7 +118,9 @@ exuberant()
 
 emacs()
 {
-	all_sources | xargs $1 -a
+	all_sources | xargs $1 -a                               \
+	--regex='/^ENTRY(\([^)]*\)).*/\1/'                      \
+	--regex='/^SYSCALL_DEFINE[0-9]?(\([^,)]*\).*/sys_\1/'
 
 	all_kconfigs | xargs $1 -a                              \
 	--regex='/^[ \t]*\(\(menu\)*config\)[ \t]+\([a-zA-Z0-9_]+\)/\3/'
-- 
1.5.6.5

Re: [PATCH v2] kbuild: add sys_* entries for syscalls in tags

[Sam Ravnborg]

On Sun, Jan 25, 2009 at 06:39:12PM +0530, Rabin Vincent wrote:
> Currently, it is no longer possible to use the tags file to jump to
> system call function definitions with sys_foo, because the definitions
> are obscured by use of the SYSCALL_DEFINE* macros.
> 
> This patch adds the appropriate option to ctags to make it see through
> the macro.  Also, it adds the ENTRY() work already done for Exuberant
> to Emacs too.
> 
> Signed-off-by: Rabin Vincent <rabin@rab.in>

Applied to kbuild-fixes.git.

	Sam