Re: [rfc -tip 3/4] x86,apic: fix missed handling of discrete apics

From: Cyrill Gorcunov
Date: Wed Aug 26 2009 - 02:22:30 EST


On 8/26/09, Ingo Molnar <mingo@xxxxxxx> wrote:
>
> * Cyrill Gorcunov <gorcunov@xxxxxxxxxx> wrote:
>
>> + if (!cpu_has_apic && (!smp_found_config || disable_apic))
>> + if (!cpu_has_apic && (!smp_found_config || disable_apic))
>> + if (cpu_has_apic || (smp_found_config && !disable_apic))
>
> these are now repeating patterns, so that should be moved into a
> helper inline or so.
>
> Ingo
>
Absolutely yes. Though i didn't invent more or less convenient name
for this helper. lapic_from_mp perhaps. Have to think. Thanks for
review, Ingo!
--
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/