[PATCH] More BUG_ON conversion

Signed-off-by: Eric Sesterhenn <snakebyte@gmx.de>
Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
Cc: Bartlomiej Zolnierkiewicz <B.Zolnierkiewicz@elka.pw.edu.pl>
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: James Bottomley <James.Bottomley@steeleye.com>
Acked-by: "Salyzyn, Mark" <mark_salyzyn@adaptec.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>

2 files changed, 7 insertions, 14 deletions

diff --git a/drivers/scsi/aacraid/aachba.c b/drivers/scsi/aacraid/aachba.c
index ea9e038813eb..83b5c7d085f2 100644
--- a/drivers/scsi/aacraid/aachba.c
+++ b/drivers/scsi/aacraid/aachba.c
@@ -396,8 +396,7 @@ static void get_container_name_callback(void *context, struct fib * fibptr)
         scsicmd->SCp.phase = AAC_OWNER_MIDLEVEL;
 
         dprintk((KERN_DEBUG "get_container_name_callback[cpu %d]: t = %ld.\n", smp_processor_id(), jiffies));
-        if (fibptr == NULL)
-                BUG();
+        BUG_ON(fibptr == NULL);
 
         get_name_reply = (struct aac_get_name_resp *) fib_data(fibptr);
         /* Failure is irrelevant, using default value instead */
@@ -956,8 +955,7 @@ static void io_callback(void *context, struct fib * fibptr)
                   smp_processor_id(), (unsigned long long)lba, jiffies);
         }
 
-        if (fibptr == NULL)
-                BUG();
+        BUG_ON(fibptr == NULL);
                 
         if(scsicmd->use_sg)
                 pci_unmap_sg(dev->pdev, 
@@ -1092,8 +1090,7 @@ static int aac_read(struct scsi_cmnd * scsicmd, int cid)
                 
                 aac_build_sgraw(scsicmd, &readcmd->sg);
                 fibsize = sizeof(struct aac_raw_io) + ((le32_to_cpu(readcmd->sg.count) - 1) * sizeof (struct sgentryraw));
-                if (fibsize > (dev->max_fib_size - sizeof(struct aac_fibhdr)))
-                        BUG();
+                BUG_ON(fibsize > (dev->max_fib_size - sizeof(struct aac_fibhdr)));
                 /*
                  *      Now send the Fib to the adapter
                  */
@@ -1261,8 +1258,7 @@ static int aac_write(struct scsi_cmnd * scsicmd, int cid)
                 
                 aac_build_sgraw(scsicmd, &writecmd->sg);
                 fibsize = sizeof(struct aac_raw_io) + ((le32_to_cpu(writecmd->sg.count) - 1) * sizeof (struct sgentryraw));
-                if (fibsize > (dev->max_fib_size - sizeof(struct aac_fibhdr)))
-                        BUG();
+                BUG_ON(fibsize > (dev->max_fib_size - sizeof(struct aac_fibhdr)));
                 /*
                  *      Now send the Fib to the adapter
                  */
@@ -1904,8 +1900,7 @@ static void aac_srb_callback(void *context, struct fib * fibptr)
         scsicmd->SCp.phase = AAC_OWNER_MIDLEVEL;
         dev = (struct aac_dev *)scsicmd->device->host->hostdata;
 
-        if (fibptr == NULL)
-                BUG();
+        BUG_ON(fibptr == NULL);
 
         srbreply = (struct aac_srb_reply *) fib_data(fibptr);
 
diff --git a/drivers/scsi/aacraid/commsup.c b/drivers/scsi/aacraid/commsup.c
index d2ef17ea44fa..3f27419c66af 100644
--- a/drivers/scsi/aacraid/commsup.c
+++ b/drivers/scsi/aacraid/commsup.c
@@ -229,8 +229,7 @@ void aac_fib_init(struct fib *fibptr)
 static void fib_dealloc(struct fib * fibptr)
 {
         struct hw_fib *hw_fib = fibptr->hw_fib;
-        if(hw_fib->header.StructType != FIB_MAGIC) 
-                BUG();
+        BUG_ON(hw_fib->header.StructType != FIB_MAGIC);
         hw_fib->header.XferState = 0;        
 }
 
@@ -530,8 +529,7 @@ int aac_fib_send(u16 command, struct fib *fibptr, unsigned long size,
                         }
                 } else
                         down(&fibptr->event_wait);
-                if(fibptr->done == 0)
-                        BUG();
+                BUG_ON(fibptr->done == 0);
                         
                 if((fibptr->flags & FIB_CONTEXT_FLAG_TIMED_OUT)){
                         return -ETIMEDOUT;