Skip to content

Commit

Permalink
net/hinic/base: fix log info for PF command channel
Browse files Browse the repository at this point in the history
[ upstream commit 1e4593d ]

When PF command channel is error, the variables in the log has been
cleared, which is not printed yet.

Fixes: 214164a ("net/hinic/base: remove unused function parameters")

Signed-off-by: Guoyang Zhou <zhouguoyang@huawei.com>
  • Loading branch information
Guoyang Zhou authored and bluca committed Nov 9, 2020
1 parent d579dd2 commit b89d883
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion drivers/net/hinic/base/hinic_pmd_mgmt.c
Original file line number Diff line number Diff line change
Expand Up @@ -614,16 +614,18 @@ static int recv_mgmt_msg_handler(struct hinic_msg_pf_to_mgmt *pf_to_mgmt,
u8 *dest_msg;
u8 seq_id, seq_len;
u32 msg_buf_max = MAX_PF_MGMT_BUF_SIZE;
u8 front_id;

seq_id = HINIC_MSG_HEADER_GET(msg_header, SEQID);
seq_len = HINIC_MSG_HEADER_GET(msg_header, SEG_LEN);
front_id = recv_msg->sed_id;

if (!check_mgmt_seq_id_and_seg_len(recv_msg, seq_id, seq_len)) {
PMD_DRV_LOG(ERR,
"Mgmt msg sequence and segment check fail, "
"func id: 0x%x, front id: 0x%x, current id: 0x%x, seg len: 0x%x",
hinic_global_func_id(pf_to_mgmt->hwdev),
recv_msg->sed_id, seq_id, seq_len);
front_id, seq_id, seq_len);
return HINIC_ERROR;
}

Expand Down

0 comments on commit b89d883

Please sign in to comment.