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