linux-kernel - Re: [PATCH] ctags usability fix

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20090803085853.GB6016@cr0.nay.redhat.com>
Date:	Mon, 3 Aug 2009 16:58:53 +0800
From:	Amerigo Wang <xiyou.wangcong@...il.com>
To:	Stefani Seibold <stefani@...bold.net>
Cc:	linux-kernel@...r.kernel.org,
	Andrew Morton <akpm@...ux-foundation.org>, sam@...nborg.org
Subject: Re: [PATCH] ctags usability fix

On Sat, Aug 01, 2009 at 10:13:13AM +0200, Stefani Seibold wrote:
>Hi,
>
>the tag file generated by the tags.sh script has some issue.
>
>First:
>  The identifier-list miss the 
>  DEFINE_TRACE,EXPORT_TRACEPOINT_SYMBOL,EXPORT_TRACEPOINT_SYMBOL_GPL
>  special handling, which can result in a wrong tag, not to jump to the
>  right variable definition or function implementation.
> 
>Second:
>  It makes no real sense to include function prototypes and external and 
>  forward variable declarations, because jumping to a tag will sometimes
>  go to this and not to the real definition and implementation. The information
>  about the declaration is still there at the definition and implementation 
>  place.
>
>So this patch make it lot easier to navigate through the kernel source 
>tree using vi.


Looks reasonable. Thanks.

Acked-by: WANG Cong <xiyou.wangcong@...il.com>

Sam?


>
>Greetings,
>Stefani
>
> tags.sh |    3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
>Signed-off-by: Stefani Seibold <stefani@...bold.net>
>
>--- linux-2.6.30.orig/scripts/tags.sh	2009-06-10 05:05:27.000000000 +0200
>+++ linux-2.6.30/scripts/tags.sh	2009-08-01 09:46:56.000000000 +0200
>@@ -101,7 +101,8 @@
> 	-I ____cacheline_aligned_in_smp                         \
> 	-I ____cacheline_internodealigned_in_smp                \
> 	-I EXPORT_SYMBOL,EXPORT_SYMBOL_GPL                      \
>-	--extra=+f --c-kinds=+px                                \
>+	-I DEFINE_TRACE,EXPORT_TRACEPOINT_SYMBOL,EXPORT_TRACEPOINT_SYMBOL_GPL \
>+	--extra=+f --c-kinds=-px                                \
> 	--regex-asm='/^ENTRY\(([^)]*)\).*/\1/'                  \
> 	--regex-c='/^SYSCALL_DEFINE[[:digit:]]?\(([^,)]*).*/sys_\1/'
> 
>
>--
>To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
>the body of a message to majordomo@...r.kernel.org
>More majordomo info at  http://vger.kernel.org/majordomo-info.html
>Please read the FAQ at  http://www.tux.org/lkml/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/