revert "dpt_i2o: convert to SCSI hotplug model"
revert
commit 55d9fcf57b
Author: Matthew Wilcox <matthew@wil.cx>
Date: Mon Jul 30 15:19:18 2007 -0600
[SCSI] dpt_i2o: convert to SCSI hotplug model
- Delete refereces to HOSTS_C
- Switch to module_init/module_exit instead of detect/release
- Don't pass around the host template and rename it to adpt_template
- Switch from scsi_register/scsi_unregister to scsi_host_alloc,
scsi_add_host, scsi_scan_host and scsi_host_put.
Because it caused (for unknown reasons) Andres' all-data-reads-as-zeroes
problem, reported at
http://groups.google.com/group/fa.linux.kernel/msg/083a9acff0330234
Cc: Matthew Wilcox <matthew@wil.cx>
Cc: "Salyzyn, Mark" <mark_salyzyn@adaptec.com>
Cc: James Bottomley <James.Bottomley@SteelEye.com>
Acked-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
Cc: Anders Henke <anders.henke@1und1.de>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Cc: <stable@kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
This commit is contained in:
parent
43cbe2cbdd
commit
24601bbcac
2 changed files with 68 additions and 73 deletions
|
@ -173,20 +173,20 @@ static struct pci_device_id dptids[] = {
|
||||||
};
|
};
|
||||||
MODULE_DEVICE_TABLE(pci,dptids);
|
MODULE_DEVICE_TABLE(pci,dptids);
|
||||||
|
|
||||||
static void adpt_exit(void);
|
static int adpt_detect(struct scsi_host_template* sht)
|
||||||
|
|
||||||
static int adpt_detect(void)
|
|
||||||
{
|
{
|
||||||
struct pci_dev *pDev = NULL;
|
struct pci_dev *pDev = NULL;
|
||||||
adpt_hba* pHba;
|
adpt_hba* pHba;
|
||||||
|
|
||||||
|
adpt_init();
|
||||||
|
|
||||||
PINFO("Detecting Adaptec I2O RAID controllers...\n");
|
PINFO("Detecting Adaptec I2O RAID controllers...\n");
|
||||||
|
|
||||||
/* search for all Adatpec I2O RAID cards */
|
/* search for all Adatpec I2O RAID cards */
|
||||||
while ((pDev = pci_get_device( PCI_DPT_VENDOR_ID, PCI_ANY_ID, pDev))) {
|
while ((pDev = pci_get_device( PCI_DPT_VENDOR_ID, PCI_ANY_ID, pDev))) {
|
||||||
if(pDev->device == PCI_DPT_DEVICE_ID ||
|
if(pDev->device == PCI_DPT_DEVICE_ID ||
|
||||||
pDev->device == PCI_DPT_RAPTOR_DEVICE_ID){
|
pDev->device == PCI_DPT_RAPTOR_DEVICE_ID){
|
||||||
if(adpt_install_hba(pDev) ){
|
if(adpt_install_hba(sht, pDev) ){
|
||||||
PERROR("Could not Init an I2O RAID device\n");
|
PERROR("Could not Init an I2O RAID device\n");
|
||||||
PERROR("Will not try to detect others.\n");
|
PERROR("Will not try to detect others.\n");
|
||||||
return hba_count-1;
|
return hba_count-1;
|
||||||
|
@ -248,33 +248,34 @@ static int adpt_detect(void)
|
||||||
}
|
}
|
||||||
|
|
||||||
for (pHba = hba_chain; pHba; pHba = pHba->next) {
|
for (pHba = hba_chain; pHba; pHba = pHba->next) {
|
||||||
if (adpt_scsi_register(pHba) < 0) {
|
if( adpt_scsi_register(pHba,sht) < 0){
|
||||||
adpt_i2o_delete_hba(pHba);
|
adpt_i2o_delete_hba(pHba);
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
pHba->initialized = TRUE;
|
pHba->initialized = TRUE;
|
||||||
pHba->state &= ~DPTI_STATE_RESET;
|
pHba->state &= ~DPTI_STATE_RESET;
|
||||||
scsi_scan_host(pHba->host);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
// Register our control device node
|
// Register our control device node
|
||||||
// nodes will need to be created in /dev to access this
|
// nodes will need to be created in /dev to access this
|
||||||
// the nodes can not be created from within the driver
|
// the nodes can not be created from within the driver
|
||||||
if (hba_count && register_chrdev(DPTI_I2O_MAJOR, DPT_DRIVER, &adpt_fops)) {
|
if (hba_count && register_chrdev(DPTI_I2O_MAJOR, DPT_DRIVER, &adpt_fops)) {
|
||||||
adpt_exit();
|
adpt_i2o_sys_shutdown();
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
return hba_count;
|
return hba_count;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
static int adpt_release(adpt_hba *pHba)
|
/*
|
||||||
|
* scsi_unregister will be called AFTER we return.
|
||||||
|
*/
|
||||||
|
static int adpt_release(struct Scsi_Host *host)
|
||||||
{
|
{
|
||||||
struct Scsi_Host *shost = pHba->host;
|
adpt_hba* pHba = (adpt_hba*) host->hostdata[0];
|
||||||
scsi_remove_host(shost);
|
|
||||||
// adpt_i2o_quiesce_hba(pHba);
|
// adpt_i2o_quiesce_hba(pHba);
|
||||||
adpt_i2o_delete_hba(pHba);
|
adpt_i2o_delete_hba(pHba);
|
||||||
scsi_host_put(shost);
|
scsi_unregister(host);
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -881,7 +882,7 @@ static int adpt_reboot_event(struct notifier_block *n, ulong code, void *p)
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
|
|
||||||
static int adpt_install_hba(struct pci_dev* pDev)
|
static int adpt_install_hba(struct scsi_host_template* sht, struct pci_dev* pDev)
|
||||||
{
|
{
|
||||||
|
|
||||||
adpt_hba* pHba = NULL;
|
adpt_hba* pHba = NULL;
|
||||||
|
@ -1028,6 +1029,8 @@ static void adpt_i2o_delete_hba(adpt_hba* pHba)
|
||||||
|
|
||||||
|
|
||||||
mutex_lock(&adpt_configuration_lock);
|
mutex_lock(&adpt_configuration_lock);
|
||||||
|
// scsi_unregister calls our adpt_release which
|
||||||
|
// does a quiese
|
||||||
if(pHba->host){
|
if(pHba->host){
|
||||||
free_irq(pHba->host->irq, pHba);
|
free_irq(pHba->host->irq, pHba);
|
||||||
}
|
}
|
||||||
|
@ -1079,6 +1082,17 @@ static void adpt_i2o_delete_hba(adpt_hba* pHba)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
static int adpt_init(void)
|
||||||
|
{
|
||||||
|
printk("Loading Adaptec I2O RAID: Version " DPT_I2O_VERSION "\n");
|
||||||
|
#ifdef REBOOT_NOTIFIER
|
||||||
|
register_reboot_notifier(&adpt_reboot_notifier);
|
||||||
|
#endif
|
||||||
|
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
static struct adpt_device* adpt_find_device(adpt_hba* pHba, u32 chan, u32 id, u32 lun)
|
static struct adpt_device* adpt_find_device(adpt_hba* pHba, u32 chan, u32 id, u32 lun)
|
||||||
{
|
{
|
||||||
struct adpt_device* d;
|
struct adpt_device* d;
|
||||||
|
@ -2164,6 +2178,37 @@ static s32 adpt_scsi_to_i2o(adpt_hba* pHba, struct scsi_cmnd* cmd, struct adpt_d
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
static s32 adpt_scsi_register(adpt_hba* pHba,struct scsi_host_template * sht)
|
||||||
|
{
|
||||||
|
struct Scsi_Host *host = NULL;
|
||||||
|
|
||||||
|
host = scsi_register(sht, sizeof(adpt_hba*));
|
||||||
|
if (host == NULL) {
|
||||||
|
printk ("%s: scsi_register returned NULL\n",pHba->name);
|
||||||
|
return -1;
|
||||||
|
}
|
||||||
|
host->hostdata[0] = (unsigned long)pHba;
|
||||||
|
pHba->host = host;
|
||||||
|
|
||||||
|
host->irq = pHba->pDev->irq;
|
||||||
|
/* no IO ports, so don't have to set host->io_port and
|
||||||
|
* host->n_io_port
|
||||||
|
*/
|
||||||
|
host->io_port = 0;
|
||||||
|
host->n_io_port = 0;
|
||||||
|
/* see comments in scsi_host.h */
|
||||||
|
host->max_id = 16;
|
||||||
|
host->max_lun = 256;
|
||||||
|
host->max_channel = pHba->top_scsi_channel + 1;
|
||||||
|
host->cmd_per_lun = 1;
|
||||||
|
host->unique_id = (uint) pHba;
|
||||||
|
host->sg_tablesize = pHba->sg_tablesize;
|
||||||
|
host->can_queue = pHba->post_fifo_size;
|
||||||
|
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
static s32 adpt_i2o_to_scsi(void __iomem *reply, struct scsi_cmnd* cmd)
|
static s32 adpt_i2o_to_scsi(void __iomem *reply, struct scsi_cmnd* cmd)
|
||||||
{
|
{
|
||||||
adpt_hba* pHba;
|
adpt_hba* pHba;
|
||||||
|
@ -3279,10 +3324,12 @@ static static void adpt_delay(int millisec)
|
||||||
|
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
static struct scsi_host_template adpt_template = {
|
static struct scsi_host_template driver_template = {
|
||||||
.name = "dpt_i2o",
|
.name = "dpt_i2o",
|
||||||
.proc_name = "dpt_i2o",
|
.proc_name = "dpt_i2o",
|
||||||
.proc_info = adpt_proc_info,
|
.proc_info = adpt_proc_info,
|
||||||
|
.detect = adpt_detect,
|
||||||
|
.release = adpt_release,
|
||||||
.info = adpt_info,
|
.info = adpt_info,
|
||||||
.queuecommand = adpt_queue,
|
.queuecommand = adpt_queue,
|
||||||
.eh_abort_handler = adpt_abort,
|
.eh_abort_handler = adpt_abort,
|
||||||
|
@ -3297,62 +3344,5 @@ static struct scsi_host_template adpt_template = {
|
||||||
.use_clustering = ENABLE_CLUSTERING,
|
.use_clustering = ENABLE_CLUSTERING,
|
||||||
.use_sg_chaining = ENABLE_SG_CHAINING,
|
.use_sg_chaining = ENABLE_SG_CHAINING,
|
||||||
};
|
};
|
||||||
|
#include "scsi_module.c"
|
||||||
static s32 adpt_scsi_register(adpt_hba* pHba)
|
|
||||||
{
|
|
||||||
struct Scsi_Host *host;
|
|
||||||
|
|
||||||
host = scsi_host_alloc(&adpt_template, sizeof(adpt_hba*));
|
|
||||||
if (host == NULL) {
|
|
||||||
printk ("%s: scsi_host_alloc returned NULL\n",pHba->name);
|
|
||||||
return -1;
|
|
||||||
}
|
|
||||||
host->hostdata[0] = (unsigned long)pHba;
|
|
||||||
pHba->host = host;
|
|
||||||
|
|
||||||
host->irq = pHba->pDev->irq;
|
|
||||||
/* no IO ports, so don't have to set host->io_port and
|
|
||||||
* host->n_io_port
|
|
||||||
*/
|
|
||||||
host->io_port = 0;
|
|
||||||
host->n_io_port = 0;
|
|
||||||
/* see comments in scsi_host.h */
|
|
||||||
host->max_id = 16;
|
|
||||||
host->max_lun = 256;
|
|
||||||
host->max_channel = pHba->top_scsi_channel + 1;
|
|
||||||
host->cmd_per_lun = 1;
|
|
||||||
host->unique_id = (uint) pHba;
|
|
||||||
host->sg_tablesize = pHba->sg_tablesize;
|
|
||||||
host->can_queue = pHba->post_fifo_size;
|
|
||||||
|
|
||||||
if (scsi_add_host(host, &pHba->pDev->dev)) {
|
|
||||||
scsi_host_put(host);
|
|
||||||
return -1;
|
|
||||||
}
|
|
||||||
|
|
||||||
return 0;
|
|
||||||
}
|
|
||||||
|
|
||||||
static int __init adpt_init(void)
|
|
||||||
{
|
|
||||||
int count;
|
|
||||||
|
|
||||||
printk("Loading Adaptec I2O RAID: Version " DPT_I2O_VERSION "\n");
|
|
||||||
#ifdef REBOOT_NOTIFIER
|
|
||||||
register_reboot_notifier(&adpt_reboot_notifier);
|
|
||||||
#endif
|
|
||||||
|
|
||||||
count = adpt_detect();
|
|
||||||
|
|
||||||
return count > 0 ? 0 : -ENODEV;
|
|
||||||
}
|
|
||||||
|
|
||||||
static void adpt_exit(void)
|
|
||||||
{
|
|
||||||
while (hba_chain)
|
|
||||||
adpt_release(hba_chain);
|
|
||||||
}
|
|
||||||
|
|
||||||
module_init(adpt_init);
|
|
||||||
module_exit(adpt_exit);
|
|
||||||
MODULE_LICENSE("GPL");
|
MODULE_LICENSE("GPL");
|
||||||
|
|
|
@ -28,9 +28,11 @@
|
||||||
* SCSI interface function Prototypes
|
* SCSI interface function Prototypes
|
||||||
*/
|
*/
|
||||||
|
|
||||||
|
static int adpt_detect(struct scsi_host_template * sht);
|
||||||
static int adpt_queue(struct scsi_cmnd * cmd, void (*cmdcomplete) (struct scsi_cmnd *));
|
static int adpt_queue(struct scsi_cmnd * cmd, void (*cmdcomplete) (struct scsi_cmnd *));
|
||||||
static int adpt_abort(struct scsi_cmnd * cmd);
|
static int adpt_abort(struct scsi_cmnd * cmd);
|
||||||
static int adpt_reset(struct scsi_cmnd* cmd);
|
static int adpt_reset(struct scsi_cmnd* cmd);
|
||||||
|
static int adpt_release(struct Scsi_Host *host);
|
||||||
static int adpt_slave_configure(struct scsi_device *);
|
static int adpt_slave_configure(struct scsi_device *);
|
||||||
|
|
||||||
static const char *adpt_info(struct Scsi_Host *pSHost);
|
static const char *adpt_info(struct Scsi_Host *pSHost);
|
||||||
|
@ -47,6 +49,8 @@ static int adpt_device_reset(struct scsi_cmnd* cmd);
|
||||||
|
|
||||||
#define DPT_DRIVER_NAME "Adaptec I2O RAID"
|
#define DPT_DRIVER_NAME "Adaptec I2O RAID"
|
||||||
|
|
||||||
|
#ifndef HOSTS_C
|
||||||
|
|
||||||
#include "dpt/sys_info.h"
|
#include "dpt/sys_info.h"
|
||||||
#include <linux/wait.h>
|
#include <linux/wait.h>
|
||||||
#include "dpt/dpti_i2o.h"
|
#include "dpt/dpti_i2o.h"
|
||||||
|
@ -285,7 +289,7 @@ static s32 adpt_i2o_init_outbound_q(adpt_hba* pHba);
|
||||||
static s32 adpt_i2o_hrt_get(adpt_hba* pHba);
|
static s32 adpt_i2o_hrt_get(adpt_hba* pHba);
|
||||||
static s32 adpt_scsi_to_i2o(adpt_hba* pHba, struct scsi_cmnd* cmd, struct adpt_device* dptdevice);
|
static s32 adpt_scsi_to_i2o(adpt_hba* pHba, struct scsi_cmnd* cmd, struct adpt_device* dptdevice);
|
||||||
static s32 adpt_i2o_to_scsi(void __iomem *reply, struct scsi_cmnd* cmd);
|
static s32 adpt_i2o_to_scsi(void __iomem *reply, struct scsi_cmnd* cmd);
|
||||||
static s32 adpt_scsi_register(adpt_hba* pHba);
|
static s32 adpt_scsi_register(adpt_hba* pHba,struct scsi_host_template * sht);
|
||||||
static s32 adpt_hba_reset(adpt_hba* pHba);
|
static s32 adpt_hba_reset(adpt_hba* pHba);
|
||||||
static s32 adpt_i2o_reset_hba(adpt_hba* pHba);
|
static s32 adpt_i2o_reset_hba(adpt_hba* pHba);
|
||||||
static s32 adpt_rescan(adpt_hba* pHba);
|
static s32 adpt_rescan(adpt_hba* pHba);
|
||||||
|
@ -295,7 +299,7 @@ static void adpt_i2o_delete_hba(adpt_hba* pHba);
|
||||||
static void adpt_inquiry(adpt_hba* pHba);
|
static void adpt_inquiry(adpt_hba* pHba);
|
||||||
static void adpt_fail_posted_scbs(adpt_hba* pHba);
|
static void adpt_fail_posted_scbs(adpt_hba* pHba);
|
||||||
static struct adpt_device* adpt_find_device(adpt_hba* pHba, u32 chan, u32 id, u32 lun);
|
static struct adpt_device* adpt_find_device(adpt_hba* pHba, u32 chan, u32 id, u32 lun);
|
||||||
static int adpt_install_hba(struct pci_dev* pDev) ;
|
static int adpt_install_hba(struct scsi_host_template* sht, struct pci_dev* pDev) ;
|
||||||
static int adpt_i2o_online_hba(adpt_hba* pHba);
|
static int adpt_i2o_online_hba(adpt_hba* pHba);
|
||||||
static void adpt_i2o_post_wait_complete(u32, int);
|
static void adpt_i2o_post_wait_complete(u32, int);
|
||||||
static int adpt_i2o_systab_send(adpt_hba* pHba);
|
static int adpt_i2o_systab_send(adpt_hba* pHba);
|
||||||
|
@ -339,4 +343,5 @@ static void adpt_i386_info(sysInfo_S* si);
|
||||||
#define FW_DEBUG_BLED_OFFSET 8
|
#define FW_DEBUG_BLED_OFFSET 8
|
||||||
|
|
||||||
#define FW_DEBUG_FLAGS_NO_HEADERS_B 0x01
|
#define FW_DEBUG_FLAGS_NO_HEADERS_B 0x01
|
||||||
|
#endif /* !HOSTS_C */
|
||||||
#endif /* _DPT_H */
|
#endif /* _DPT_H */
|
||||||
|
|
Loading…
Reference in a new issue