net/qede/base: add/change/revise logs
Changes include: - adding new log to print Dcbx version - reformatting log in ecore_dmae_operation_wait() in case of DMA engine failure - changing verbosity of some log messages such as: VFs incorrect behavior should be logged on PF with DP_VERBOSE(), not DP_NOTICE(). In general keep IOV-related logs at low verbosity. Log the critical issues from VF perspective, like message to PF times-out or the PF rejects the VF configuration, as NOTICE than VERBOSE - Add a printout of some MCP CPU info in case of no MFW response Signed-off-by: Rasesh Mody <rasesh.mody@cavium.com>
This commit is contained in:
parent
aa96bcbdae
commit
49f4b9dc8a
@ -1164,6 +1164,9 @@ ecore_dcbx_set_local_params(struct ecore_hwfn *p_hwfn,
|
||||
local_admin->config = DCBX_CONFIG_VERSION_DISABLED;
|
||||
}
|
||||
|
||||
DP_VERBOSE(p_hwfn, ECORE_MSG_DCB, "Dcbx version = %d\n",
|
||||
local_admin->config);
|
||||
|
||||
if (params->override_flags & ECORE_DCBX_OVERRIDE_PFC_CFG)
|
||||
ecore_dcbx_set_pfc_data(p_hwfn, &local_admin->features.pfc,
|
||||
¶ms->config.params);
|
||||
|
@ -744,10 +744,10 @@ ecore_dmae_execute_sub_operation(struct ecore_hwfn *p_hwfn,
|
||||
|
||||
if (ecore_status != ECORE_SUCCESS) {
|
||||
DP_NOTICE(p_hwfn, ECORE_MSG_HW,
|
||||
"ecore_dmae_host2grc: Wait Failed. source_addr"
|
||||
" 0x%lx, grc_addr 0x%lx, size_in_dwords 0x%x\n",
|
||||
"Wait Failed. source_addr 0x%lx, grc_addr 0x%lx, size_in_dwords 0x%x, intermediate buffer 0x%lx.\n",
|
||||
(unsigned long)src_addr, (unsigned long)dst_addr,
|
||||
length_dw);
|
||||
length_dw,
|
||||
(unsigned long)p_hwfn->dmae_info.intermediate_buffer_phys_addr);
|
||||
return ecore_status;
|
||||
}
|
||||
|
||||
|
@ -447,6 +447,24 @@ static void ecore_mcp_cmd_set_blocking(struct ecore_hwfn *p_hwfn,
|
||||
block_cmd ? "Block" : "Unblock");
|
||||
}
|
||||
|
||||
void ecore_mcp_print_cpu_info(struct ecore_hwfn *p_hwfn,
|
||||
struct ecore_ptt *p_ptt)
|
||||
{
|
||||
u32 cpu_mode, cpu_state, cpu_pc_0, cpu_pc_1, cpu_pc_2;
|
||||
|
||||
cpu_mode = ecore_rd(p_hwfn, p_ptt, MCP_REG_CPU_MODE);
|
||||
cpu_state = ecore_rd(p_hwfn, p_ptt, MCP_REG_CPU_STATE);
|
||||
cpu_pc_0 = ecore_rd(p_hwfn, p_ptt, MCP_REG_CPU_PROGRAM_COUNTER);
|
||||
OSAL_UDELAY(CHIP_MCP_RESP_ITER_US);
|
||||
cpu_pc_1 = ecore_rd(p_hwfn, p_ptt, MCP_REG_CPU_PROGRAM_COUNTER);
|
||||
OSAL_UDELAY(CHIP_MCP_RESP_ITER_US);
|
||||
cpu_pc_2 = ecore_rd(p_hwfn, p_ptt, MCP_REG_CPU_PROGRAM_COUNTER);
|
||||
|
||||
DP_NOTICE(p_hwfn, false,
|
||||
"MCP CPU info: mode 0x%08x, state 0x%08x, pc {0x%08x, 0x%08x, 0x%08x}\n",
|
||||
cpu_mode, cpu_state, cpu_pc_0, cpu_pc_1, cpu_pc_2);
|
||||
}
|
||||
|
||||
static enum _ecore_status_t
|
||||
_ecore_mcp_cmd_and_union(struct ecore_hwfn *p_hwfn, struct ecore_ptt *p_ptt,
|
||||
struct ecore_mcp_mb_params *p_mb_params,
|
||||
@ -526,6 +544,7 @@ _ecore_mcp_cmd_and_union(struct ecore_hwfn *p_hwfn, struct ecore_ptt *p_ptt,
|
||||
DP_NOTICE(p_hwfn, false,
|
||||
"The MFW failed to respond to command 0x%08x [param 0x%08x].\n",
|
||||
p_mb_params->cmd, p_mb_params->param);
|
||||
ecore_mcp_print_cpu_info(p_hwfn, p_ptt);
|
||||
|
||||
OSAL_SPIN_LOCK(&p_hwfn->mcp_info->cmd_lock);
|
||||
ecore_mcp_cmd_del_elem(p_hwfn, p_cmd_elem);
|
||||
|
@ -3206,8 +3206,8 @@ static void ecore_iov_vf_mbx_vport_update(struct ecore_hwfn *p_hwfn,
|
||||
"Upper-layer prevents said VF"
|
||||
" configuration\n");
|
||||
else
|
||||
DP_NOTICE(p_hwfn, true,
|
||||
"No feature tlvs found for vport update\n");
|
||||
DP_VERBOSE(p_hwfn, ECORE_MSG_IOV,
|
||||
"No feature tlvs found for vport update\n");
|
||||
status = PFVF_STATUS_NOT_SUPPORTED;
|
||||
goto out;
|
||||
}
|
||||
|
@ -130,14 +130,20 @@ ecore_send_msg2pf(struct ecore_hwfn *p_hwfn,
|
||||
}
|
||||
|
||||
if (!*done) {
|
||||
DP_VERBOSE(p_hwfn, ECORE_MSG_IOV,
|
||||
"VF <-- PF Timeout [Type %d]\n",
|
||||
p_req->first_tlv.tl.type);
|
||||
DP_NOTICE(p_hwfn, true,
|
||||
"VF <-- PF Timeout [Type %d]\n",
|
||||
p_req->first_tlv.tl.type);
|
||||
rc = ECORE_TIMEOUT;
|
||||
} else {
|
||||
DP_VERBOSE(p_hwfn, ECORE_MSG_IOV,
|
||||
"PF response: %d [Type %d]\n",
|
||||
*done, p_req->first_tlv.tl.type);
|
||||
if ((*done != PFVF_STATUS_SUCCESS) &&
|
||||
(*done != PFVF_STATUS_NO_RESOURCE))
|
||||
DP_NOTICE(p_hwfn, false,
|
||||
"PF response: %d [Type %d]\n",
|
||||
*done, p_req->first_tlv.tl.type);
|
||||
else
|
||||
DP_VERBOSE(p_hwfn, ECORE_MSG_IOV,
|
||||
"PF response: %d [Type %d]\n",
|
||||
*done, p_req->first_tlv.tl.type);
|
||||
}
|
||||
|
||||
return rc;
|
||||
|
Loading…
x
Reference in New Issue
Block a user