diff options
author | Daniel Santos <daniel.santos@pobox.com> | 2013-02-21 19:41:42 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2013-02-21 20:22:15 -0500 |
commit | 6ae8d04871f84d853673e9e9f3f713e77a2fe145 (patch) | |
tree | 53df264231784be05e925e6c1468f0119a6e477a | |
parent | 733ed6e43756b0aec25c9429b810ba74e24c980c (diff) |
compiler{,-gcc4}.h, bug.h: Remove duplicate macros
__linktime_error() does the same thing as __compiletime_error() and is
only used in bug.h. Since the macro defines a function attribute that
will cause a failure at compile-time (not link-time), it makes more sense
to keep __compiletime_error(), which is also neatly mated with
__compiletime_warning().
Signed-off-by: Daniel Santos <daniel.santos@pobox.com>
Acked-by: David Rientjes <rientjes@google.com>
Acked-by: Borislav Petkov <bp@alien8.de>
Cc: Andi Kleen <ak@linux.intel.com>
Cc: Joe Perches <joe@perches.com>
Cc: Josh Triplett <josh@joshtriplett.org>
Cc: Paul Gortmaker <paul.gortmaker@windriver.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | include/linux/bug.h | 2 | ||||
-rw-r--r-- | include/linux/compiler-gcc4.h | 2 | ||||
-rw-r--r-- | include/linux/compiler.h | 3 |
3 files changed, 1 insertions, 6 deletions
diff --git a/include/linux/bug.h b/include/linux/bug.h index b1cf40de847e..2a11774c5e64 100644 --- a/include/linux/bug.h +++ b/include/linux/bug.h | |||
@@ -74,7 +74,7 @@ extern int __build_bug_on_failed; | |||
74 | #define BUILD_BUG() \ | 74 | #define BUILD_BUG() \ |
75 | do { \ | 75 | do { \ |
76 | extern void __build_bug_failed(void) \ | 76 | extern void __build_bug_failed(void) \ |
77 | __linktime_error("BUILD_BUG failed"); \ | 77 | __compiletime_error("BUILD_BUG failed");\ |
78 | __build_bug_failed(); \ | 78 | __build_bug_failed(); \ |
79 | } while (0) | 79 | } while (0) |
80 | 80 | ||
diff --git a/include/linux/compiler-gcc4.h b/include/linux/compiler-gcc4.h index a9ffdfe7713c..68b162d92254 100644 --- a/include/linux/compiler-gcc4.h +++ b/include/linux/compiler-gcc4.h | |||
@@ -33,8 +33,6 @@ | |||
33 | the kernel context */ | 33 | the kernel context */ |
34 | #define __cold __attribute__((__cold__)) | 34 | #define __cold __attribute__((__cold__)) |
35 | 35 | ||
36 | #define __linktime_error(message) __attribute__((__error__(message))) | ||
37 | |||
38 | #define __UNIQUE_ID(prefix) __PASTE(__PASTE(__UNIQUE_ID_, prefix), __COUNTER__) | 36 | #define __UNIQUE_ID(prefix) __PASTE(__PASTE(__UNIQUE_ID_, prefix), __COUNTER__) |
39 | 37 | ||
40 | #ifndef __CHECKER__ | 38 | #ifndef __CHECKER__ |
diff --git a/include/linux/compiler.h b/include/linux/compiler.h index dd852b73b286..4c638be76093 100644 --- a/include/linux/compiler.h +++ b/include/linux/compiler.h | |||
@@ -308,9 +308,6 @@ void ftrace_likely_update(struct ftrace_branch_data *f, int val, int expect); | |||
308 | #ifndef __compiletime_error | 308 | #ifndef __compiletime_error |
309 | # define __compiletime_error(message) | 309 | # define __compiletime_error(message) |
310 | #endif | 310 | #endif |
311 | #ifndef __linktime_error | ||
312 | # define __linktime_error(message) | ||
313 | #endif | ||
314 | /* | 311 | /* |
315 | * Prevent the compiler from merging or refetching accesses. The compiler | 312 | * Prevent the compiler from merging or refetching accesses. The compiler |
316 | * is also forbidden from reordering successive instances of ACCESS_ONCE(), | 313 | * is also forbidden from reordering successive instances of ACCESS_ONCE(), |