diff options
author | Tejun Heo <tj@kernel.org> | 2010-12-09 11:47:21 +0100 |
---|---|---|
committer | Thomas Gleixner <tglx@linutronix.de> | 2010-12-10 13:46:26 +0100 |
commit | 0aa002fe602939370e9476e5ec32b562000a0425 (patch) | |
tree | 9328b22674ec6eab144e464274f9f635fef37666 /net/nonet.c | |
parent | 0e3fa13f4ee110de007bca3bf395b77997319fc8 (diff) | |
download | blackbird-op-linux-0aa002fe602939370e9476e5ec32b562000a0425.tar.gz blackbird-op-linux-0aa002fe602939370e9476e5ec32b562000a0425.zip |
x86: apic: Cleanup and simplify setup_local_APIC()
setup_local_APIC() is used to setup local APIC early during CPU
initialization and already assumes that preemption is disabled on
entry. However, The function unnecessarily disables and enables
preemption and uses smp_processor_id() multiple times in and out of
the nested preemption disabled section. This gives the wrong
impression that the function might be able to handle being called with
preemption enabled and/or migrated to another processor in the middle.
Make it clear that the function is always called with preemption
disabled, drop the confusing preemption disable block and call
smp_processor_id() once at the beginning of the function.
Signed-off-by: Tejun Heo <tj@kernel.org>
Acked-by: Cyrill Gorcunov <gorcunov@gmail.com>
Reviewed-by: Pekka Enberg <penberg@kernel.org>
Cc: Yinghai Lu <yinghai@kernel.org>
Cc: brgerst@gmail.com
LKML-Reference: <4D00B3B9.7060702@kernel.org>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Diffstat (limited to 'net/nonet.c')
0 files changed, 0 insertions, 0 deletions