lkml.org 
[lkml]   [2001]   [Jan]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: [PATCH] make drivers/scsi/megaraid.c check ioremaps return code (241p9)
On Tue, Jan 23, 2001 at 12:45:20AM +0100, Rasmus Andersen wrote:

(This is another updated patch with the comments from the earlier mail
still valid.)

--- linux-ac10-clean/drivers/scsi/megaraid.c Sat Jan 20 15:17:13 2001
+++ linux-ac10/drivers/scsi/megaraid.c Tue Jan 23 23:06:14 2001
@@ -1491,15 +1491,19 @@
megaBase = pci_resource_start (pdev, 0);
megaIrq = pdev->irq;

- if (flag & BOARD_QUARTZ)
+ if (flag & BOARD_QUARTZ) {
megaBase = (long) ioremap (megaBase, 128);
+ if (!megaBase)
+ continue;
+ }
else
megaBase += 0x10;

/* Initialize SCSI Host structure */
host = scsi_register (pHostTmpl, sizeof (mega_host_config));
if(host == NULL)
- continue;
+ goto err_unmap;
+
megaCfg = (mega_host_config *) host->hostdata;
memset (megaCfg, 0, sizeof (mega_host_config));

@@ -1528,8 +1532,7 @@
/* Request our IO Range */
if (!request_region (megaBase, 16, "megaraid")) {
printk (KERN_WARNING "megaraid: Couldn't register I/O range!" CRLFSTR);
- scsi_unregister (host);
- continue;
+ goto err_unregister;
}
}

@@ -1538,8 +1541,7 @@
"megaraid", megaCfg)) {
printk (KERN_WARNING "megaraid: Couldn't register IRQ %d!" CRLFSTR,
megaIrq);
- scsi_unregister (host);
- continue;
+ goto err_release;
}

mega_register_mailbox (megaCfg, virt_to_bus ((void *) &megaCfg->mailbox64));
@@ -1585,6 +1587,16 @@
}

numFound++;
+ continue;
+
+ err_release:
+ if (flag != BOARD_QUARTZ)
+ release_region(megaBase, 16);
+ err_unregister:
+ scsi_unregister (host);
+ err_unmap:
+ if (flag & BOARD_QUARTZ)
+ iounmap((void *)megaBase);
}
return numFound;
}
--
Regards,
Rasmus(rasmus@jaquet.dk)

Half this game is ninety percent mental.
-Philadelphia Phillies manager Danny Ozark
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 13:18    [W:0.040 / U:0.180 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site