Commit 42d7c10f authored by Manish Rangankar's avatar Manish Rangankar Committed by Martin K. Petersen

scsi: qedi: Limit number for CQ queues.

[qed_sp_iscsi_func_start:189(host_7-0)]Cannot satisfy CQ amount. Queues
requested 8, CQs available 4. Aborting function start

Above condition will resolve as management firmware is capable of
telling us the number of CQs available for a given PF, qed will
communicate the same number to qedi, So that qedi will know how much CQs
are allowed.
Signed-off-by: default avatarManish Rangankar <manish.rangankar@cavium.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent 30b67de3
...@@ -54,8 +54,8 @@ struct qedi_endpoint; ...@@ -54,8 +54,8 @@ struct qedi_endpoint;
/* MAX Length for cached SGL */ /* MAX Length for cached SGL */
#define MAX_SGLEN_FOR_CACHESGL ((1U << 16) - 1) #define MAX_SGLEN_FOR_CACHESGL ((1U << 16) - 1)
#define MAX_NUM_MSIX_PF 8 #define MIN_NUM_CPUS_MSIX(x) min_t(u32, x->dev_info.num_cqs, \
#define MIN_NUM_CPUS_MSIX(x) min((x)->msix_count, num_online_cpus()) num_online_cpus())
#define QEDI_LOCAL_PORT_MIN 60000 #define QEDI_LOCAL_PORT_MIN 60000
#define QEDI_LOCAL_PORT_MAX 61024 #define QEDI_LOCAL_PORT_MAX 61024
...@@ -301,7 +301,6 @@ struct qedi_ctx { ...@@ -301,7 +301,6 @@ struct qedi_ctx {
u16 bdq_prod_idx; u16 bdq_prod_idx;
u16 rq_num_entries; u16 rq_num_entries;
u32 msix_count;
u32 max_sqes; u32 max_sqes;
u8 num_queues; u8 num_queues;
u32 max_active_conns; u32 max_active_conns;
......
...@@ -534,7 +534,7 @@ static int qedi_iscsi_offload_conn(struct qedi_endpoint *qedi_ep) ...@@ -534,7 +534,7 @@ static int qedi_iscsi_offload_conn(struct qedi_endpoint *qedi_ep)
SET_FIELD(conn_info->tcp_flags, TCP_OFFLOAD_PARAMS_DA_CNT_EN, 1); SET_FIELD(conn_info->tcp_flags, TCP_OFFLOAD_PARAMS_DA_CNT_EN, 1);
SET_FIELD(conn_info->tcp_flags, TCP_OFFLOAD_PARAMS_KA_EN, 1); SET_FIELD(conn_info->tcp_flags, TCP_OFFLOAD_PARAMS_KA_EN, 1);
conn_info->default_cq = (qedi_ep->fw_cid % 8); conn_info->default_cq = (qedi_ep->fw_cid % qedi->num_queues);
conn_info->ka_max_probe_cnt = DEF_KA_MAX_PROBE_COUNT; conn_info->ka_max_probe_cnt = DEF_KA_MAX_PROBE_COUNT;
conn_info->dup_ack_theshold = 3; conn_info->dup_ack_theshold = 3;
......
...@@ -794,13 +794,14 @@ static int qedi_set_iscsi_pf_param(struct qedi_ctx *qedi) ...@@ -794,13 +794,14 @@ static int qedi_set_iscsi_pf_param(struct qedi_ctx *qedi)
u32 log_page_size; u32 log_page_size;
int rval = 0; int rval = 0;
QEDI_INFO(&qedi->dbg_ctx, QEDI_LOG_DISC, "Min number of MSIX %d\n",
MIN_NUM_CPUS_MSIX(qedi));
num_sq_pages = (MAX_OUSTANDING_TASKS_PER_CON * 8) / PAGE_SIZE; num_sq_pages = (MAX_OUSTANDING_TASKS_PER_CON * 8) / PAGE_SIZE;
qedi->num_queues = MIN_NUM_CPUS_MSIX(qedi); qedi->num_queues = MIN_NUM_CPUS_MSIX(qedi);
QEDI_INFO(&qedi->dbg_ctx, QEDI_LOG_INFO,
"Number of CQ count is %d\n", qedi->num_queues);
memset(&qedi->pf_params.iscsi_pf_params, 0, memset(&qedi->pf_params.iscsi_pf_params, 0,
sizeof(qedi->pf_params.iscsi_pf_params)); sizeof(qedi->pf_params.iscsi_pf_params));
...@@ -2179,9 +2180,12 @@ static int __qedi_probe(struct pci_dev *pdev, int mode) ...@@ -2179,9 +2180,12 @@ static int __qedi_probe(struct pci_dev *pdev, int mode)
goto free_host; goto free_host;
} }
qedi->msix_count = MAX_NUM_MSIX_PF;
atomic_set(&qedi->link_state, QEDI_LINK_DOWN); atomic_set(&qedi->link_state, QEDI_LINK_DOWN);
rc = qedi_ops->fill_dev_info(qedi->cdev, &qedi->dev_info);
if (rc)
goto free_host;
if (mode != QEDI_MODE_RECOVERY) { if (mode != QEDI_MODE_RECOVERY) {
rc = qedi_set_iscsi_pf_param(qedi); rc = qedi_set_iscsi_pf_param(qedi);
if (rc) { if (rc) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment