diff options
| author | Eric Sesterhenn <snakebyte@gmx.de> | 2006-10-03 23:33:23 +0200 |
|---|---|---|
| committer | Adrian Bunk <bunk@stusta.de> | 2006-10-03 23:33:23 +0200 |
| commit | 52e5f9d1cf0b10b24317037dcd1c9be38ca7011c (patch) | |
| tree | a0646ee04b91d5d195075dc4ad8d38cf603179d8 | |
| parent | 70d63ccc71fe1a413ce82b88aa175b1dcf28654e (diff) | |
| download | op-kernel-dev-52e5f9d1cf0b10b24317037dcd1c9be38ca7011c.zip op-kernel-dev-52e5f9d1cf0b10b24317037dcd1c9be38ca7011c.tar.gz | |
BUG_ON cleanup for drivers/md/
This changes two if() BUG(); usages to BUG_ON(); so people
can disable it safely.
Signed-off-by: Eric Sesterhenn <snakebyte@gmx.de>
Signed-off-by: Adrian Bunk <bunk@stusta.de>
| -rw-r--r-- | drivers/md/md.c | 3 | ||||
| -rw-r--r-- | drivers/md/raid5.c | 2 |
2 files changed, 2 insertions, 3 deletions
diff --git a/drivers/md/md.c b/drivers/md/md.c index 38a0a57..cb82816 100644 --- a/drivers/md/md.c +++ b/drivers/md/md.c @@ -4438,8 +4438,7 @@ static int md_release(struct inode *inode, struct file * file) { mddev_t *mddev = inode->i_bdev->bd_disk->private_data; - if (!mddev) - BUG(); + BUG_ON(!mddev); mddev_put(mddev); return 0; diff --git a/drivers/md/raid5.c b/drivers/md/raid5.c index 37e4ff6..e14f457 100644 --- a/drivers/md/raid5.c +++ b/drivers/md/raid5.c @@ -1105,7 +1105,7 @@ static void compute_parity6(struct stripe_head *sh, int method) if (test_and_clear_bit(R5_Overlap, &sh->dev[i].flags)) wake_up(&conf->wait_for_overlap); - if (sh->dev[i].written) BUG(); + BUG_ON(sh->dev[i].written); sh->dev[i].written = chosen; } break; |
