Skip to content

Commit

Permalink
[SCSI] be2iscsi: Fix max supported EQ count to 8.
Browse files Browse the repository at this point in the history
The maximum EQ that can be created for a function is 8. Check the
CPU online count and create only 8 EQ if CPU_Count >= 8

Signed-off-by: John Soni Jose <sony.john-n@emulex.com>
Signed-off-by: Jayamohan Kallickal <jayamohan.kallickal@emulex.com>
Reviewed-by: Mike Christie <michaelc@cs.wisc.edu>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>
  • Loading branch information
John Soni Jose authored and James Bottomley committed Nov 27, 2012
1 parent d629c47 commit 107dfcb
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 9 deletions.
23 changes: 14 additions & 9 deletions drivers/scsi/be2iscsi/be_main.c
Original file line number Diff line number Diff line change
Expand Up @@ -2890,7 +2890,7 @@ static int beiscsi_create_eqs(struct beiscsi_hba *phba,
}
return 0;
create_eq_error:
for (i = 0; i < (phba->num_cpus + 1); i++) {
for (i = 0; i < (phba->num_cpus + eq_for_mcc); i++) {
eq = &phwi_context->be_eq[i].q;
mem = &eq->dma_mem;
if (mem->va)
Expand Down Expand Up @@ -3315,15 +3315,20 @@ static int be_mcc_queues_create(struct beiscsi_hba *phba,
return -ENOMEM;
}

static int find_num_cpus(void)
/**
* find_num_cpus()- Get the CPU online count
* @phba: ptr to priv structure
*
* CPU count is used for creating EQ.
**/
static void find_num_cpus(struct beiscsi_hba *phba)
{
int num_cpus = 0;

num_cpus = num_online_cpus();
if (num_cpus >= MAX_CPUS)
num_cpus = MAX_CPUS - 1;

return num_cpus;
phba->num_cpus = (num_cpus >= BEISCSI_MAX_NUM_CPU) ?
(BEISCSI_MAX_NUM_CPU - 1) : num_cpus;
}

static int hwi_init_port(struct beiscsi_hba *phba)
Expand Down Expand Up @@ -4542,7 +4547,7 @@ static int __devinit beiscsi_dev_probe(struct pci_dev *pcidev,
struct hwi_controller *phwi_ctrlr;
struct hwi_context_memory *phwi_context;
struct be_eq_obj *pbe_eq;
int ret, num_cpus, i;
int ret, i;
u8 *real_offset = 0;
u32 value = 0;

Expand Down Expand Up @@ -4578,10 +4583,10 @@ static int __devinit beiscsi_dev_probe(struct pci_dev *pcidev,
}

if (enable_msix)
num_cpus = find_num_cpus();
find_num_cpus(phba);
else
num_cpus = 1;
phba->num_cpus = num_cpus;
phba->num_cpus = 1;

beiscsi_log(phba, KERN_INFO, BEISCSI_LOG_INIT,
"BM_%d : num_cpus = %d\n",
phba->num_cpus);
Expand Down
1 change: 1 addition & 0 deletions drivers/scsi/be2iscsi/be_main.h
Original file line number Diff line number Diff line change
Expand Up @@ -61,6 +61,7 @@
#define BE2_DEFPDU_DATA_SZ 8192

#define MAX_CPUS 31
#define BEISCSI_MAX_NUM_CPU 8
#define BEISCSI_SGLIST_ELEMENTS 30

#define BEISCSI_CMD_PER_LUN 128 /* scsi_host->cmd_per_lun */
Expand Down

0 comments on commit 107dfcb

Please sign in to comment.