From 124ca8a5a98a5e64e6a221b5f30c48361dac6f24 Mon Sep 17 00:00:00 2001 From: Brian Behlendorf Date: Wed, 20 May 2009 15:30:13 -0700 Subject: SLES10 Fixes (part 7) - Initial SLES testing uncovered a long standing bug in the debug tracing. The tcd_for_each() macro expected a NULL to terminate the trace_data[i] array but this was only ever true due to luck. All trace_data[] iterators are now properly capped by TCD_TYPE_MAX. - SPLAT_MAJOR 229 conflicted with a 'hvc' device on my SLES system. Since this was always an arbitrary choice I picked something else. - The HAVE_PGDAT_LIST case should set pgdat_list_addr to the value stored at the address of the memory location returned by kallsyms_lookup_name(). --- module/spl/spl-debug.c | 2 +- module/spl/spl-kmem.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) (limited to 'module/spl') diff --git a/module/spl/spl-debug.c b/module/spl/spl-debug.c index d2b96b0c3..a60740519 100644 --- a/module/spl/spl-debug.c +++ b/module/spl/spl-debug.c @@ -1261,7 +1261,7 @@ trace_fini(void) } } - for (i = 0; trace_data[i] != NULL; i++) { + for (i = 0; i < TCD_TYPE_MAX && trace_data[i] != NULL; i++) { kfree(trace_data[i]); trace_data[i] = NULL; } diff --git a/module/spl/spl-kmem.c b/module/spl/spl-kmem.c index 4009aa3b2..4edd3d7e1 100644 --- a/module/spl/spl-kmem.c +++ b/module/spl/spl-kmem.c @@ -1847,7 +1847,7 @@ spl_kmem_init_kallsyms_lookup(void) #else /* HAVE_PGDAT_HELPERS */ # ifndef HAVE_PGDAT_LIST - pgdat_list_addr = (struct pglist_data *) + pgdat_list_addr = *(struct pglist_data **) spl_kallsyms_lookup_name("pgdat_list"); if (!pgdat_list_addr) { printk(KERN_ERR "Error: Unknown symbol pgdat_list\n"); -- cgit v1.2.3