diff options
author | Eric Sesterhenn <snakebyte@gmx.de> | 2006-03-31 18:20:23 -0500 |
---|---|---|
committer | Adrian Bunk <bunk@stusta.de> | 2006-03-31 18:20:23 -0500 |
commit | 9bc98fc6fd3d07f37b597601489df200e0024222 (patch) | |
tree | 06242303dc70efbeda901e9d0ca3a7067976a501 /ipc/util.c | |
parent | 99cee0cd7560fc4e7f3646ee18d90e328bd1cb32 (diff) |
BUG_ON() Conversion in ipc/util.c
this changes if() BUG(); constructs to BUG_ON() which is
cleaner, contains unlikely() and can better optimized away.
Signed-off-by: Eric Sesterhenn <snakebyte@gmx.de>
Signed-off-by: Adrian Bunk <bunk@stusta.de>
Diffstat (limited to 'ipc/util.c')
-rw-r--r-- | ipc/util.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/ipc/util.c b/ipc/util.c index 23151ef32590..5e785a29e1e6 100644 --- a/ipc/util.c +++ b/ipc/util.c | |||
@@ -266,8 +266,7 @@ struct kern_ipc_perm* ipc_rmid(struct ipc_ids* ids, int id) | |||
266 | { | 266 | { |
267 | struct kern_ipc_perm* p; | 267 | struct kern_ipc_perm* p; |
268 | int lid = id % SEQ_MULTIPLIER; | 268 | int lid = id % SEQ_MULTIPLIER; |
269 | if(lid >= ids->entries->size) | 269 | BUG_ON(lid >= ids->entries->size); |
270 | BUG(); | ||
271 | 270 | ||
272 | /* | 271 | /* |
273 | * do not need a rcu_dereference()() here to force ordering | 272 | * do not need a rcu_dereference()() here to force ordering |
@@ -275,8 +274,7 @@ struct kern_ipc_perm* ipc_rmid(struct ipc_ids* ids, int id) | |||
275 | */ | 274 | */ |
276 | p = ids->entries->p[lid]; | 275 | p = ids->entries->p[lid]; |
277 | ids->entries->p[lid] = NULL; | 276 | ids->entries->p[lid] = NULL; |
278 | if(p==NULL) | 277 | BUG_ON(p==NULL); |
279 | BUG(); | ||
280 | ids->in_use--; | 278 | ids->in_use--; |
281 | 279 | ||
282 | if (lid == ids->max_id) { | 280 | if (lid == ids->max_id) { |