On Wed, Jul 31, 2019 at 02:56:27PM +0300, Leon Romanovsky wrote:
> From: Leon Romanovsky <leonro@xxxxxxxxxxxx>
>
> Delete DEBU ODP dead code which is leftover from development
DEBU -> DEBUG
> stage and doesn't need to be part of the upstream kernel.
>
> Signed-off-by: Leon Romanovsky <leonro@xxxxxxxxxxxx>
> ---
> drivers/infiniband/hw/mlx5/odp.c | 24 ------------------------
> 1 file changed, 24 deletions(-)
>
> diff --git a/drivers/infiniband/hw/mlx5/odp.c b/drivers/infiniband/hw/mlx5/odp.c
> index 74310d885a90..6f1de5edbe8e 100644
> --- a/drivers/infiniband/hw/mlx5/odp.c
> +++ b/drivers/infiniband/hw/mlx5/odp.c
> @@ -1033,9 +1033,6 @@ static int mlx5_ib_mr_initiator_pfault_handler(
> u32 transport_caps;
> struct mlx5_base_av *av;
> unsigned ds, opcode;
> -#if defined(DEBUG)
> - u32 ctrl_wqe_index, ctrl_qpn;
> -#endif
> u32 qpn = qp->trans_qp.base.mqp.qpn;
>
> ds = be32_to_cpu(ctrl->qpn_ds) & MLX5_WQE_CTRL_DS_MASK;
> @@ -1051,27 +1048,6 @@ static int mlx5_ib_mr_initiator_pfault_handler(
> return -EFAULT;
> }
>
> -#if defined(DEBUG)
> - ctrl_wqe_index = (be32_to_cpu(ctrl->opmod_idx_opcode) &
> - MLX5_WQE_CTRL_WQE_INDEX_MASK) >>
> - MLX5_WQE_CTRL_WQE_INDEX_SHIFT;
> - if (wqe_index != ctrl_wqe_index) {
> - mlx5_ib_err(dev, "Got WQE with invalid wqe_index. wqe_index=0x%x, qpn=0x%x ctrl->wqe_index=0x%x\n",
> - wqe_index, qpn,
> - ctrl_wqe_index);
> - return -EFAULT;
> - }
> -
> - ctrl_qpn = (be32_to_cpu(ctrl->qpn_ds) & MLX5_WQE_CTRL_QPN_MASK) >>
> - MLX5_WQE_CTRL_QPN_SHIFT;
> - if (qpn != ctrl_qpn) {
> - mlx5_ib_err(dev, "Got WQE with incorrect QP number. wqe_index=0x%x, qpn=0x%x ctrl->qpn=0x%x\n",
> - wqe_index, qpn,
> - ctrl_qpn);
> - return -EFAULT;
> - }
> -#endif /* DEBUG */
> -
> *wqe_end = *wqe + ds * MLX5_WQE_DS_UNITS;
> *wqe += sizeof(*ctrl);
>
> --
> 2.20.1
>
