Re: [PATCH] x86/apic: Eliminate cpu_mask_to_apicid() operation

[Suresh Siddha]

On Fri, 2012-06-08 at 18:53 +0200, Alexander Gordeev wrote:
> Since there are only two locations where cpu_mask_to_apicid() is called
> from, remove the operation and use only cpu_mask_to_apicid_and() instead.
> 
> Signed-off-by: Alexander Gordeev <agordeev@xxxxxxxxxx>
> ---
>  arch/x86/include/asm/apic.h           |   33 ++++++++-------------------------
>  arch/x86/kernel/apic/apic.c           |   24 ++++++------------------
>  arch/x86/kernel/apic/apic_flat_64.c   |    2 --
>  arch/x86/kernel/apic/apic_noop.c      |    1 -
>  arch/x86/kernel/apic/apic_numachip.c  |    1 -
>  arch/x86/kernel/apic/bigsmp_32.c      |    1 -
>  arch/x86/kernel/apic/es7000_32.c      |    4 +---
>  arch/x86/kernel/apic/io_apic.c        |    3 ++-
>  arch/x86/kernel/apic/numaq_32.c       |    8 --------
>  arch/x86/kernel/apic/probe_32.c       |    1 -
>  arch/x86/kernel/apic/summit_32.c      |    3 +--
>  arch/x86/kernel/apic/x2apic_cluster.c |   17 -----------------
>  arch/x86/kernel/apic/x2apic_phys.c    |    1 -
>  arch/x86/kernel/apic/x2apic_uv_x.c    |   29 ++++++-----------------------
>  arch/x86/platform/uv/uv_irq.c         |    2 +-
>  15 files changed, 25 insertions(+), 105 deletions(-)
> 

Looks good to me.

Acked-by: Suresh Siddha <suresh.b.siddha@xxxxxxxxx>

--
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/