From 52e5f9d1cf0b10b24317037dcd1c9be38ca7011c Mon Sep 17 00:00:00 2001 From: Eric Sesterhenn Date: Tue, 3 Oct 2006 23:33:23 +0200 Subject: [PATCH] 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 Signed-off-by: Adrian Bunk --- drivers/md/md.c | 3 +-- 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 38a0a5741d..cb8281605b 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 37e4ff661b..e14f457807 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; -- 2.39.5