On Wed, May 22, 2019 at 02:21:37PM -0300, Jason Gunthorpe wrote:
> On Mon, Apr 29, 2019 at 11:34:47AM +0300, Leon Romanovsky wrote:
> > From: Mark Zhang <markz@xxxxxxxxxxxx>
> >
> > This patch adds the ability to return all available counters
> > together with their properties and hwstats.
> >
> > Signed-off-by: Mark Zhang <markz@xxxxxxxxxxxx>
> > Reviewed-by: Majd Dibbiny <majd@xxxxxxxxxxxx>
> > Signed-off-by: Leon Romanovsky <leonro@xxxxxxxxxxxx>
> > drivers/infiniband/core/counters.c | 28 +++++
> > drivers/infiniband/core/device.c | 2 +
> > drivers/infiniband/core/nldev.c | 173 +++++++++++++++++++++++++++++
> > include/rdma/ib_verbs.h | 10 ++
> > include/rdma/rdma_counter.h | 3 +
> > include/uapi/rdma/rdma_netlink.h | 10 +-
> > 6 files changed, 225 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/infiniband/core/counters.c b/drivers/infiniband/core/counters.c
> > index 665e0d43c21b..36cd9eca1e46 100644
> > +++ b/drivers/infiniband/core/counters.c
> > @@ -62,6 +62,9 @@ static struct rdma_counter *rdma_counter_alloc(struct ib_device *dev, u8 port,
> > {
> > struct rdma_counter *counter;
> >
> > + if (!dev->ops.counter_alloc_stats)
> > + return NULL;
> > +
>
> Seems weird to add this now, why was it Ok to have counters prior to
> this patch?
Prior to this patch, "sysfs" counters and "netlink" counters were
independent from user perspective.
Thanks
>
> Jason
