diff options
author | Dale Rahn <drahn@cvs.openbsd.org> | 2001-11-19 05:13:51 +0000 |
---|---|---|
committer | Dale Rahn <drahn@cvs.openbsd.org> | 2001-11-19 05:13:51 +0000 |
commit | 6c6cba13d91caa30e3fa45952afcb78986e44247 (patch) | |
tree | ee9a007e8851b0137f20860f19d2820a0d192c45 /sys/arch/powerpc/include | |
parent | a7a3088671f75ad2d9d929833d25169f7d3cc68d (diff) |
Remove pointless additional 'syncing' instructions in the powerpc spl*()
functions. Seems to have no effect on system run time (it should have...)
reduces GENERIC kernel size by 52k.
Diffstat (limited to 'sys/arch/powerpc/include')
-rw-r--r-- | sys/arch/powerpc/include/intr.h | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/sys/arch/powerpc/include/intr.h b/sys/arch/powerpc/include/intr.h index 0618a585495..f84c00c85b6 100644 --- a/sys/arch/powerpc/include/intr.h +++ b/sys/arch/powerpc/include/intr.h @@ -1,4 +1,4 @@ -/* $OpenBSD: intr.h,v 1.12 2001/09/01 15:49:05 drahn Exp $ */ +/* $OpenBSD: intr.h,v 1.13 2001/11/19 05:13:50 drahn Exp $ */ /* * Copyright (c) 1997 Per Fogelstrom, Opsycon AB and RTMX Inc, USA. @@ -68,30 +68,30 @@ volatile extern int cpl, ipending, astpending, tickspending; extern int imask[7]; /* - * Reorder protection in the following inline functions is - * achived with the "eieio" instruction which the assembler - * seems to detect and then doen't move instructions past.... + * Reorder protection in the following inline functions is + * achived with an empty asm volatile statement. the compiler + * will not move instructions past asm volatiles. */ static __inline int splraise(int newcpl) { int oldcpl; - __asm__ volatile("sync; eieio\n"); /* don't reorder.... */ + __asm__ volatile(""); /* don't reorder.... */ oldcpl = cpl; cpl = oldcpl | newcpl; - __asm__ volatile("sync; eieio\n"); /* reorder protect */ + __asm__ volatile(""); /* reorder protect */ return(oldcpl); } static __inline void splx(int newcpl) { - __asm__ volatile("sync; eieio\n"); /* reorder protect */ + __asm__ volatile(""); /* reorder protect */ cpl = newcpl; if(ipending & ~newcpl) do_pending_int(); - __asm__ volatile("sync; eieio\n"); /* reorder protect */ + __asm__ volatile(""); /* reorder protect */ } static __inline int @@ -99,12 +99,12 @@ spllower(int newcpl) { int oldcpl; - __asm__ volatile("sync; eieio\n"); /* reorder protect */ + __asm__ volatile(""); /* reorder protect */ oldcpl = cpl; cpl = newcpl; if(ipending & ~newcpl) do_pending_int(); - __asm__ volatile("sync; eieio\n"); /* reorder protect */ + __asm__ volatile(""); /* reorder protect */ return(oldcpl); } |