Commit 532ec6f1 authored by Grant Grundler's avatar Grant Grundler Committed by Roland Dreier

SRPT: Fix odd use of WARN_ON()

While WARN_ON("const string") will work, it's intent is not obvious.
The warning is more useful by showing the "state" value.
Signed-off-by: default avatarGrant Grundler <grundler@chromium.org>
Signed-off-by: default avatarRoland Dreier <roland@purestorage.com>
parent 41ef2d56
...@@ -1374,7 +1374,7 @@ static int srpt_abort_cmd(struct srpt_send_ioctx *ioctx) ...@@ -1374,7 +1374,7 @@ static int srpt_abort_cmd(struct srpt_send_ioctx *ioctx)
target_put_sess_cmd(ioctx->ch->sess, &ioctx->cmd); target_put_sess_cmd(ioctx->ch->sess, &ioctx->cmd);
break; break;
default: default:
WARN_ON("ERROR: unexpected command state"); WARN(1, "Unexpected command state (%d)", state);
break; break;
} }
......
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