Commit 3d092ef0 authored by Corey Minyard's avatar Corey Minyard

ipmi: When handling send message responses, don't process the message

A chunk was dropped when the code handling send messages was rewritten.
Those messages shouldn't be processed normally, they are just an
indication that the message was successfully sent and the timers should
be started for the real response that should be coming later.

Add back in the missing chunk to just discard the message and go on.

Fixes: 059747c2 ("ipmi: Add support for IPMB direct messages")
Reported-by: default avatarJoe Wiese <jwiese@rackspace.com>
Cc: stable@vger.kernel.org # v5.16+
Signed-off-by: default avatarCorey Minyard <cminyard@mvista.com>
Tested-by: default avatarJoe Wiese <jwiese@rackspace.com>
parent ae085d7f
...@@ -4518,6 +4518,8 @@ static int handle_one_recv_msg(struct ipmi_smi *intf, ...@@ -4518,6 +4518,8 @@ static int handle_one_recv_msg(struct ipmi_smi *intf,
} else } else
/* The message was sent, start the timer. */ /* The message was sent, start the timer. */
intf_start_seq_timer(intf, msg->msgid); intf_start_seq_timer(intf, msg->msgid);
requeue = 0;
goto out;
} else if (((msg->rsp[0] >> 2) != ((msg->data[0] >> 2) | 1)) } else if (((msg->rsp[0] >> 2) != ((msg->data[0] >> 2) | 1))
|| (msg->rsp[1] != msg->data[1])) { || (msg->rsp[1] != msg->data[1])) {
/* /*
......
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