scsi: be2iscsi: Increase HDQ default queue size

Currently, ASYNC PDU default queue size is set to max connections.  This
leaves only one buffer per connection for any ASYNC PDUs from targets.

Double the size of the default queue.

Signed-off-by: Jitendra Bhivare <jitendra.bhivare@broadcom.com>
Reviewed-by: Tomas Henzl <thenzl@redhat.com>
Reviewed-by: Chris Leech <cleech@redhat.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
This commit is contained in:
Jitendra Bhivare 2017-03-24 14:11:44 +05:30 committed by Martin K. Petersen
parent 90e96313a9
commit fecc382469
2 changed files with 17 additions and 16 deletions

View File

@ -636,7 +636,6 @@ static void beiscsi_get_params(struct beiscsi_hba *phba)
(total_cid_count + (total_cid_count +
BE2_TMFS + BE2_NOPOUT_REQ)); BE2_TMFS + BE2_NOPOUT_REQ));
phba->params.cxns_per_ctrl = total_cid_count; phba->params.cxns_per_ctrl = total_cid_count;
phba->params.asyncpdus_per_ctrl = total_cid_count;
phba->params.icds_per_ctrl = total_icd_count; phba->params.icds_per_ctrl = total_icd_count;
phba->params.num_sge_per_io = BE2_SGE; phba->params.num_sge_per_io = BE2_SGE;
phba->params.defpdu_hdr_sz = BE2_DEFPDU_HDR_SZ; phba->params.defpdu_hdr_sz = BE2_DEFPDU_HDR_SZ;
@ -2407,22 +2406,22 @@ static void beiscsi_find_mem_req(struct beiscsi_hba *phba)
if (test_bit(ulp_num, &phba->fw_config.ulp_supported)) { if (test_bit(ulp_num, &phba->fw_config.ulp_supported)) {
num_async_pdu_buf_sgl_pages = num_async_pdu_buf_sgl_pages =
PAGES_REQUIRED(BEISCSI_GET_CID_COUNT( PAGES_REQUIRED(BEISCSI_ASYNC_HDQ_SIZE(
phba, ulp_num) * phba, ulp_num) *
sizeof(struct phys_addr)); sizeof(struct phys_addr));
num_async_pdu_buf_pages = num_async_pdu_buf_pages =
PAGES_REQUIRED(BEISCSI_GET_CID_COUNT( PAGES_REQUIRED(BEISCSI_ASYNC_HDQ_SIZE(
phba, ulp_num) * phba, ulp_num) *
phba->params.defpdu_hdr_sz); phba->params.defpdu_hdr_sz);
num_async_pdu_data_pages = num_async_pdu_data_pages =
PAGES_REQUIRED(BEISCSI_GET_CID_COUNT( PAGES_REQUIRED(BEISCSI_ASYNC_HDQ_SIZE(
phba, ulp_num) * phba, ulp_num) *
phba->params.defpdu_data_sz); phba->params.defpdu_data_sz);
num_async_pdu_data_sgl_pages = num_async_pdu_data_sgl_pages =
PAGES_REQUIRED(BEISCSI_GET_CID_COUNT( PAGES_REQUIRED(BEISCSI_ASYNC_HDQ_SIZE(
phba, ulp_num) * phba, ulp_num) *
sizeof(struct phys_addr)); sizeof(struct phys_addr));
@ -2459,21 +2458,21 @@ static void beiscsi_find_mem_req(struct beiscsi_hba *phba)
mem_descr_index = (HWI_MEM_ASYNC_HEADER_HANDLE_ULP0 + mem_descr_index = (HWI_MEM_ASYNC_HEADER_HANDLE_ULP0 +
(ulp_num * MEM_DESCR_OFFSET)); (ulp_num * MEM_DESCR_OFFSET));
phba->mem_req[mem_descr_index] = phba->mem_req[mem_descr_index] =
BEISCSI_GET_CID_COUNT(phba, ulp_num) * BEISCSI_ASYNC_HDQ_SIZE(phba, ulp_num) *
sizeof(struct hd_async_handle); sizeof(struct hd_async_handle);
mem_descr_index = (HWI_MEM_ASYNC_DATA_HANDLE_ULP0 + mem_descr_index = (HWI_MEM_ASYNC_DATA_HANDLE_ULP0 +
(ulp_num * MEM_DESCR_OFFSET)); (ulp_num * MEM_DESCR_OFFSET));
phba->mem_req[mem_descr_index] = phba->mem_req[mem_descr_index] =
BEISCSI_GET_CID_COUNT(phba, ulp_num) * BEISCSI_ASYNC_HDQ_SIZE(phba, ulp_num) *
sizeof(struct hd_async_handle); sizeof(struct hd_async_handle);
mem_descr_index = (HWI_MEM_ASYNC_PDU_CONTEXT_ULP0 + mem_descr_index = (HWI_MEM_ASYNC_PDU_CONTEXT_ULP0 +
(ulp_num * MEM_DESCR_OFFSET)); (ulp_num * MEM_DESCR_OFFSET));
phba->mem_req[mem_descr_index] = phba->mem_req[mem_descr_index] =
sizeof(struct hd_async_context) + sizeof(struct hd_async_context) +
(BEISCSI_GET_CID_COUNT(phba, ulp_num) * (BEISCSI_ASYNC_HDQ_SIZE(phba, ulp_num) *
sizeof(struct hd_async_entry)); sizeof(struct hd_async_entry));
} }
} }
} }
@ -2757,7 +2756,7 @@ static int hwi_init_async_pdu_ctx(struct beiscsi_hba *phba)
((long unsigned int)pasync_ctx + ((long unsigned int)pasync_ctx +
sizeof(struct hd_async_context)); sizeof(struct hd_async_context));
pasync_ctx->num_entries = BEISCSI_GET_CID_COUNT(phba, pasync_ctx->num_entries = BEISCSI_ASYNC_HDQ_SIZE(phba,
ulp_num); ulp_num);
/* setup header buffers */ /* setup header buffers */
mem_descr = (struct be_mem_descriptor *)phba->init_mem; mem_descr = (struct be_mem_descriptor *)phba->init_mem;
@ -2895,7 +2894,7 @@ static int hwi_init_async_pdu_ctx(struct beiscsi_hba *phba)
phba->params.defpdu_data_sz); phba->params.defpdu_data_sz);
num_per_mem = 0; num_per_mem = 0;
for (index = 0; index < BEISCSI_GET_CID_COUNT for (index = 0; index < BEISCSI_ASYNC_HDQ_SIZE
(phba, ulp_num); index++) { (phba, ulp_num); index++) {
pasync_header_h->cri = -1; pasync_header_h->cri = -1;
pasync_header_h->is_header = 1; pasync_header_h->is_header = 1;
@ -3772,7 +3771,7 @@ static int hwi_init_port(struct beiscsi_hba *phba)
for (ulp_num = 0; ulp_num < BEISCSI_ULP_COUNT; ulp_num++) { for (ulp_num = 0; ulp_num < BEISCSI_ULP_COUNT; ulp_num++) {
if (test_bit(ulp_num, &phba->fw_config.ulp_supported)) { if (test_bit(ulp_num, &phba->fw_config.ulp_supported)) {
def_pdu_ring_sz = def_pdu_ring_sz =
BEISCSI_GET_CID_COUNT(phba, ulp_num) * BEISCSI_ASYNC_HDQ_SIZE(phba, ulp_num) *
sizeof(struct phys_addr); sizeof(struct phys_addr);
status = beiscsi_create_def_hdr(phba, phwi_context, status = beiscsi_create_def_hdr(phba, phwi_context,

View File

@ -235,7 +235,6 @@ struct sgl_handle {
struct hba_parameters { struct hba_parameters {
unsigned int ios_per_ctrl; unsigned int ios_per_ctrl;
unsigned int cxns_per_ctrl; unsigned int cxns_per_ctrl;
unsigned int asyncpdus_per_ctrl;
unsigned int icds_per_ctrl; unsigned int icds_per_ctrl;
unsigned int num_sge_per_io; unsigned int num_sge_per_io;
unsigned int defpdu_hdr_sz; unsigned int defpdu_hdr_sz;
@ -599,6 +598,9 @@ struct hd_async_handle {
u8 is_final; u8 is_final;
}; };
#define BEISCSI_ASYNC_HDQ_SIZE(phba, ulp) \
(BEISCSI_GET_CID_COUNT((phba), (ulp)) * 2)
/** /**
* This has list of async PDUs that are waiting to be processed. * This has list of async PDUs that are waiting to be processed.
* Buffers live in this list for a brief duration before they get * Buffers live in this list for a brief duration before they get