Re: [PATCH][next] RDMA/mlx5: remove duplicated assignment to resp.response_length
On Thu, Jun 04, 2020 at 03:39:02PM +0100, Colin King wrote: > From: Colin Ian King > > The assignment to resp.response_length is never read since it is being > updated again on the next statement. The assignment is redundant so > removed it. > > Addresses-Coverity: ("Unused value") > Signed-off-by: Colin Ian King > Acked-by: Leon Romanovsky > --- > drivers/infiniband/hw/mlx5/qp.c | 2 -- > 1 file changed, 2 deletions(-) Applied to for-rc, with a fixes line Thanks, Jason
Re: [PATCH][next] RDMA/mlx5: remove duplicated assignment to resp.response_length
On Thu, Jun 04, 2020 at 03:39:02PM +0100, Colin King wrote: > From: Colin Ian King > > The assignment to resp.response_length is never read since it is being > updated again on the next statement. The assignment is redundant so > removed it. > > Addresses-Coverity: ("Unused value") > Signed-off-by: Colin Ian King > --- > drivers/infiniband/hw/mlx5/qp.c | 2 -- > 1 file changed, 2 deletions(-) > Thanks, Acked-by: Leon Romanovsky
[PATCH][next] RDMA/mlx5: remove duplicated assignment to resp.response_length
From: Colin Ian King The assignment to resp.response_length is never read since it is being updated again on the next statement. The assignment is redundant so removed it. Addresses-Coverity: ("Unused value") Signed-off-by: Colin Ian King --- drivers/infiniband/hw/mlx5/qp.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/infiniband/hw/mlx5/qp.c b/drivers/infiniband/hw/mlx5/qp.c index 81bf6b975e0e..d61ca85033de 100644 --- a/drivers/infiniband/hw/mlx5/qp.c +++ b/drivers/infiniband/hw/mlx5/qp.c @@ -4162,8 +4162,6 @@ static int mlx5_ib_modify_dct(struct ib_qp *ibqp, struct ib_qp_attr *attr, if (udata->outlen < min_resp_len) return -EINVAL; - resp.response_length = min_resp_len; - /* * If we don't have enough space for the ECE options, * simply indicate it with resp.response_length. -- 2.25.1