net/hinic/base: fix log info for PF command channel
authorGuoyang Zhou <zhouguoyang@huawei.com>
Wed, 4 Nov 2020 02:03:20 +0000 (10:03 +0800)
committerFerruh Yigit <ferruh.yigit@intel.com>
Wed, 4 Nov 2020 14:07:07 +0000 (15:07 +0100)
When PF command channel is error, the variables in the log has been
cleared, which is not printed yet.

Fixes: 214164a6bf7f ("net/hinic/base: remove unused function parameters")
Cc: stable@dpdk.org
Signed-off-by: Guoyang Zhou <zhouguoyang@huawei.com>
drivers/net/hinic/base/hinic_pmd_mgmt.c

index babb93a..fb31bc8 100644 (file)
@@ -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 failed, "
                        "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;
        }