]> pilppa.com Git - linux-2.6-omap-h63xx.git/commitdiff
[SCSI] handle scsi_init_queue failure properly
authorFUJITA Tomonori <tomof@acm.org>
Fri, 25 Jan 2008 14:25:14 +0000 (23:25 +0900)
committerJames Bottomley <James.Bottomley@HansenPartnership.com>
Wed, 30 Jan 2008 19:14:25 +0000 (13:14 -0600)
scsi_init_queue is expected to clean up allocated things when it
fails.

Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
Signed-off-by: James Bottomley <James.Bottomley@HansenPartnership.com>
drivers/scsi/scsi_lib.c

index 7bfec7e7a8a0b2aff8f0c70c49b190e3b54a87b6..b12fb310e3999af497c21778b62f40c46892e8fd 100644 (file)
@@ -1682,7 +1682,7 @@ int __init scsi_init_queue(void)
                                        0, 0, NULL);
        if (!scsi_bidi_sdb_cache) {
                printk(KERN_ERR "SCSI: can't init scsi bidi sdb cache\n");
-               return -ENOMEM;
+               goto cleanup_io_context;
        }
 
        for (i = 0; i < SG_MEMPOOL_NR; i++) {
@@ -1694,6 +1694,7 @@ int __init scsi_init_queue(void)
                if (!sgp->slab) {
                        printk(KERN_ERR "SCSI: can't init sg slab %s\n",
                                        sgp->name);
+                       goto cleanup_bidi_sdb;
                }
 
                sgp->pool = mempool_create_slab_pool(SG_MEMPOOL_SIZE,
@@ -1701,10 +1702,25 @@ int __init scsi_init_queue(void)
                if (!sgp->pool) {
                        printk(KERN_ERR "SCSI: can't init sg mempool %s\n",
                                        sgp->name);
+                       goto cleanup_bidi_sdb;
                }
        }
 
        return 0;
+
+cleanup_bidi_sdb:
+       for (i = 0; i < SG_MEMPOOL_NR; i++) {
+               struct scsi_host_sg_pool *sgp = scsi_sg_pools + i;
+               if (sgp->pool)
+                       mempool_destroy(sgp->pool);
+               if (sgp->slab)
+                       kmem_cache_destroy(sgp->slab);
+       }
+       kmem_cache_destroy(scsi_bidi_sdb_cache);
+cleanup_io_context:
+       kmem_cache_destroy(scsi_io_context_cache);
+
+       return -ENOMEM;
 }
 
 void scsi_exit_queue(void)