diff options
author | Fernando Luis Vázquez Cao <fernando@oss.ntt.co.jp> | 2006-09-26 04:52:33 -0400 |
---|---|---|
committer | Andi Kleen <andi@basil.nowhere.org> | 2006-09-26 04:52:33 -0400 |
commit | 2b94ab2fd55768030e177a6ec224dedd031ad06b (patch) | |
tree | 270ebf8207932190d2ec0b725c82dc8039b24fe7 | |
parent | 52d522f53f137c7903db22f9196a48ad8658fb2b (diff) |
[PATCH] Replace local_save_flags+local_irq_disable with
The combination of "local_save_flags" and "local_irq_disable" seems to be
equivalent to "local_irq_save" (see code snips below). Consequently, replace
occurrences of local_save_flags+local_irq_disable with local_irq_save.
* local_irq_save
#define raw_local_irq_save(flags) \
do { (flags) = __raw_local_irq_save(); } while (0)
static inline unsigned long __raw_local_irq_save(void)
{
unsigned long flags = __raw_local_save_flags();
raw_local_irq_disable();
return flags;
}
* local_save_flags
#define raw_local_save_flags(flags) \
do { (flags) = __raw_local_save_flags(); } while (0)
Signed-off-by: Fernando Vazquez <fernando@intellilink.co.jp>
Signed-off-by: Andi Kleen <ak@suse.de>
-rw-r--r-- | arch/x86_64/kernel/apic.c | 3 | ||||
-rw-r--r-- | arch/x86_64/kernel/genapic_flat.c | 3 |
2 files changed, 2 insertions, 4 deletions
diff --git a/arch/x86_64/kernel/apic.c b/arch/x86_64/kernel/apic.c index 660602c51f51..b95d1e378b96 100644 --- a/arch/x86_64/kernel/apic.c +++ b/arch/x86_64/kernel/apic.c | |||
@@ -468,8 +468,7 @@ static int lapic_suspend(struct sys_device *dev, pm_message_t state) | |||
468 | apic_pm_state.apic_tmict = apic_read(APIC_TMICT); | 468 | apic_pm_state.apic_tmict = apic_read(APIC_TMICT); |
469 | apic_pm_state.apic_tdcr = apic_read(APIC_TDCR); | 469 | apic_pm_state.apic_tdcr = apic_read(APIC_TDCR); |
470 | apic_pm_state.apic_thmr = apic_read(APIC_LVTTHMR); | 470 | apic_pm_state.apic_thmr = apic_read(APIC_LVTTHMR); |
471 | local_save_flags(flags); | 471 | local_irq_save(flags); |
472 | local_irq_disable(); | ||
473 | disable_local_APIC(); | 472 | disable_local_APIC(); |
474 | local_irq_restore(flags); | 473 | local_irq_restore(flags); |
475 | return 0; | 474 | return 0; |
diff --git a/arch/x86_64/kernel/genapic_flat.c b/arch/x86_64/kernel/genapic_flat.c index 2ceb8b872b57..50ad153eaac4 100644 --- a/arch/x86_64/kernel/genapic_flat.c +++ b/arch/x86_64/kernel/genapic_flat.c | |||
@@ -49,8 +49,7 @@ static void flat_send_IPI_mask(cpumask_t cpumask, int vector) | |||
49 | unsigned long cfg; | 49 | unsigned long cfg; |
50 | unsigned long flags; | 50 | unsigned long flags; |
51 | 51 | ||
52 | local_save_flags(flags); | 52 | local_irq_save(flags); |
53 | local_irq_disable(); | ||
54 | 53 | ||
55 | /* | 54 | /* |
56 | * Wait for idle. | 55 | * Wait for idle. |