On Tue, May 15, 2018 at 05:09:49PM +0300, Leon Romanovsky wrote:
> From: Raed Salem <raeds@xxxxxxxxxxxx>
>
> Implements the flow counters read wrapper.
>
> Reviewed-by: Yishai Hadas <yishaih@xxxxxxxxxxxx>
> Signed-off-by: Raed Salem <raeds@xxxxxxxxxxxx>
> Signed-off-by: Leon Romanovsky <leonro@xxxxxxxxxxxx>
> drivers/infiniband/hw/mlx5/main.c | 22 ++++++++++++++++++++++
> drivers/infiniband/hw/mlx5/mlx5_ib.h | 5 +++++
> 2 files changed, 27 insertions(+)
>
> diff --git a/drivers/infiniband/hw/mlx5/main.c b/drivers/infiniband/hw/mlx5/main.c
> index 61fefee40922..59ee1e96cf2e 100644
> +++ b/drivers/infiniband/hw/mlx5/main.c
> @@ -3180,6 +3180,28 @@ static void set_underlay_qp(struct mlx5_ib_dev *dev,
> }
> }
>
> +static int read_flow_counters(struct ib_device *ibdev,
> + struct mlx5_read_counters_attr *read_attr)
> +{
> + struct mlx5_fc *fc = (struct mlx5_fc *)(read_attr->hw_cntrs_hndl);
> + struct mlx5_ib_dev *dev = to_mdev(ibdev);
> +
> + return mlx5_fc_query(dev->mdev, fc->id, &read_attr->out[0],
> + &read_attr->out[1]);
This would make much more sense as:
return mlx5_fc_query(dev->mdev, fc->id,
&read_attr->out[IB_COUNTER_PACKETS],
&read_attr->out[IB_COUNTER_BYTES]);
If I followed the flow properly..
Jason
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
