Christoph Lameter wrote:
At the mininum we need to have a declaration of the ia64_timestamp function otherwise ia64 sn2 does not even compile.Signed-off-by: Christoph Lameter <clameter@xxxxxxx> Index: linux-2.6.19-rc2-mm2/include/asm-ia64/timex.h =================================================================== --- linux-2.6.19-rc2-mm2.orig/include/asm-ia64/timex.h 2006-10-13 11:25:04.000000000 -0500 +++ linux-2.6.19-rc2-mm2/include/asm-ia64/timex.h 2006-10-23 15:48:05.565981377 -0500 @@ -39,4 +39,6 @@ get_cycles (void) return ret; }+extern unsigned long long (*ia64_timestamp_clock)(void);+ #endif /* _ASM_IA64_TIMEX_H */
Fine with me. I don't see how I could have broken it, as my patch just renames ia64_printk_clock() to ia64_timetamp_clock():diff -puN arch/ia64/sn/kernel/setup.c~statistics-infrastructure-make-printk_clock-a-generic-kernel-wide-nsec-resolution arch/ia64/sn/kernel/setup.c --- a/arch/ia64/sn/kernel/setup.c~statistics-infrastructure-make-printk_clock-a-generic-kernel-wide-nsec-resolution
+++ a/arch/ia64/sn/kernel/setup.c @@ -66,7 +66,7 @@ extern unsigned long last_time_offset; extern void (*ia64_mark_idle) (int); extern void snidle(int); extern unsigned char acpi_kbd_controller_present; -extern unsigned long long (*ia64_printk_clock)(void); +extern unsigned long long (*ia64_timestamp_clock)(void); unsigned long sn_rtc_cycles_per_second; EXPORT_SYMBOL(sn_rtc_cycles_per_second); But such extern declarations in c files instead of header files looks wrong in the first place and probably contributed to the issue your patch addresses. Martin - To unsubscribe from this list: send the line "unsubscribe linux-ia64" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
![]() |