diff options
author | Mike Christie <michaelc@cs.wisc.edu> | 2006-02-01 22:06:58 -0500 |
---|---|---|
committer | <jejb@mulgrave.il.steeleye.com> | 2006-02-04 17:18:27 -0500 |
commit | 142e301fc818de9b116706835cd9fc864e73f203 (patch) | |
tree | b37eb345383e77a6dc1bc27b33c2db22f5af0000 /drivers/scsi | |
parent | 5b940adf5b341b12dbb94e7cbdb416b35f52017b (diff) |
[SCSI] iscsi update: setup pool before using
>From andmike@us.ibm.com:
Ensure that pool data is setup prior to calling mempool_create as it will
call the the alloc function during create.
Signed-off-by: Mike Anderson <andmike@us.ibm.com>
Signed-off-by: Mike Christie <michaelc@cs.wisc.edu>
Signed-off-by: Alex Aizman <itn780@yahoo.com>
Signed-off-by: Dmitry Yusupov <dmitry_yus@yahoo.com>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
Diffstat (limited to 'drivers/scsi')
-rw-r--r-- | drivers/scsi/scsi_transport_iscsi.c | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/drivers/scsi/scsi_transport_iscsi.c b/drivers/scsi/scsi_transport_iscsi.c index 79ca29ee1aee..448fd78777f9 100644 --- a/drivers/scsi/scsi_transport_iscsi.c +++ b/drivers/scsi/scsi_transport_iscsi.c | |||
@@ -530,6 +530,12 @@ mempool_zone_init(unsigned max, unsigned size, unsigned hiwat) | |||
530 | if (!zp) | 530 | if (!zp) |
531 | return NULL; | 531 | return NULL; |
532 | 532 | ||
533 | zp->size = size; | ||
534 | zp->hiwat = hiwat; | ||
535 | INIT_LIST_HEAD(&zp->freequeue); | ||
536 | spin_lock_init(&zp->freelock); | ||
537 | atomic_set(&zp->allocated, 0); | ||
538 | |||
533 | zp->pool = mempool_create(max, mempool_zone_alloc_skb, | 539 | zp->pool = mempool_create(max, mempool_zone_alloc_skb, |
534 | mempool_zone_free_skb, zp); | 540 | mempool_zone_free_skb, zp); |
535 | if (!zp->pool) { | 541 | if (!zp->pool) { |
@@ -537,13 +543,6 @@ mempool_zone_init(unsigned max, unsigned size, unsigned hiwat) | |||
537 | return NULL; | 543 | return NULL; |
538 | } | 544 | } |
539 | 545 | ||
540 | zp->size = size; | ||
541 | zp->hiwat = hiwat; | ||
542 | |||
543 | INIT_LIST_HEAD(&zp->freequeue); | ||
544 | spin_lock_init(&zp->freelock); | ||
545 | atomic_set(&zp->allocated, 0); | ||
546 | |||
547 | return zp; | 546 | return zp; |
548 | } | 547 | } |
549 | 548 | ||