On Sun, May 27, 2018 at 01:23:44PM +0300, Leon Romanovsky wrote:

> +     if (!mcounters->hw_cntrs_hndl) {
> +             mcounters->hw_cntrs_hndl =
> +                     (void 
> *)mlx5_fc_create(to_mdev(ibcounters->device)->mdev,
> +                                            false);

No unnecessary casts

> +struct mlx5_ib_flow_counters_data {
> +     __aligned_u64   counters_data;

I think this is supposed to use RDMA_UAPI_PTR() these days.

Jason

Reply via email to