Re: [PATCH] i386: move apic init in init_IRQs

From: Eric W. Biederman
Date: Tue Oct 04 2005 - 10:49:53 EST


"Maciej W. Rozycki" <macro@xxxxxxxxxxxxxx> writes:

> On Tue, 4 Oct 2005, Eric W. Biederman wrote:
>
>> - if (enable_local_apic < 0)
>> - clear_bit(X86_FEATURE_APIC, boot_cpu_data.x86_capability);
>
> I think this should stay.

lapic_disable() already does this. I am just testing the results.

>> + if (enable_local_apic < 0) {
>> + printk(KERN_INFO "Apic disabled\n");
>
> Capitalisation. ;-)
>
> Otherwise it seems reasonable -- provided it works for you. ;-)

So what should the capitalization be? "APIC disabled\n" ?

Sorry.

Eric


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/