x86: I/O APIC: Always report how the timer has been set up
authorMaciej W. Rozycki <macro@linux-mips.org>
Mon, 14 Jul 2008 18:08:13 +0000 (19:08 +0100)
committerIngo Molnar <mingo@elte.hu>
Fri, 18 Jul 2008 12:27:47 +0000 (14:27 +0200)
commit49a66a0bcea8737e82035a31ad0d8cd64d027a5d
treefbb10b4d77ce97b8f328e243955d75053c3f3bcc
parentbaa1318841d4bc95d783e6c15219b264720002c8
x86: I/O APIC: Always report how the timer has been set up

Following recent (and less so) issues with the 8254 timer when routed
through the I/O or local APIC, always report which configurations have
been tried and which one has been set up eventually.  This is so that logs
posted by people for some other reason can be used as a cross-reference
when investigating any possible future problems.

The change unifies messages printed on 32-bit and 64-bit platforms and
adds trailing newlines (removes leading ones), so that proper log level
annotation can be used and any possible interspersed output will not cause
a mess.

I have chosen to use apic_printk(APIC_QUIET, ...) rather than printk(...)
so that the distinction of these messages is maintained making possible
future decisions about changes in this area easier.  A change posted
separately making apic_verbosity unsigned removes any extra code that
would otherwise be generated as a result of this design decision.

Signed-off-by: Maciej W. Rozycki <macro@linux-mips.org>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
arch/x86/kernel/io_apic_32.c
arch/x86/kernel/io_apic_64.c