net/mlx5: Add NOT_READY command return status
Add a new command status MLX5_CMD_STAT_NOT_READY to handle cases where the firmware is not ready. Signed-off-by: Shay Drory <shayd@nvidia.com> Signed-off-by: Tariq Toukan <tariqt@nvidia.com> Signed-off-by: Saeed Mahameed <saeedm@nvidia.com> Reviewed-by: Jacob Keller <jacob.e.keller@intel.com> Reviewed-by: Kalesh AP <kalesh-anakkur.purayil@broadcom.com> Link: https://patch.msgid.link/20240911201757.1505453-14-saeed@kernel.org Signed-off-by: Jakub Kicinski <kuba@kernel.org>
This commit is contained in:
parent
9c754d0970
commit
5bd877093f
@ -754,6 +754,8 @@ static const char *cmd_status_str(u8 status)
|
|||||||
return "bad resource";
|
return "bad resource";
|
||||||
case MLX5_CMD_STAT_RES_BUSY:
|
case MLX5_CMD_STAT_RES_BUSY:
|
||||||
return "resource busy";
|
return "resource busy";
|
||||||
|
case MLX5_CMD_STAT_NOT_READY:
|
||||||
|
return "FW not ready";
|
||||||
case MLX5_CMD_STAT_LIM_ERR:
|
case MLX5_CMD_STAT_LIM_ERR:
|
||||||
return "limits exceeded";
|
return "limits exceeded";
|
||||||
case MLX5_CMD_STAT_BAD_RES_STATE_ERR:
|
case MLX5_CMD_STAT_BAD_RES_STATE_ERR:
|
||||||
@ -787,6 +789,7 @@ static int cmd_status_to_err(u8 status)
|
|||||||
case MLX5_CMD_STAT_BAD_SYS_STATE_ERR: return -EIO;
|
case MLX5_CMD_STAT_BAD_SYS_STATE_ERR: return -EIO;
|
||||||
case MLX5_CMD_STAT_BAD_RES_ERR: return -EINVAL;
|
case MLX5_CMD_STAT_BAD_RES_ERR: return -EINVAL;
|
||||||
case MLX5_CMD_STAT_RES_BUSY: return -EBUSY;
|
case MLX5_CMD_STAT_RES_BUSY: return -EBUSY;
|
||||||
|
case MLX5_CMD_STAT_NOT_READY: return -EAGAIN;
|
||||||
case MLX5_CMD_STAT_LIM_ERR: return -ENOMEM;
|
case MLX5_CMD_STAT_LIM_ERR: return -ENOMEM;
|
||||||
case MLX5_CMD_STAT_BAD_RES_STATE_ERR: return -EINVAL;
|
case MLX5_CMD_STAT_BAD_RES_STATE_ERR: return -EINVAL;
|
||||||
case MLX5_CMD_STAT_IX_ERR: return -EINVAL;
|
case MLX5_CMD_STAT_IX_ERR: return -EINVAL;
|
||||||
@ -815,14 +818,16 @@ EXPORT_SYMBOL(mlx5_cmd_out_err);
|
|||||||
static void cmd_status_print(struct mlx5_core_dev *dev, void *in, void *out)
|
static void cmd_status_print(struct mlx5_core_dev *dev, void *in, void *out)
|
||||||
{
|
{
|
||||||
u16 opcode, op_mod;
|
u16 opcode, op_mod;
|
||||||
|
u8 status;
|
||||||
u16 uid;
|
u16 uid;
|
||||||
|
|
||||||
opcode = in_to_opcode(in);
|
opcode = in_to_opcode(in);
|
||||||
op_mod = MLX5_GET(mbox_in, in, op_mod);
|
op_mod = MLX5_GET(mbox_in, in, op_mod);
|
||||||
uid = MLX5_GET(mbox_in, in, uid);
|
uid = MLX5_GET(mbox_in, in, uid);
|
||||||
|
status = MLX5_GET(mbox_out, out, status);
|
||||||
|
|
||||||
if (!uid && opcode != MLX5_CMD_OP_DESTROY_MKEY &&
|
if (!uid && opcode != MLX5_CMD_OP_DESTROY_MKEY &&
|
||||||
opcode != MLX5_CMD_OP_CREATE_UCTX)
|
opcode != MLX5_CMD_OP_CREATE_UCTX && status != MLX5_CMD_STAT_NOT_READY)
|
||||||
mlx5_cmd_out_err(dev, opcode, op_mod, out);
|
mlx5_cmd_out_err(dev, opcode, op_mod, out);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -1449,6 +1449,7 @@ enum {
|
|||||||
MLX5_CMD_STAT_BAD_SYS_STATE_ERR = 0x4,
|
MLX5_CMD_STAT_BAD_SYS_STATE_ERR = 0x4,
|
||||||
MLX5_CMD_STAT_BAD_RES_ERR = 0x5,
|
MLX5_CMD_STAT_BAD_RES_ERR = 0x5,
|
||||||
MLX5_CMD_STAT_RES_BUSY = 0x6,
|
MLX5_CMD_STAT_RES_BUSY = 0x6,
|
||||||
|
MLX5_CMD_STAT_NOT_READY = 0x7,
|
||||||
MLX5_CMD_STAT_LIM_ERR = 0x8,
|
MLX5_CMD_STAT_LIM_ERR = 0x8,
|
||||||
MLX5_CMD_STAT_BAD_RES_STATE_ERR = 0x9,
|
MLX5_CMD_STAT_BAD_RES_STATE_ERR = 0x9,
|
||||||
MLX5_CMD_STAT_IX_ERR = 0xa,
|
MLX5_CMD_STAT_IX_ERR = 0xa,
|
||||||
|
Loading…
Reference in New Issue
Block a user