[patch 2.6.10-rc1] Include useful absolute symbols in kallsyms

[patch 2.6.10-rc1] Include useful absolute symbols in kallsyms

[Keith Owens]

Some absolute symbols are useful, they can even appear in back traces.
Tweak kallsyms to retain the useful absolute symbols.

Signed-off-by: Keith Owens <kaos@sgi.com>

---

This list is from ia64, add absolute symbols from other architectures
as required.

Index: linux/scripts/kallsyms.c
===================================================================
--- linux.orig/scripts/kallsyms.c	Fri Oct 29 17:01:13 2004
+++ linux/scripts/kallsyms.c	Fri Oct 29 18:31:05 2004
@@ -121,7 +121,14 @@ read_symbol(FILE *in, struct sym_entry *
 		_sinittext = s->addr;
 	else if (strcmp(str, "_einittext") == 0)
 		_einittext = s->addr;
-	else if (toupper(s->type) == 'A' || toupper(s->type) == 'U')
+	else if (toupper(s->type) == 'A') {
+		/* Keep these useful absolute symbols */
+		if (strcmp(str, "__kernel_syscall_via_break") &&
+		    strcmp(str, "__kernel_syscall_via_epc") &&
+		    strcmp(str, "__kernel_sigtramp") &&
+		    strcmp(str, "__gp"))
+			return -1;
+	} else if (toupper(s->type) == 'U')
 		return -1;
 
 	/* include the type field in the symbol name, so that it gets

Re: [patch 2.6.10-rc1] Include useful absolute symbols in kallsyms

[Sam Ravnborg]

On Fri, Oct 29, 2004 at 07:37:01PM +1000, Keith Owens wrote:
> Some absolute symbols are useful, they can even appear in back traces.
> Tweak kallsyms to retain the useful absolute symbols.
> 
> Signed-off-by: Keith Owens <kaos@sgi.com>
> 
> ---
> 
> This list is from ia64, add absolute symbols from other architectures
> as required.

I've applied the following patch (clash with some arm changes):
===== scripts/kallsyms.c 1.14 vs edited =====
--- 1.14/scripts/kallsyms.c	2004-10-08 17:34:20 +02:00
+++ edited/scripts/kallsyms.c	2004-10-31 01:12:45 +02:00
@@ -132,7 +132,17 @@
 		_sinittext = s->addr;
 	else if (strcmp(str, "_einittext") == 0)
 		_einittext = s->addr;
-	else if (toupper(s->type) == 'A' || toupper(s->type) == 'U' ||
+	else if (toupper(s->type) == 'A')
+	{
+		/* Keep these useful absolute symbols */
+		if (strcmp(str, "__kernel_syscall_via_break") &&
+		    strcmp(str, "__kernel_syscall_via_epc") &&
+		    strcmp(str, "__kernel_sigtramp") &&
+		    strcmp(str, "__gp"))
+			return -1;
+
+	}
+	else if (toupper(s->type) == 'U' ||
 		 is_arm_mapping_symbol(str))
 		return -1;