diff options
| author | Don Mullis <dwm@meer.net> | 2006-12-08 05:39:51 -0500 |
|---|---|---|
| committer | Linus Torvalds <torvalds@woody.osdl.org> | 2006-12-08 11:29:03 -0500 |
| commit | 08b3df2d16cbebf7d72c09dcbc071696c14d07e3 (patch) | |
| tree | 2cc0baeda87e24d9839845c0d34cd068453b1518 /lib | |
| parent | 5d0ffa2b84e6f0fdc5bf96c0de6178f3d2779544 (diff) | |
[PATCH] fault-injection: Use bool-true-false throughout
Use bool-true-false throughout.
Signed-off-by: Don Mullis <dwm@meer.net>
Cc: Akinobu Mita <akinobu.mita@gmail.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/fault-inject.c | 40 |
1 files changed, 19 insertions, 21 deletions
diff --git a/lib/fault-inject.c b/lib/fault-inject.c index 361c6e9cd77f..244782940349 100644 --- a/lib/fault-inject.c +++ b/lib/fault-inject.c | |||
| @@ -48,7 +48,7 @@ static void fail_dump(struct fault_attr *attr) | |||
| 48 | 48 | ||
| 49 | #define atomic_dec_not_zero(v) atomic_add_unless((v), -1, 0) | 49 | #define atomic_dec_not_zero(v) atomic_add_unless((v), -1, 0) |
| 50 | 50 | ||
| 51 | static int fail_task(struct fault_attr *attr, struct task_struct *task) | 51 | static bool fail_task(struct fault_attr *attr, struct task_struct *task) |
| 52 | { | 52 | { |
| 53 | return !in_interrupt() && task->make_it_fail; | 53 | return !in_interrupt() && task->make_it_fail; |
| 54 | } | 54 | } |
| @@ -68,15 +68,15 @@ static asmlinkage int fail_stacktrace_callback(struct unwind_frame_info *info, | |||
| 68 | break; | 68 | break; |
| 69 | if (attr->reject_start <= UNW_PC(info) && | 69 | if (attr->reject_start <= UNW_PC(info) && |
| 70 | UNW_PC(info) < attr->reject_end) | 70 | UNW_PC(info) < attr->reject_end) |
| 71 | return 0; | 71 | return false; |
| 72 | if (attr->require_start <= UNW_PC(info) && | 72 | if (attr->require_start <= UNW_PC(info) && |
| 73 | UNW_PC(info) < attr->require_end) | 73 | UNW_PC(info) < attr->require_end) |
| 74 | found = 1; | 74 | found = true; |
| 75 | } | 75 | } |
| 76 | return found; | 76 | return found; |
| 77 | } | 77 | } |
| 78 | 78 | ||
| 79 | static int fail_stacktrace(struct fault_attr *attr) | 79 | static bool fail_stacktrace(struct fault_attr *attr) |
| 80 | { | 80 | { |
| 81 | struct unwind_frame_info info; | 81 | struct unwind_frame_info info; |
| 82 | 82 | ||
| @@ -85,9 +85,7 @@ static int fail_stacktrace(struct fault_attr *attr) | |||
| 85 | 85 | ||
| 86 | #elif defined(CONFIG_STACKTRACE) | 86 | #elif defined(CONFIG_STACKTRACE) |
| 87 | 87 | ||
| 88 | #define MAX_STACK_TRACE_DEPTH 32 | 88 | static bool fail_stacktrace(struct fault_attr *attr) |
| 89 | |||
| 90 | static int fail_stacktrace(struct fault_attr *attr) | ||
| 91 | { | 89 | { |
| 92 | struct stack_trace trace; | 90 | struct stack_trace trace; |
| 93 | int depth = attr->stacktrace_depth; | 91 | int depth = attr->stacktrace_depth; |
| @@ -109,26 +107,26 @@ static int fail_stacktrace(struct fault_attr *attr) | |||
| 109 | for (n = 0; n < trace.nr_entries; n++) { | 107 | for (n = 0; n < trace.nr_entries; n++) { |
| 110 | if (attr->reject_start <= entries[n] && | 108 | if (attr->reject_start <= entries[n] && |
| 111 | entries[n] < attr->reject_end) | 109 | entries[n] < attr->reject_end) |
| 112 | return 0; | 110 | return false; |
| 113 | if (attr->require_start <= entries[n] && | 111 | if (attr->require_start <= entries[n] && |
| 114 | entries[n] < attr->require_end) | 112 | entries[n] < attr->require_end) |
| 115 | found = 1; | 113 | found = true; |
| 116 | } | 114 | } |
| 117 | return found; | 115 | return found; |
| 118 | } | 116 | } |
| 119 | 117 | ||
| 120 | #else | 118 | #else |
| 121 | 119 | ||
| 122 | static inline int fail_stacktrace(struct fault_attr *attr) | 120 | static inline bool fail_stacktrace(struct fault_attr *attr) |
| 123 | { | 121 | { |
| 124 | static int firsttime = 1; | 122 | static bool firsttime = true; |
| 125 | 123 | ||
| 126 | if (firsttime) { | 124 | if (firsttime) { |
| 127 | printk(KERN_WARNING | 125 | printk(KERN_WARNING |
| 128 | "This architecture does not implement save_stack_trace()\n"); | 126 | "This architecture does not implement save_stack_trace()\n"); |
| 129 | firsttime = 0; | 127 | firsttime = false; |
| 130 | } | 128 | } |
| 131 | return 0; | 129 | return false; |
| 132 | } | 130 | } |
| 133 | 131 | ||
| 134 | #endif | 132 | #endif |
| @@ -138,32 +136,32 @@ static inline int fail_stacktrace(struct fault_attr *attr) | |||
| 138 | * http://www.nongnu.org/failmalloc/ | 136 | * http://www.nongnu.org/failmalloc/ |
| 139 | */ | 137 | */ |
| 140 | 138 | ||
| 141 | int should_fail(struct fault_attr *attr, ssize_t size) | 139 | bool should_fail(struct fault_attr *attr, ssize_t size) |
| 142 | { | 140 | { |
| 143 | if (attr->task_filter && !fail_task(attr, current)) | 141 | if (attr->task_filter && !fail_task(attr, current)) |
| 144 | return 0; | 142 | return false; |
| 145 | 143 | ||
| 146 | if (!fail_stacktrace(attr)) | 144 | if (!fail_stacktrace(attr)) |
| 147 | return 0; | 145 | return false; |
| 148 | 146 | ||
| 149 | if (atomic_read(&attr->times) == 0) | 147 | if (atomic_read(&attr->times) == 0) |
| 150 | return 0; | 148 | return false; |
| 151 | 149 | ||
| 152 | if (atomic_read(&attr->space) > size) { | 150 | if (atomic_read(&attr->space) > size) { |
| 153 | atomic_sub(size, &attr->space); | 151 | atomic_sub(size, &attr->space); |
| 154 | return 0; | 152 | return false; |
| 155 | } | 153 | } |
| 156 | 154 | ||
| 157 | if (attr->interval > 1) { | 155 | if (attr->interval > 1) { |
| 158 | attr->count++; | 156 | attr->count++; |
| 159 | if (attr->count % attr->interval) | 157 | if (attr->count % attr->interval) |
| 160 | return 0; | 158 | return false; |
| 161 | } | 159 | } |
| 162 | 160 | ||
| 163 | if (attr->probability > random32() % 100) | 161 | if (attr->probability > random32() % 100) |
| 164 | goto fail; | 162 | goto fail; |
| 165 | 163 | ||
| 166 | return 0; | 164 | return false; |
| 167 | 165 | ||
| 168 | fail: | 166 | fail: |
| 169 | fail_dump(attr); | 167 | fail_dump(attr); |
| @@ -171,7 +169,7 @@ fail: | |||
| 171 | if (atomic_read(&attr->times) != -1) | 169 | if (atomic_read(&attr->times) != -1) |
| 172 | atomic_dec_not_zero(&attr->times); | 170 | atomic_dec_not_zero(&attr->times); |
| 173 | 171 | ||
| 174 | return 1; | 172 | return true; |
| 175 | } | 173 | } |
| 176 | 174 | ||
| 177 | #ifdef CONFIG_FAULT_INJECTION_DEBUG_FS | 175 | #ifdef CONFIG_FAULT_INJECTION_DEBUG_FS |
