summaryrefslogtreecommitdiffstats
path: root/drivers/md
diff options
context:
space:
mode:
authorEric Sesterhenn <snakebyte@gmx.de>2006-10-03 23:33:23 +0200
committerAdrian Bunk <bunk@stusta.de>2006-10-03 23:33:23 +0200
commit52e5f9d1cf0b10b24317037dcd1c9be38ca7011c (patch)
treea0646ee04b91d5d195075dc4ad8d38cf603179d8 /drivers/md
parent70d63ccc71fe1a413ce82b88aa175b1dcf28654e (diff)
downloadop-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>
Diffstat (limited to 'drivers/md')
-rw-r--r--drivers/md/md.c3
-rw-r--r--drivers/md/raid5.c2
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;
OpenPOWER on IntegriCloud