diff options
author | Krishna Gudipati <kgudipat@brocade.com> | 2010-03-05 22:36:47 -0500 |
---|---|---|
committer | James Bottomley <James.Bottomley@suse.de> | 2010-03-07 02:31:23 -0500 |
commit | 2993cc71d1bff61999ade7f2b6b3ea2dd1e2c8d9 (patch) | |
tree | d4efc1c68195a0cbf4d30abcc891a78b76439755 /drivers/scsi/bfa/bfad.c | |
parent | 9693e7dff5c2911b4e445f5f656ef57b3a5bffac (diff) |
[SCSI] bfa: AEN and byte alignment fixes.
Replace enum types with int and rearrange the fields to fix some
alignment issue.
Local var ioc_attr is causing the stack to overflow, so removed the
usage of the local ioc_attr var and now invoking an API to return the
ioc_type.
Fix some AEN issues.
Signed-off-by: Krishna Gudipati <kgudipat@brocade.com>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
Diffstat (limited to 'drivers/scsi/bfa/bfad.c')
-rw-r--r-- | drivers/scsi/bfa/bfad.c | 9 |
1 files changed, 1 insertions, 8 deletions
diff --git a/drivers/scsi/bfa/bfad.c b/drivers/scsi/bfa/bfad.c index 4ccaeaecd14c..79956c152af9 100644 --- a/drivers/scsi/bfa/bfad.c +++ b/drivers/scsi/bfa/bfad.c | |||
@@ -677,7 +677,6 @@ bfad_drv_init(struct bfad_s *bfad) | |||
677 | bfa_status_t rc; | 677 | bfa_status_t rc; |
678 | unsigned long flags; | 678 | unsigned long flags; |
679 | struct bfa_fcs_driver_info_s driver_info; | 679 | struct bfa_fcs_driver_info_s driver_info; |
680 | int i; | ||
681 | 680 | ||
682 | bfad->cfg_data.rport_del_timeout = rport_del_timeout; | 681 | bfad->cfg_data.rport_del_timeout = rport_del_timeout; |
683 | bfad->cfg_data.lun_queue_depth = bfa_lun_queue_depth; | 682 | bfad->cfg_data.lun_queue_depth = bfa_lun_queue_depth; |
@@ -697,12 +696,7 @@ bfad_drv_init(struct bfad_s *bfad) | |||
697 | bfa_init_log(&bfad->bfa, bfad->logmod); | 696 | bfa_init_log(&bfad->bfa, bfad->logmod); |
698 | bfa_init_trc(&bfad->bfa, bfad->trcmod); | 697 | bfa_init_trc(&bfad->bfa, bfad->trcmod); |
699 | bfa_init_aen(&bfad->bfa, bfad->aen); | 698 | bfa_init_aen(&bfad->bfa, bfad->aen); |
700 | INIT_LIST_HEAD(&bfad->file_q); | 699 | memset(bfad->file_map, 0, sizeof(bfad->file_map)); |
701 | INIT_LIST_HEAD(&bfad->file_free_q); | ||
702 | for (i = 0; i < BFAD_AEN_MAX_APPS; i++) { | ||
703 | bfa_q_qe_init(&bfad->file_buf[i].qe); | ||
704 | list_add_tail(&bfad->file_buf[i].qe, &bfad->file_free_q); | ||
705 | } | ||
706 | bfa_init_plog(&bfad->bfa, &bfad->plog_buf); | 700 | bfa_init_plog(&bfad->bfa, &bfad->plog_buf); |
707 | bfa_plog_init(&bfad->plog_buf); | 701 | bfa_plog_init(&bfad->plog_buf); |
708 | bfa_plog_str(&bfad->plog_buf, BFA_PL_MID_DRVR, BFA_PL_EID_DRIVER_START, | 702 | bfa_plog_str(&bfad->plog_buf, BFA_PL_MID_DRVR, BFA_PL_EID_DRIVER_START, |
@@ -799,7 +793,6 @@ bfad_drv_uninit(struct bfad_s *bfad) | |||
799 | bfa_isr_disable(&bfad->bfa); | 793 | bfa_isr_disable(&bfad->bfa); |
800 | bfa_detach(&bfad->bfa); | 794 | bfa_detach(&bfad->bfa); |
801 | bfad_remove_intr(bfad); | 795 | bfad_remove_intr(bfad); |
802 | bfa_assert(list_empty(&bfad->file_q)); | ||
803 | bfad_hal_mem_release(bfad); | 796 | bfad_hal_mem_release(bfad); |
804 | 797 | ||
805 | bfad->bfad_flags &= ~BFAD_DRV_INIT_DONE; | 798 | bfad->bfad_flags &= ~BFAD_DRV_INIT_DONE; |