> From: Cao, Yahui <yahui.cao@xxxxxxxxx>
> Sent: Tuesday, November 21, 2023 10:51 AM
> +
> + while (msg_slot->opcode != VIRTCHNL_OP_UNKNOWN) {
> + struct ice_rq_event_info event;
> + u64 slot_sz;
> +
> + slot_sz = struct_size(msg_slot, msg_buffer, msg_slot-
> >msg_len);
> + dev_dbg(dev, "VF %d replay virtchnl message op code: %d,
> msg len: %d\n",
> + vf->vf_id, msg_slot->opcode, msg_slot->msg_len);
> + event.desc.cookie_high = cpu_to_le32(msg_slot->opcode);
> + event.msg_len = msg_slot->msg_len;
> + event.desc.retval = cpu_to_le16(vf->vf_id);
> + event.msg_buf = (unsigned char *)msg_slot->msg_buffer;
> + ret = ice_vc_process_vf_msg(vf->pf, &event, NULL);
> + if (ret) {
> + dev_err(dev, "VF %d failed to replay virtchnl message
> op code: %d\n",
> + vf->vf_id, msg_slot->opcode);
> + goto out_clear_replay;
> + }
> + event.msg_buf = NULL;
this line is unnecessary.
