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().

2 files changed, 2 insertions, 2 deletions

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");