Re: Linux 2.6.24-rc5 x86 architecture no longer Oopses...

From: Ingo Molnar
Date: Fri Dec 21 2007 - 05:30:53 EST


* Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> wrote:

> > > raw_local_save_flags() doesn't disable interrupts?
> >
> > argh. Indeed! (I wanted us to fix that misleading name eons ago, to
>
> The naming of those functions is truly awful and it goes back to year 0.
>
> > *_save_flags_only(), but some stupid bikeshed painting discussion
> > prevented it from ever happening.)
>
> local_read_flags().
>
> > that should have been raw_local_irq_save(flags)!
>
> So raw_local_save_flags() and raw_local_irq_save() have different semantics.
>
> omigawd, what have we done?

i really tried to get this fixed years ago - but did not think of
local_read_flags as a solution. The patch below fixes this.

(if it's too burdensome to carry right now then i can regenerate this
patch in 30 seconds in the 2.6.25 merge window - but please remind me
then. Perhaps put in a patch into your queue that adds a comment to
remind me :-) )

Ingo

-------------->
Subject: rename: local_save_flags => local_read_flags
From: Ingo Molnar <mingo@xxxxxxx>

The bug in commit c0a698b744 could have been avoided if
local_save_flags and local_irq_save wasnt so similarly
named. The syntax is similar as well.

rename all local_save_flags variants to local_read_flags,
as suggested by Andrew Morton.

Signed-off-by: Ingo Molnar <mingo@xxxxxxx>
---
Documentation/cli-sti-removal.txt | 4 ++--
arch/cris/arch-v10/mm/fault.c | 2 +-
arch/cris/arch-v10/mm/tlb.c | 2 +-
arch/cris/arch-v32/kernel/time.c | 2 +-
arch/cris/arch-v32/kernel/traps.c | 2 +-
arch/powerpc/kernel/traps.c | 2 +-
arch/x86/kernel/apm_32.c | 2 +-
arch/x86/kernel/io_apic_32.c | 2 +-
arch/x86/kernel/io_apic_64.c | 2 +-
arch/x86/kernel/setup64.c | 2 +-
arch/xtensa/mm/tlb.c | 6 +++---
drivers/block/ataflop.c | 2 +-
drivers/scsi/arm/acornscsi.c | 2 +-
drivers/scsi/atari_NCR5380.c | 2 +-
drivers/scsi/sun3_NCR5380.c | 2 +-
drivers/serial/68328serial.c | 2 +-
drivers/serial/crisv10.c | 2 +-
drivers/serial/mcfserial.c | 2 +-
include/asm-alpha/system.h | 2 +-
include/asm-arm/irqflags.h | 4 ++--
include/asm-avr32/irqflags.h | 10 +++++-----
include/asm-blackfin/system.h | 4 ++--
include/asm-cris/arch-v10/system.h | 4 ++--
include/asm-cris/arch-v32/system.h | 4 ++--
include/asm-frv/system.h | 8 ++++----
include/asm-h8300/system.h | 2 +-
include/asm-ia64/system.h | 6 +++---
include/asm-m32r/system.h | 4 ++--
include/asm-m68k/system.h | 6 +++---
include/asm-m68knommu/system.h | 6 +++---
include/asm-mips/irqflags.h | 6 +++---
include/asm-parisc/system.h | 4 ++--
include/asm-powerpc/hw_irq.h | 4 ++--
include/asm-powerpc/irqflags.h | 2 +-
include/asm-powerpc/spinlock.h | 2 +-
include/asm-s390/irqflags.h | 2 +-
include/asm-sh/irqflags.h | 8 ++++----
include/asm-sh64/system.h | 4 ++--
include/asm-sparc/irqflags.h | 2 +-
include/asm-sparc64/irqflags.h | 10 +++++-----
include/asm-um/system-generic.h | 8 ++++----
include/asm-v850/system.h | 6 +++---
include/asm-x86/irqflags.h | 12 ++++++------
include/asm-x86/paravirt.h | 4 ++--
include/asm-xtensa/system.h | 4 ++--
include/linux/ide.h | 2 +-
include/linux/interrupt.h | 2 +-
include/linux/irqflags.h | 4 ++--
48 files changed, 94 insertions(+), 94 deletions(-)

Index: linux-x86.q/Documentation/cli-sti-removal.txt
===================================================================
--- linux-x86.q.orig/Documentation/cli-sti-removal.txt
+++ linux-x86.q/Documentation/cli-sti-removal.txt
@@ -96,7 +96,7 @@ released.
drivers that want to disable local interrupts (interrupts on the
current CPU), can use the following five macros:

- local_irq_disable(), local_irq_enable(), local_save_flags(flags),
+ local_irq_disable(), local_irq_enable(), local_read_flags(flags),
local_irq_save(flags), local_irq_restore(flags)

but beware, their meaning and semantics are much simpler, far from
@@ -107,7 +107,7 @@ SMP meaning:

local_irq_enable() => turn local IRQs on

