diff options
author | Eric Sesterhenn <snakebyte@gmx.de> | 2006-04-02 07:34:29 -0400 |
---|---|---|
committer | Adrian Bunk <bunk@stusta.de> | 2006-04-02 07:34:29 -0400 |
commit | b6385483840e903d99cb753593faea215ae8d324 (patch) | |
tree | 9617999432507882287ef4cd2d9be1f20017d6ba /drivers | |
parent | 43dab9bbe9fda735589f2749004f639349966b20 (diff) |
BUG_ON() Conversion in md/raid10.c
this changes if() BUG(); constructs to BUG_ON() which is
cleaner and can better optimized away
Signed-off-by: Eric Sesterhenn <snakebyte@gmx.de>
Signed-off-by: Adrian Bunk <bunk@stusta.de>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/md/raid10.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/md/raid10.c b/drivers/md/raid10.c index ab90a6d12020..617012bc107a 100644 --- a/drivers/md/raid10.c +++ b/drivers/md/raid10.c | |||
@@ -1117,8 +1117,7 @@ static int end_sync_read(struct bio *bio, unsigned int bytes_done, int error) | |||
1117 | for (i=0; i<conf->copies; i++) | 1117 | for (i=0; i<conf->copies; i++) |
1118 | if (r10_bio->devs[i].bio == bio) | 1118 | if (r10_bio->devs[i].bio == bio) |
1119 | break; | 1119 | break; |
1120 | if (i == conf->copies) | 1120 | BUG_ON(i == conf->copies); |
1121 | BUG(); | ||
1122 | update_head_pos(i, r10_bio); | 1121 | update_head_pos(i, r10_bio); |
1123 | d = r10_bio->devs[i].devnum; | 1122 | d = r10_bio->devs[i].devnum; |
1124 | 1123 | ||
@@ -1518,8 +1517,7 @@ static int init_resync(conf_t *conf) | |||
1518 | int buffs; | 1517 | int buffs; |
1519 | 1518 | ||
1520 | buffs = RESYNC_WINDOW / RESYNC_BLOCK_SIZE; | 1519 | buffs = RESYNC_WINDOW / RESYNC_BLOCK_SIZE; |
1521 | if (conf->r10buf_pool) | 1520 | BUG_ON(conf->r10buf_pool); |
1522 | BUG(); | ||
1523 | conf->r10buf_pool = mempool_create(buffs, r10buf_pool_alloc, r10buf_pool_free, conf); | 1521 | conf->r10buf_pool = mempool_create(buffs, r10buf_pool_alloc, r10buf_pool_free, conf); |
1524 | if (!conf->r10buf_pool) | 1522 | if (!conf->r10buf_pool) |
1525 | return -ENOMEM; | 1523 | return -ENOMEM; |