Re: [ANNOUNCE] megaraid linux driver version 2.00.7

From: 'Christoph Hellwig' (hch@infradead.org)
Date: Mon Aug 04 2003 - 15:47:06 EST


On Mon, Aug 04, 2003 at 02:18:59PM -0400, Mukker, Atul wrote:
> > and incorporating my patch to kill
> > the useless hostlock redefintion?
> >
> ??, can you resend this patch

diff -Nru a/drivers/scsi/megaraid.c b/drivers/scsi/megaraid.c
--- a/drivers/scsi/megaraid.c Mon Jun 23 11:03:07 2003
+++ b/drivers/scsi/megaraid.c Mon Jun 23 11:03:07 2003
@@ -32,15 +32,16 @@
 
 #include <linux/mm.h>
 #include <linux/fs.h>
-#include <linux/blk.h>
-#include <asm/uaccess.h>
-#include <asm/io.h>
+#include <linux/blkdev.h>
+#include <linux/spinlock.h>
 #include <linux/delay.h>
 #include <linux/reboot.h>
 #include <linux/module.h>
 #include <linux/list.h>
 #include <linux/interrupt.h>
 #include <scsi/scsicam.h>
+#include <asm/uaccess.h>
+#include <asm/io.h>
 
 #include "scsi.h"
 #include "hosts.h"
@@ -368,8 +369,6 @@
                 INIT_LIST_HEAD(&adapter->completed_list);
 
                 adapter->flag = flag;
- spin_lock_init(&adapter->lock);
- scsi_assign_lock(host, &adapter->lock);
 
                 host->cmd_per_lun = max_cmd_per_lun;
                 host->max_sectors = max_sectors_per_io;
@@ -1754,7 +1753,7 @@
         /*
          * loop till F/W has more commands for us to complete.
          */
- spin_lock_irqsave(&adapter->lock, flags);
+ spin_lock_irqsave(adapter->host->host_lock, flags);
 
         do {
                 /* Check if a valid interrupt is pending */
@@ -1800,7 +1799,7 @@
 
  out_unlock:
 
- spin_unlock_irqrestore(&adapter->lock, flags);
+ spin_unlock_irqrestore(adapter->host->host_lock, flags);
 
         return IRQ_RETVAL(handled);
 }
@@ -1831,7 +1830,7 @@
         /*
          * loop till F/W has more commands for us to complete.
          */
- spin_lock_irqsave(&adapter->lock, flags);
+ spin_lock_irqsave(adapter->host->host_lock, flags);
 
         do {
                 /* Check if a valid interrupt is pending */
@@ -1880,7 +1879,7 @@
 
  out_unlock:
 
- spin_unlock_irqrestore(&adapter->lock, flags);
+ spin_unlock_irqrestore(adapter->host->host_lock, flags);
 
         return IRQ_RETVAL(handled);
 }
@@ -2608,7 +2607,7 @@
         mc.cmd = MEGA_CLUSTER_CMD;
         mc.opcode = MEGA_RESET_RESERVATIONS;
 
- spin_unlock_irq(&adapter->lock);
+ spin_unlock_irq(adapter->host->host_lock);
         if( mega_internal_command(adapter, LOCK_INT, &mc, NULL) != 0 ) {
                 printk(KERN_WARNING
                                 "megaraid: reservation reset failed.\n");
@@ -2616,7 +2615,7 @@
         else {
                 printk(KERN_INFO "megaraid: reservation reset.\n");
         }
- spin_lock_irq(&adapter->lock);
+ spin_lock_irq(adapter->host->host_lock);
 #endif
 
         rval = megaraid_abort_and_reset(adapter, cmd, SCB_RESET);
@@ -4860,7 +4859,7 @@
 
         rval = mega_do_del_logdrv(adapter, logdrv);
 
- spin_lock_irqsave(&adapter->lock, flags);
+ spin_lock_irqsave(adapter->host->host_lock, flags);
 
         /*
          * If delete operation was successful, add 0x80 to the logical drive
@@ -4879,7 +4878,7 @@
 
         mega_runpendq(adapter);
 
- spin_unlock_irqrestore(&adapter->lock, flags);
+ spin_unlock_irqrestore(adapter->host->host_lock, flags);
 
         return rval;
 }
@@ -5258,11 +5257,11 @@
         /*
          * Get the lock only if the caller has not acquired it already
          */
- if( ls == LOCK_INT ) spin_lock_irqsave(&adapter->lock, flags);
-
+ if( ls == LOCK_INT )
+ spin_lock_irqsave(adapter->host->host_lock, flags);
         megaraid_queue(scmd, mega_internal_done);
-
- if( ls == LOCK_INT ) spin_unlock_irqrestore(&adapter->lock, flags);
+ if( ls == LOCK_INT )
+ spin_unlock_irqrestore(adapter->host->host_lock, flags);
 
         /*
          * Wait till this command finishes. Do not use
diff -Nru a/drivers/scsi/megaraid.h b/drivers/scsi/megaraid.h
--- a/drivers/scsi/megaraid.h Mon Jun 23 11:03:07 2003
+++ b/drivers/scsi/megaraid.h Mon Jun 23 11:03:07 2003
@@ -1,10 +1,6 @@
 #ifndef __MEGARAID_H__
 #define __MEGARAID_H__
 
-#include <linux/version.h>
-#include <linux/spinlock.h>
-
-
 #define MEGARAID_VERSION \
         "v2.00.3 (Release Date: Wed Feb 19 08:51:30 EST 2003)\n"
 
@@ -879,8 +875,6 @@
                                            drive needs to be done. Stop
                                            sending requests to the hba till
                                            delete operation is completed */
- spinlock_t lock;
-
         u8 logdrv_chan[MAX_CHANNELS+NVIRT_CHAN]; /* logical drive are on
                                                         what channels. */
         int mega_ch_class;
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Thu Aug 07 2003 - 22:00:25 EST