- local_save_flags(flags) => save the current IRQ state into flags. The
+ local_read_flags(flags) => save the current IRQ state into flags. The
state can be on or off. (on some
architectures there's even more bits in it.)

Index: linux-x86.q/arch/cris/arch-v10/mm/fault.c
===================================================================
--- linux-x86.q.orig/arch/cris/arch-v10/mm/fault.c
+++ linux-x86.q/arch/cris/arch-v10/mm/fault.c
@@ -80,7 +80,7 @@ handle_mmu_bus_fault(struct pt_regs *reg
* do_page_fault may have flushed the TLB so we have to restore
* the MMU registers.
*/
- local_save_flags(flags);
+ local_read_flags(flags);
local_irq_disable();
pmd = (pmd_t *)(pgd + pgd_index(address));
if (pmd_none(*pmd))
Index: linux-x86.q/arch/cris/arch-v10/mm/tlb.c
===================================================================
--- linux-x86.q.orig/arch/cris/arch-v10/mm/tlb.c
+++ linux-x86.q/arch/cris/arch-v10/mm/tlb.c
@@ -147,7 +147,7 @@ dump_tlb_all(void)

printk("TLB dump. LO is: pfn | reserved | global | valid | kernel | we |\n");

- local_save_flags(flags);
+ local_read_flags(flags);
local_irq_disable();
for(i = 0; i < NUM_TLB_ENTRIES; i++) {
*R_TLB_SELECT = ( IO_FIELD(R_TLB_SELECT, index, i) );
Index: linux-x86.q/arch/cris/arch-v32/kernel/time.c
===================================================================
--- linux-x86.q.orig/arch/cris/arch-v32/kernel/time.c
+++ linux-x86.q/arch/cris/arch-v32/kernel/time.c
@@ -336,7 +336,7 @@ time_init(void)
driver or infrastructure support yet. */
{
unsigned long flags;
- local_save_flags(flags);
+ local_read_flags(flags);
flags |= (1<<30); /* NMI M flag is at bit 30 */
local_irq_restore(flags);
}
Index: linux-x86.q/arch/cris/arch-v32/kernel/traps.c
===================================================================
--- linux-x86.q.orig/arch/cris/arch-v32/kernel/traps.c
+++ linux-x86.q/arch/cris/arch-v32/kernel/traps.c
@@ -153,7 +153,7 @@ die_if_kernel(const char *str, struct pt
void arch_enable_nmi(void)
{
unsigned long flags;
- local_save_flags(flags);
+ local_read_flags(flags);
flags |= (1<<30); /* NMI M flag is at bit 30 */
local_irq_restore(flags);
}
Index: linux-x86.q/arch/powerpc/kernel/traps.c
===================================================================
--- linux-x86.q.orig/arch/powerpc/kernel/traps.c
+++ linux-x86.q/arch/powerpc/kernel/traps.c
@@ -122,7 +122,7 @@ int die(const char *str, struct pt_regs
if (machine_is(powermac))
pmac_backlight_unblank();
} else {
- local_save_flags(flags);
+ local_read_flags(flags);
}

if (++die.lock_owner_depth < 3) {
Index: linux-x86.q/arch/x86/kernel/apm_32.c
===================================================================
--- linux-x86.q.orig/arch/x86/kernel/apm_32.c
+++ linux-x86.q/arch/x86/kernel/apm_32.c
@@ -535,7 +535,7 @@ static inline void apm_restore_cpus(cpum
static inline unsigned long __apm_irq_save(void)
{
unsigned long flags;
- local_save_flags(flags);
+ local_read_flags(flags);
if (apm_info.allow_ints) {
if (irqs_disabled_flags(flags))
local_irq_enable();
Index: linux-x86.q/arch/x86/kernel/io_apic_32.c
===================================================================
--- linux-x86.q.orig/arch/x86/kernel/io_apic_32.c
+++ linux-x86.q/arch/x86/kernel/io_apic_32.c
@@ -1885,7 +1885,7 @@ static int __init timer_irq_works(void)
if (no_timer_check)
return 1;

- local_save_flags(flags);
+ local_read_flags(flags);
local_irq_enable();
/* Let ten ticks pass... */
mdelay((10 * 1000) / HZ);
Index: linux-x86.q/arch/x86/kernel/io_apic_64.c
===================================================================
--- linux-x86.q.orig/arch/x86/kernel/io_apic_64.c
+++ linux-x86.q/arch/x86/kernel/io_apic_64.c
@@ -1284,7 +1284,7 @@ static int __init timer_irq_works(void)
unsigned long t1 = jiffies;
unsigned long flags;

- local_save_flags(flags);
+ local_read_flags(flags);
local_irq_enable();
/* Let ten ticks pass... */
mdelay((10 * 1000) / HZ);
Index: linux-x86.q/arch/x86/kernel/setup64.c
===================================================================
--- linux-x86.q.orig/arch/x86/kernel/setup64.c
+++ linux-x86.q/arch/x86/kernel/setup64.c
@@ -292,5 +292,5 @@ void __cpuinit cpu_init (void)

fpu_init();

- raw_local_save_flags(kernel_eflags);
+ raw_local_read_flags(kernel_eflags);
}
Index: linux-x86.q/arch/xtensa/mm/tlb.c
===================================================================
--- linux-x86.q.orig/arch/xtensa/mm/tlb.c
+++ linux-x86.q/arch/xtensa/mm/tlb.c
@@ -65,7 +65,7 @@ void flush_tlb_mm(struct mm_struct *mm)
{
if (mm == current->active_mm) {
int flags;
- local_save_flags(flags);
+ local_read_flags(flags);
__get_new_mmu_context(mm);
__load_mmu_context(mm);
local_irq_restore(flags);
@@ -95,7 +95,7 @@ void flush_tlb_range (struct vm_area_str
printk("[tlbrange<%02lx,%08lx,%08lx>]\n",
(unsigned long)mm->context, start, end);
#endif
- local_save_flags(flags);
+ local_read_flags(flags);

if (end-start + (PAGE_SIZE-1) <= _TLB_ENTRIES << PAGE_SHIFT) {
int oldpid = get_rasid_register();
@@ -129,7 +129,7 @@ void flush_tlb_page (struct vm_area_stru
if(mm->context == NO_CONTEXT)
return;

- local_save_flags(flags);
+ local_read_flags(flags);

oldpid = get_rasid_register();

Index: linux-x86.q/drivers/block/ataflop.c
===================================================================
--- linux-x86.q.orig/drivers/block/ataflop.c
+++ linux-x86.q/drivers/block/ataflop.c
@@ -1476,7 +1476,7 @@ void do_fd_request(struct request_queue
stdma_lock(floppy_irq, NULL);

atari_disable_irq( IRQ_MFP_FDC );
- local_save_flags(flags); /* The request function is called with ints
+ local_read_flags(flags); /* The request function is called with ints
local_irq_disable(); * disabled... so must save the IPL for later */
redo_fd_request();
local_irq_restore(flags);
Index: linux-x86.q/drivers/scsi/arm/acornscsi.c
===================================================================
--- linux-x86.q.orig/drivers/scsi/arm/acornscsi.c
+++ linux-x86.q/drivers/scsi/arm/acornscsi.c
@@ -316,7 +316,7 @@ acornscsi_csdelay(unsigned int cs)

target_jiffies = jiffies + 1 + cs * HZ / 100;

- local_save_flags(flags);
+ local_read_flags(flags);
local_irq_enable();

while (time_before(jiffies, target_jiffies)) barrier();
Index: linux-x86.q/drivers/scsi/atari_NCR5380.c
===================================================================
--- linux-x86.q.orig/drivers/scsi/atari_NCR5380.c
+++ linux-x86.q/drivers/scsi/atari_NCR5380.c
@@ -1066,7 +1066,7 @@ static void NCR5380_main(struct work_str
return;
main_running = 1;

- local_save_flags(flags);
+ local_read_flags(flags);
do {
local_irq_disable(); /* Freeze request queues */
done = 1;
Index: linux-x86.q/drivers/scsi/sun3_NCR5380.c
===================================================================
--- linux-x86.q.orig/drivers/scsi/sun3_NCR5380.c
+++ linux-x86.q/drivers/scsi/sun3_NCR5380.c
@@ -1063,7 +1063,7 @@ static void NCR5380_main (struct work_st
return;
main_running = 1;

- local_save_flags(flags);
+ local_read_flags(flags);
do {
local_irq_disable(); /* Freeze request queues */
done = 1;
Index: linux-x86.q/drivers/serial/68328serial.c
===================================================================
--- linux-x86.q.orig/drivers/serial/68328serial.c
+++ linux-x86.q/drivers/serial/68328serial.c
@@ -746,7 +746,7 @@ static int rs_write(struct tty_struct *
if (!tty || !info->xmit_buf)
return 0;

- local_save_flags(flags);
+ local_read_flags(flags);
while (1) {
local_irq_disable();
c = min_t(int, count, min(SERIAL_XMIT_SIZE - info->xmit_cnt - 1,
Index: linux-x86.q/drivers/serial/crisv10.c
===================================================================
--- linux-x86.q.orig/drivers/serial/crisv10.c
+++ linux-x86.q/drivers/serial/crisv10.c
@@ -3166,7 +3166,7 @@ static int rs_raw_write(struct tty_struc
SIMCOUT(buf, count);
return count;
#endif
- local_save_flags(flags);
+ local_read_flags(flags);
DFLOW(DEBUG_LOG(info->line, "write count %i ", count));
DFLOW(DEBUG_LOG(info->line, "ldisc %i\n", tty->ldisc.chars_in_buffer(tty)));

Index: linux-x86.q/drivers/serial/mcfserial.c
===================================================================
--- linux-x86.q.orig/drivers/serial/mcfserial.c
+++ linux-x86.q/drivers/serial/mcfserial.c
@@ -770,7 +770,7 @@ static int mcfrs_write(struct tty_struct
if (!tty || !info->xmit_buf)
return 0;

- local_save_flags(flags);
+ local_read_flags(flags);
while (1) {
local_irq_disable();
c = min(count, (int) min(((int)SERIAL_XMIT_SIZE) - info->xmit_cnt - 1,
Index: linux-x86.q/include/asm-alpha/system.h
===================================================================
--- linux-x86.q.orig/include/asm-alpha/system.h
+++ linux-x86.q/include/asm-alpha/system.h
@@ -281,7 +281,7 @@ extern int __min_ipl;

#define local_irq_disable() do { setipl(IPL_MAX); barrier(); } while(0)
#define local_irq_enable() do { barrier(); setipl(IPL_MIN); } while(0)
-#define local_save_flags(flags) ((flags) = rdps())
+#define local_read_flags(flags) ((flags) = rdps())
#define local_irq_save(flags) do { (flags) = swpipl(IPL_MAX); barrier(); } while(0)
#define local_irq_restore(flags) do { barrier(); setipl(flags); barrier(); } while(0)

Index: linux-x86.q/include/asm-arm/irqflags.h
===================================================================
--- linux-x86.q.orig/include/asm-arm/irqflags.h
+++ linux-x86.q/include/asm-arm/irqflags.h
@@ -106,10 +106,10 @@
/*
* Save the current interrupt enable state.
*/
-#define raw_local_save_flags(x) \
+#define raw_local_read_flags(x) \
({ \
__asm__ __volatile__( \
- "mrs %0, cpsr @ local_save_flags" \
+ "mrs %0, cpsr @ local_read_flags" \
: "=r" (x) : : "memory", "cc"); \
})

Index: linux-x86.q/include/asm-avr32/irqflags.h
===================================================================
--- linux-x86.q.orig/include/asm-avr32/irqflags.h
+++ linux-x86.q/include/asm-avr32/irqflags.h
@@ -10,13 +10,13 @@

#include <asm/sysreg.h>

-static inline unsigned long __raw_local_save_flags(void)
+static inline unsigned long __raw_local_read_flags(void)
{
return sysreg_read(SR);
}

-#define raw_local_save_flags(x) \
- do { (x) = __raw_local_save_flags(); } while (0)
+#define raw_local_read_flags(x) \
+ do { (x) = __raw_local_read_flags(); } while (0)

/*
* This will restore ALL status register flags, not only the interrupt
@@ -48,14 +48,14 @@ static inline int raw_irqs_disabled_flag

static inline int raw_irqs_disabled(void)
{
- unsigned long flags = __raw_local_save_flags();
+ unsigned long flags = __raw_local_read_flags();

return raw_irqs_disabled_flags(flags);
}

static inline unsigned long __raw_local_irq_save(void)
{
- unsigned long flags = __raw_local_save_flags();
+ unsigned long flags = __raw_local_read_flags();

raw_local_irq_disable();

Index: linux-x86.q/include/asm-blackfin/system.h
===================================================================
--- linux-x86.q.orig/include/asm-blackfin/system.h
+++ linux-x86.q/include/asm-blackfin/system.h
@@ -92,7 +92,7 @@ extern unsigned long irq_flags;
)
#endif

-#define local_save_flags(x) \
+#define local_read_flags(x) \
__asm__ __volatile__( \
"cli %0;" \
"sti %0;" \
@@ -117,7 +117,7 @@ extern unsigned long irq_flags;
#define irqs_disabled() \
({ \
unsigned long flags; \
- local_save_flags(flags); \
+ local_read_flags(flags); \
!irqs_enabled_from_flags(flags); \
})

Index: linux-x86.q/include/asm-cris/arch-v10/system.h
===================================================================
--- linux-x86.q.orig/include/asm-cris/arch-v10/system.h
+++ linux-x86.q/include/asm-cris/arch-v10/system.h
@@ -45,7 +45,7 @@ struct __xchg_dummy { unsigned long a[10
#define __xg(x) ((struct __xchg_dummy *)(x))

/* interrupt control.. */
-#define local_save_flags(x) __asm__ __volatile__ ("move $ccr,%0" : "=rm" (x) : : "memory");
+#define local_read_flags(x) __asm__ __volatile__ ("move $ccr,%0" : "=rm" (x) : : "memory");
#define local_irq_restore(x) __asm__ __volatile__ ("move %0,$ccr" : : "rm" (x) : "memory");
#define local_irq_disable() __asm__ __volatile__ ( "di" : : :"memory");
#define local_irq_enable() __asm__ __volatile__ ( "ei" : : :"memory");
@@ -53,7 +53,7 @@ struct __xchg_dummy { unsigned long a[10
#define irqs_disabled() \
({ \
unsigned long flags; \
- local_save_flags(flags); \
+ local_read_flags(flags); \
!(flags & (1<<5)); \
})

Index: linux-x86.q/include/asm-cris/arch-v32/system.h
===================================================================
--- linux-x86.q.orig/include/asm-cris/arch-v32/system.h
+++ linux-x86.q/include/asm-cris/arch-v32/system.h
@@ -45,7 +45,7 @@ struct __xchg_dummy { unsigned long a[10
#define __xg(x) ((struct __xchg_dummy *)(x))

/* Used for interrupt control. */
-#define local_save_flags(x) \
+#define local_read_flags(x) \
__asm__ __volatile__ ("move $ccs, %0" : "=rm" (x) : : "memory");

#define local_irq_restore(x) \
@@ -58,7 +58,7 @@ struct __xchg_dummy { unsigned long a[10
({ \
unsigned long flags; \
\
- local_save_flags(flags);\
+ local_read_flags(flags);\
!(flags & (1 << I_CCS_BITNR)); \
})

Index: linux-x86.q/include/asm-frv/system.h
===================================================================
--- linux-x86.q.orig/include/asm-frv/system.h
+++ linux-x86.q/include/asm-frv/system.h
@@ -70,7 +70,7 @@ do { \
); \
} while(0)

-#define local_save_flags(flags) \
+#define local_read_flags(flags) \
do { \
typecheck(unsigned long, flags); \
asm volatile("movsg ccr,%0" \
@@ -86,12 +86,12 @@ do { \
} while(0)

#define irqs_disabled() \
- ({unsigned long flags; local_save_flags(flags); flags; })
+ ({unsigned long flags; local_read_flags(flags); flags; })

#define local_irq_save(flags) \
do { \
typecheck(unsigned long, flags); \
- local_save_flags(flags); \
+ local_read_flags(flags); \
local_irq_disable(); \
} while(0)

@@ -137,7 +137,7 @@ do { \
: "memory"); \
} while(0)

-#define __local_save_flags(flags) \
+#define __local_read_flags(flags) \
do { \
typecheck(unsigned long, flags); \
asm("movsg psr,%0" \
Index: linux-x86.q/include/asm-h8300/system.h
===================================================================
--- linux-x86.q.orig/include/asm-h8300/system.h
+++ linux-x86.q/include/asm-h8300/system.h
@@ -72,7 +72,7 @@ asmlinkage void resume(void);
#define local_irq_enable() __sti()
#define local_irq_save(x) ({ __save_flags(x); local_irq_disable(); })
#define local_irq_restore(x) __restore_flags(x)
-#define local_save_flags(x) __save_flags(x)
+#define local_read_flags(x) __save_flags(x)

/*
* Force strict CPU ordering.
Index: linux-x86.q/include/asm-ia64/system.h
===================================================================
--- linux-x86.q.orig/include/asm-ia64/system.h
+++ linux-x86.q/include/asm-ia64/system.h
@@ -160,7 +160,7 @@ do { \
do { \
unsigned long old_psr, psr = (x); \
\
- local_save_flags(old_psr); \
+ local_read_flags(old_psr); \
__local_irq_restore(psr); \
if ((old_psr & IA64_PSR_I) && !(psr & IA64_PSR_I)) \
__save_ip(); \
@@ -173,12 +173,12 @@ do { \
#endif /* !CONFIG_IA64_DEBUG_IRQ */

#define local_irq_enable() ({ ia64_stop(); ia64_ssm(IA64_PSR_I); ia64_srlz_d(); })
-#define local_save_flags(flags) ({ ia64_stop(); (flags) = ia64_getreg(_IA64_REG_PSR); })
+#define local_read_flags(flags) ({ ia64_stop(); (flags) = ia64_getreg(_IA64_REG_PSR); })

#define irqs_disabled() \
({ \
unsigned long __ia64_id_flags; \
- local_save_flags(__ia64_id_flags); \
+ local_read_flags(__ia64_id_flags); \
(__ia64_id_flags & IA64_PSR_I) == 0; \
})

Index: linux-x86.q/include/asm-m32r/system.h
===================================================================
--- linux-x86.q.orig/include/asm-m32r/system.h
+++ linux-x86.q/include/asm-m32r/system.h
@@ -84,7 +84,7 @@ static inline void local_irq_disable(voi
}
#endif /* CONFIG_CHIP_M32102 || CONFIG_CHIP_M32104 */

-#define local_save_flags(x) \
+#define local_read_flags(x) \
__asm__ __volatile__("mvfc %0,psw" : "=r"(x) : /* no input */)

#define local_irq_restore(x) \
@@ -115,7 +115,7 @@ static inline void local_irq_disable(voi
#define irqs_disabled() \
({ \
unsigned long flags; \
- local_save_flags(flags); \
+ local_read_flags(flags); \
!(flags & 0x40); \
})

Index: linux-x86.q/include/asm-m68k/system.h
===================================================================
--- linux-x86.q.orig/include/asm-m68k/system.h
+++ linux-x86.q/include/asm-m68k/system.h
@@ -73,18 +73,18 @@ asmlinkage void resume(void);
})
#endif
#define local_irq_disable() asm volatile ("oriw #0x0700,%%sr": : : "memory")
-#define local_save_flags(x) asm volatile ("movew %%sr,%0":"=d" (x) : : "memory")
+#define local_read_flags(x) asm volatile ("movew %%sr,%0":"=d" (x) : : "memory")
#define local_irq_restore(x) asm volatile ("movew %0,%%sr": :"d" (x) : "memory")

static inline int irqs_disabled(void)
{
unsigned long flags;
- local_save_flags(flags);
+ local_read_flags(flags);
return flags & ~ALLOWINT;
}

/* For spinlocks etc */
-#define local_irq_save(x) ({ local_save_flags(x); local_irq_disable(); })
+#define local_irq_save(x) ({ local_read_flags(x); local_irq_disable(); })

#define xchg(ptr,x) ((__typeof__(*(ptr)))__xchg((unsigned long)(x),(ptr),sizeof(*(ptr))))

Index: linux-x86.q/include/asm-m68knommu/system.h
===================================================================
--- linux-x86.q.orig/include/asm-m68knommu/system.h
+++ linux-x86.q/include/asm-m68knommu/system.h
@@ -79,18 +79,18 @@ asmlinkage void resume(void);
#define local_irq_disable() asm volatile ("oriw #0x0700,%%sr": : : "memory")
#endif

-#define local_save_flags(x) asm volatile ("movew %%sr,%0":"=d" (x) : : "memory")
+#define local_read_flags(x) asm volatile ("movew %%sr,%0":"=d" (x) : : "memory")
#define local_irq_restore(x) asm volatile ("movew %0,%%sr": :"d" (x) : "memory")

/* For spinlocks etc */
#ifndef local_irq_save
-#define local_irq_save(x) do { local_save_flags(x); local_irq_disable(); } while (0)
+#define local_irq_save(x) do { local_read_flags(x); local_irq_disable(); } while (0)
#endif

#define irqs_disabled() \
({ \
unsigned long flags; \
- local_save_flags(flags); \
+ local_read_flags(flags); \
((flags & 0x0700) == 0x0700); \
})

Index: linux-x86.q/include/asm-mips/irqflags.h
===================================================================
--- linux-x86.q.orig/include/asm-mips/irqflags.h
+++ linux-x86.q/include/asm-mips/irqflags.h
@@ -97,7 +97,7 @@ static inline void raw_local_irq_disable
}

__asm__(
- " .macro raw_local_save_flags flags \n"
+ " .macro raw_local_read_flags flags \n"
" .set push \n"
" .set reorder \n"
#ifdef CONFIG_MIPS_MT_SMTC
@@ -108,9 +108,9 @@ __asm__(
" .set pop \n"
" .endm \n");

-#define raw_local_save_flags(x) \
+#define raw_local_read_flags(x) \
__asm__ __volatile__( \
- "raw_local_save_flags %0" \
+ "raw_local_read_flags %0" \
: "=r" (x))

__asm__(
Index: linux-x86.q/include/asm-parisc/system.h
===================================================================
--- linux-x86.q.orig/include/asm-parisc/system.h
+++ linux-x86.q/include/asm-parisc/system.h
@@ -49,7 +49,7 @@ extern struct task_struct *_switch_to(st
} while(0)

/* interrupt control */
-#define local_save_flags(x) __asm__ __volatile__("ssm 0, %0" : "=r" (x) : : "memory")
+#define local_read_flags(x) __asm__ __volatile__("ssm 0, %0" : "=r" (x) : : "memory")
#define local_irq_disable() __asm__ __volatile__("rsm %0,%%r0\n" : : "i" (PSW_I) : "memory" )
#define local_irq_enable() __asm__ __volatile__("ssm %0,%%r0\n" : : "i" (PSW_I) : "memory" )

@@ -61,7 +61,7 @@ extern struct task_struct *_switch_to(st
#define irqs_disabled() \
({ \
unsigned long flags; \
- local_save_flags(flags); \
+ local_read_flags(flags); \
(flags & PSW_I) == 0; \
})

Index: linux-x86.q/include/asm-powerpc/hw_irq.h
===================================================================
--- linux-x86.q.orig/include/asm-powerpc/hw_irq.h
+++ linux-x86.q/include/asm-powerpc/hw_irq.h
@@ -43,7 +43,7 @@ extern void local_irq_restore(unsigned l
extern void iseries_handle_interrupts(void);

#define local_irq_enable() local_irq_restore(1)
-#define local_save_flags(flags) ((flags) = local_get_flags())
+#define local_read_flags(flags) ((flags) = local_get_flags())
#define local_irq_save(flags) ((flags) = local_irq_disable())

#define irqs_disabled() (local_get_flags() == 0)
@@ -105,7 +105,7 @@ static inline void local_irq_save_ptr(un
__asm__ __volatile__("": : :"memory");
}

-#define local_save_flags(flags) ((flags) = mfmsr())
+#define local_read_flags(flags) ((flags) = mfmsr())
#define local_irq_save(flags) local_irq_save_ptr(&flags)
#define irqs_disabled() ((mfmsr() & MSR_EE) == 0)

Index: linux-x86.q/include/asm-powerpc/irqflags.h
===================================================================
--- linux-x86.q.orig/include/asm-powerpc/irqflags.h
+++ linux-x86.q/include/asm-powerpc/irqflags.h
@@ -11,7 +11,7 @@
#define _ASM_IRQFLAGS_H

/*
- * Get definitions for raw_local_save_flags(x), etc.
+ * Get definitions for raw_local_read_flags(x), etc.
*/
#include <asm-powerpc/hw_irq.h>

Index: linux-x86.q/include/asm-powerpc/spinlock.h
===================================================================
--- linux-x86.q.orig/include/asm-powerpc/spinlock.h
+++ linux-x86.q/include/asm-powerpc/spinlock.h
@@ -126,7 +126,7 @@ static void __inline__ __raw_spin_lock_f
while (1) {
if (likely(__spin_trylock(lock) == 0))
break;
- local_save_flags(flags_dis);
+ local_read_flags(flags_dis);
local_irq_restore(flags);
do {
HMT_low();
Index: linux-x86.q/include/asm-s390/irqflags.h
===================================================================
--- linux-x86.q.orig/include/asm-s390/irqflags.h
+++ linux-x86.q/include/asm-s390/irqflags.h
@@ -83,7 +83,7 @@ static inline unsigned long raw_local_ir
return __raw_local_irq_stnsm(0xfc);
}

-#define raw_local_save_flags(x) \
+#define raw_local_read_flags(x) \
do { \
typecheck(unsigned long, x); \
(x) = __raw_local_irq_stosm(0x00); \
Index: linux-x86.q/include/asm-sh/irqflags.h
===================================================================
--- linux-x86.q.orig/include/asm-sh/irqflags.h
+++ linux-x86.q/include/asm-sh/irqflags.h
@@ -62,7 +62,7 @@ static inline void clear_bl_bit(void)
);
}

-static inline unsigned long __raw_local_save_flags(void)
+static inline unsigned long __raw_local_read_flags(void)
{
unsigned long flags;

@@ -77,8 +77,8 @@ static inline unsigned long __raw_local_
return flags;
}

-#define raw_local_save_flags(flags) \
- do { (flags) = __raw_local_save_flags(); } while (0)
+#define raw_local_read_flags(flags) \
+ do { (flags) = __raw_local_read_flags(); } while (0)

static inline int raw_irqs_disabled_flags(unsigned long flags)
{
@@ -87,7 +87,7 @@ static inline int raw_irqs_disabled_flag

static inline int raw_irqs_disabled(void)
{
- unsigned long flags = __raw_local_save_flags();
+ unsigned long flags = __raw_local_read_flags();

return raw_irqs_disabled_flags(flags);
}
Index: linux-x86.q/include/asm-sh64/system.h
===================================================================
--- linux-x86.q.orig/include/asm-sh64/system.h
+++ linux-x86.q/include/asm-sh64/system.h
@@ -96,7 +96,7 @@ static __inline__ void local_irq_disable
: "r" (__dummy1));
}

-#define local_save_flags(x) \
+#define local_read_flags(x) \
(__extension__ ({ unsigned long long __dummy=SR_MASK_LL; \
__asm__ __volatile__( \
"getcon " __SR ", %0\n\t" \
@@ -123,7 +123,7 @@ static __inline__ void local_irq_disable
#define irqs_disabled() \
({ \
unsigned long flags; \
- local_save_flags(flags); \
+ local_read_flags(flags); \
(flags != 0); \
})

Index: linux-x86.q/include/asm-sparc/irqflags.h
===================================================================
--- linux-x86.q.orig/include/asm-sparc/irqflags.h
+++ linux-x86.q/include/asm-sparc/irqflags.h
@@ -24,7 +24,7 @@ static inline unsigned long getipl(void)
return retval;
}

-#define raw_local_save_flags(flags) ((flags) = getipl())
+#define raw_local_read_flags(flags) ((flags) = getipl())
#define raw_local_irq_save(flags) ((flags) = __raw_local_irq_save())
#define raw_local_irq_disable() ((void) __raw_local_irq_save())
#define raw_irqs_disabled() ((getipl() & PSR_PIL) != 0)
Index: linux-x86.q/include/asm-sparc64/irqflags.h
===================================================================
--- linux-x86.q.orig/include/asm-sparc64/irqflags.h
+++ linux-x86.q/include/asm-sparc64/irqflags.h
@@ -12,7 +12,7 @@

#ifndef __ASSEMBLY__

-static inline unsigned long __raw_local_save_flags(void)
+static inline unsigned long __raw_local_read_flags(void)
{
unsigned long flags;

@@ -24,8 +24,8 @@ static inline unsigned long __raw_local_
return flags;
}

-#define raw_local_save_flags(flags) \
- do { (flags) = __raw_local_save_flags(); } while (0)
+#define raw_local_read_flags(flags) \
+ do { (flags) = __raw_local_read_flags(); } while (0)

static inline void raw_local_irq_restore(unsigned long flags)
{
@@ -64,7 +64,7 @@ static inline int raw_irqs_disabled_flag

static inline int raw_irqs_disabled(void)
{
- unsigned long flags = __raw_local_save_flags();
+ unsigned long flags = __raw_local_read_flags();

return raw_irqs_disabled_flags(flags);
}
@@ -74,7 +74,7 @@ static inline int raw_irqs_disabled(void
*/
static inline unsigned long __raw_local_irq_save(void)
{
- unsigned long flags = __raw_local_save_flags();
+ unsigned long flags = __raw_local_read_flags();

raw_local_irq_disable();

Index: linux-x86.q/include/asm-um/system-generic.h
===================================================================
--- linux-x86.q.orig/include/asm-um/system-generic.h
+++ linux-x86.q/include/asm-um/system-generic.h
@@ -8,7 +8,7 @@
#undef local_irq_restore
#undef local_irq_disable
#undef local_irq_enable
-#undef local_save_flags
+#undef local_read_flags
#undef local_irq_restore
#undef local_irq_enable
#undef local_irq_disable
@@ -23,12 +23,12 @@ extern int get_signals(void);
extern void block_signals(void);
extern void unblock_signals(void);

-#define local_save_flags(flags) do { typecheck(unsigned long, flags); \
+#define local_read_flags(flags) do { typecheck(unsigned long, flags); \
(flags) = get_signals(); } while(0)
#define local_irq_restore(flags) do { typecheck(unsigned long, flags); \
set_signals(flags); } while(0)

-#define local_irq_save(flags) do { local_save_flags(flags); \
+#define local_irq_save(flags) do { local_read_flags(flags); \
local_irq_disable(); } while(0)

#define local_irq_enable() unblock_signals()
@@ -37,7 +37,7 @@ extern void unblock_signals(void);
#define irqs_disabled() \
({ \
unsigned long flags; \
- local_save_flags(flags); \
+ local_read_flags(flags); \
(flags == 0); \
})

Index: linux-x86.q/include/asm-v850/system.h
===================================================================
--- linux-x86.q.orig/include/asm-v850/system.h
+++ linux-x86.q/include/asm-v850/system.h
@@ -37,14 +37,14 @@ extern void *switch_thread (struct threa
#define local_irq_enable() __asm__ __volatile__ ("ei")
#define local_irq_disable() __asm__ __volatile__ ("di")

-#define local_save_flags(flags) \
+#define local_read_flags(flags) \
__asm__ __volatile__ ("stsr %1, %0" : "=r" (flags) : "i" (SR_PSW))
#define local_restore_flags(flags) \
__asm__ __volatile__ ("ldsr %0, %1" :: "r" (flags), "i" (SR_PSW))

/* For spinlocks etc */
#define local_irq_save(flags) \
- do { local_save_flags (flags); local_irq_disable (); } while (0)
+ do { local_read_flags (flags); local_irq_disable (); } while (0)
#define local_irq_restore(flags) \
local_restore_flags (flags);

@@ -52,7 +52,7 @@ extern void *switch_thread (struct threa
static inline int irqs_disabled (void)
{
unsigned flags;
- local_save_flags (flags);
+ local_read_flags (flags);
return !!(flags & 0x20);
}

Index: linux-x86.q/include/asm-x86/irqflags.h
===================================================================
--- linux-x86.q.orig/include/asm-x86/irqflags.h
+++ linux-x86.q/include/asm-x86/irqflags.h
@@ -60,7 +60,7 @@ static inline void native_halt(void)
#else
#ifndef __ASSEMBLY__

-static inline unsigned long __raw_local_save_flags(void)
+static inline unsigned long __raw_local_read_flags(void)
{
return native_save_fl();
}
@@ -103,7 +103,7 @@ static inline void halt(void)
*/
static inline unsigned long __raw_local_irq_save(void)
{
- unsigned long flags = __raw_local_save_flags();
+ unsigned long flags = __raw_local_read_flags();

raw_local_irq_disable();

@@ -131,8 +131,8 @@ static inline unsigned long __raw_local_
#endif /* CONFIG_PARAVIRT */

#ifndef __ASSEMBLY__
-#define raw_local_save_flags(flags) \
- do { (flags) = __raw_local_save_flags(); } while (0)
+#define raw_local_read_flags(flags) \
+ do { (flags) = __raw_local_read_flags(); } while (0)

#define raw_local_irq_save(flags) \
do { (flags) = __raw_local_irq_save(); } while (0)
@@ -144,7 +144,7 @@ static inline int raw_irqs_disabled_flag

static inline int raw_irqs_disabled(void)
{
- unsigned long flags = __raw_local_save_flags();
+ unsigned long flags = __raw_local_read_flags();

return raw_irqs_disabled_flags(flags);
}
@@ -165,7 +165,7 @@ static inline void trace_hardirqs_fixup_

static inline void trace_hardirqs_fixup(void)
{
- unsigned long flags = __raw_local_save_flags();
+ unsigned long flags = __raw_local_read_flags();

trace_hardirqs_fixup_flags(flags);
}
Index: linux-x86.q/include/asm-x86/paravirt.h
===================================================================
--- linux-x86.q.orig/include/asm-x86/paravirt.h
+++ linux-x86.q/include/asm-x86/paravirt.h
@@ -1124,7 +1124,7 @@ extern struct paravirt_patch_site __para
#define PV_FLAGS_ARG "D"
#endif

-static inline unsigned long __raw_local_save_flags(void)
+static inline unsigned long __raw_local_read_flags(void)
{
unsigned long f;

@@ -1176,7 +1176,7 @@ static inline unsigned long __raw_local_
{
unsigned long f;

- f = __raw_local_save_flags();
+ f = __raw_local_read_flags();
raw_local_irq_disable();
return f;
}
Index: linux-x86.q/include/asm-xtensa/system.h
===================================================================
--- linux-x86.q.orig/include/asm-xtensa/system.h
+++ linux-x86.q/include/asm-xtensa/system.h
@@ -17,7 +17,7 @@

/* interrupt control */

-#define local_save_flags(x) \
+#define local_read_flags(x) \
__asm__ __volatile__ ("rsr %0,"__stringify(PS) : "=a" (x));
#define local_irq_restore(x) do { \
__asm__ __volatile__ ("wsr %0, "__stringify(PS)" ; rsync" \
@@ -42,7 +42,7 @@ static inline void local_irq_enable(void
static inline int irqs_disabled(void)
{
unsigned long flags;
- local_save_flags(flags);
+ local_read_flags(flags);
return flags & 0xf;
}

Index: linux-x86.q/include/linux/ide.h
===================================================================
--- linux-x86.q.orig/include/linux/ide.h
+++ linux-x86.q/include/linux/ide.h
@@ -1394,7 +1394,7 @@ extern struct mutex ide_cfg_mtx;
* ide_drive_t->hwif: constant, no locking
*/

-#define local_irq_set(flags) do { local_save_flags((flags)); local_irq_enable_in_hardirq(); } while (0)
+#define local_irq_set(flags) do { local_read_flags((flags)); local_irq_enable_in_hardirq(); } while (0)

extern struct bus_type ide_bus_type;

Index: linux-x86.q/include/linux/interrupt.h
===================================================================
--- linux-x86.q.orig/include/linux/interrupt.h
+++ linux-x86.q/include/linux/interrupt.h
@@ -213,7 +213,7 @@ static inline void __deprecated sti(void
}
static inline void __deprecated save_flags(unsigned long *x)
{
- local_save_flags(*x);
+ local_read_flags(*x);
}
#define save_flags(x) save_flags(&x)
static inline void __deprecated restore_flags(unsigned long x)
Index: linux-x86.q/include/linux/irqflags.h
===================================================================
--- linux-x86.q.orig/include/linux/irqflags.h
+++ linux-x86.q/include/linux/irqflags.h
@@ -80,13 +80,13 @@
raw_safe_halt(); \
} while (0)

-#define local_save_flags(flags) raw_local_save_flags(flags)
+#define local_read_flags(flags) raw_local_read_flags(flags)

#define irqs_disabled() \
({ \
unsigned long flags; \
\
- raw_local_save_flags(flags); \
+ raw_local_read_flags(flags); \
raw_irqs_disabled_flags(flags); \
})

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