diff options
author | Tejun Heo <[email protected]> | 2010-12-09 11:47:21 +0100 |
---|---|---|
committer | Thomas Gleixner <[email protected]> | 2010-12-10 13:46:26 +0100 |
commit | 0aa002fe602939370e9476e5ec32b562000a0425 (patch) | |
tree | 9328b22674ec6eab144e464274f9f635fef37666 /net/lapb/lapb_iface.c | |
parent | 0e3fa13f4ee110de007bca3bf395b77997319fc8 (diff) |
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 <[email protected]>
Acked-by: Cyrill Gorcunov <[email protected]>
Reviewed-by: Pekka Enberg <[email protected]>
Cc: Yinghai Lu <[email protected]>
Cc: [email protected]
LKML-Reference: <[email protected]>
Signed-off-by: Thomas Gleixner <[email protected]>
Diffstat (limited to 'net/lapb/lapb_iface.c')
0 files changed, 0 insertions, 0 deletions