Преглед изворни кода

[PATCH] qualifiers in return types - easy cases

a bunch of functions switched from volatile to __attribute__((noreturn)) and
from const to __attribute_pure__

Signed-off-by: Al Viro <viro@parcelfarce.linux.theplanet.co.uk>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Al Viro пре 19 година
родитељ
комит
33215652e4

+ 1 - 1
arch/arm/kernel/traps.c

@@ -617,7 +617,7 @@ baddataabort(int code, unsigned long instr, struct pt_regs *regs)
 	notify_die("unknown data abort code", regs, &info, instr, 0);
 	notify_die("unknown data abort code", regs, &info, instr, 0);
 }
 }
 
 
-volatile void __bug(const char *file, int line, void *data)
+void __attribute__((noreturn)) __bug(const char *file, int line, void *data)
 {
 {
 	printk(KERN_CRIT"kernel BUG at %s:%d!", file, line);
 	printk(KERN_CRIT"kernel BUG at %s:%d!", file, line);
 	if (data)
 	if (data)

+ 3 - 3
arch/arm/nwfpe/fpopcode.h

@@ -370,20 +370,20 @@ TABLE 5
 #define getRoundingMode(opcode)		((opcode & MASK_ROUNDING_MODE) >> 5)
 #define getRoundingMode(opcode)		((opcode & MASK_ROUNDING_MODE) >> 5)
 
 
 #ifdef CONFIG_FPE_NWFPE_XP
 #ifdef CONFIG_FPE_NWFPE_XP
-static inline const floatx80 getExtendedConstant(const unsigned int nIndex)
+static inline __attribute_pure__ floatx80 getExtendedConstant(const unsigned int nIndex)
 {
 {
 	extern const floatx80 floatx80Constant[];
 	extern const floatx80 floatx80Constant[];
 	return floatx80Constant[nIndex];
 	return floatx80Constant[nIndex];
 }
 }
 #endif
 #endif
 
 
-static inline const float64 getDoubleConstant(const unsigned int nIndex)
+static inline __attribute_pure__ float64 getDoubleConstant(const unsigned int nIndex)
 {
 {
 	extern const float64 float64Constant[];
 	extern const float64 float64Constant[];
 	return float64Constant[nIndex];
 	return float64Constant[nIndex];
 }
 }
 
 
-static inline const float32 getSingleConstant(const unsigned int nIndex)
+static inline __attribute_pure__ float32 getSingleConstant(const unsigned int nIndex)
 {
 {
 	extern const float32 float32Constant[];
 	extern const float32 float32Constant[];
 	return float32Constant[nIndex];
 	return float32Constant[nIndex];

+ 1 - 1
include/asm-arm/bug.h

@@ -5,7 +5,7 @@
 
 
 #ifdef CONFIG_BUG
 #ifdef CONFIG_BUG
 #ifdef CONFIG_DEBUG_BUGVERBOSE
 #ifdef CONFIG_DEBUG_BUGVERBOSE
-extern volatile void __bug(const char *file, int line, void *data);
+extern void __bug(const char *file, int line, void *data) __attribute__((noreturn));
 
 
 /* give file/line information */
 /* give file/line information */
 #define BUG()		__bug(__FILE__, __LINE__, NULL)
 #define BUG()		__bug(__FILE__, __LINE__, NULL)

+ 1 - 1
include/asm-arm/cpu-multi32.h

@@ -31,7 +31,7 @@ extern struct processor {
 	/*
 	/*
 	 * Special stuff for a reset
 	 * Special stuff for a reset
 	 */
 	 */
-	volatile void (*reset)(unsigned long addr);
+	void (*reset)(unsigned long addr) __attribute__((noreturn));
 	/*
 	/*
 	 * Idle the processor
 	 * Idle the processor
 	 */
 	 */

+ 1 - 1
include/asm-arm/cpu-single.h

@@ -41,4 +41,4 @@ extern int cpu_do_idle(void);
 extern void cpu_dcache_clean_area(void *, int);
 extern void cpu_dcache_clean_area(void *, int);
 extern void cpu_do_switch_mm(unsigned long pgd_phys, struct mm_struct *mm);
 extern void cpu_do_switch_mm(unsigned long pgd_phys, struct mm_struct *mm);
 extern void cpu_set_pte(pte_t *ptep, pte_t pte);
 extern void cpu_set_pte(pte_t *ptep, pte_t pte);
-extern volatile void cpu_reset(unsigned long addr);
+extern void cpu_reset(unsigned long addr) __attribute__((noreturn));

+ 1 - 1
include/asm-ppc/time.h

@@ -58,7 +58,7 @@ static __inline__ void set_dec(unsigned int val)
 /* Accessor functions for the timebase (RTC on 601) registers. */
 /* Accessor functions for the timebase (RTC on 601) registers. */
 /* If one day CONFIG_POWER is added just define __USE_RTC as 1 */
 /* If one day CONFIG_POWER is added just define __USE_RTC as 1 */
 #ifdef CONFIG_6xx
 #ifdef CONFIG_6xx
-extern __inline__ int const __USE_RTC(void) {
+extern __inline__ int __attribute_pure__ __USE_RTC(void) {
 	return (mfspr(SPRN_PVR)>>16) == 1;
 	return (mfspr(SPRN_PVR)>>16) == 1;
 }
 }
 #else
 #else