On Wed, Mar 14, 2018 at 01:00:27PM -0500, Steve Wise wrote:
> > On Wed, Mar 14, 2018 at 07:33:53PM +0200, Leon Romanovsky wrote:
> > > On Tue, Mar 13, 2018 at 10:30:32AM -0700, Steve Wise wrote:
> > > > Each provider can register a "fill entry" function with the restrack
> core.
> > > > This function will be called when filling out a resource, allowing the
> > > > provider to add provider-specific details. The details consist of a
> > > > table of nested attributes, that are in the form of "key, value"
> tuple.
> > > > The key nlattr must be strings, and the value nlattr can be one of
> > > > provider attributes that are generic, but typed, allowing the
> nlmessage
> > > > to ve validated. Currently the types include string, s32, u32, x32,
> s64,
> > > > u64, and x64. The inclusion of x, s, and u variants for numeric
> attributes
> > > > allows the user tool to print the number in the desired format.
> > > >
> > > > More attrs can be defined as they become needed by providers.
> > > >
> > > > Signed-off-by: Steve Wise <swise@xxxxxxxxxxxxxxxxxxxxx>
> > > > drivers/infiniband/core/nldev.c | 40
> > ++++++++++++++++++++++++++++++++++++++++
> > > > include/rdma/restrack.h | 10 ++++++++++
> > > > include/uapi/rdma/rdma_netlink.h | 17 +++++++++++++++++
> > > > 3 files changed, 67 insertions(+)
> > > >
> > > > diff --git a/drivers/infiniband/core/nldev.c
> b/drivers/infiniband/core/nldev.c
> > > > index 192084c..933df61 100644
> > > > +++ b/drivers/infiniband/core/nldev.c
> > > > @@ -95,8 +95,28 @@
> > > > [RDMA_NLDEV_ATTR_RES_PD_ENTRY] = { .type =
> > NLA_NESTED },
> > > > [RDMA_NLDEV_ATTR_RES_LOCAL_DMA_LKEY] = { .type = NLA_U32 },
> > > > [RDMA_NLDEV_ATTR_RES_UNSAFE_GLOBAL_RKEY] = { .type =
> > NLA_U32 },
> > > > + [RDMA_NLDEV_ATTR_PROVIDER] = { .type =
> > NLA_NESTED },
> > > > + [RDMA_NLDEV_ATTR_PROVIDER_ENTRY] = { .type =
> > NLA_NESTED },
> > > > + [RDMA_NLDEV_ATTR_PROVIDER_STRING] = { .type =
> > NLA_NUL_STRING,
> > > > + .len =
> > RDMA_NLDEV_ATTR_PROVIDER_STRLEN },
> > > > + [RDMA_NLDEV_ATTR_PROVIDER_D32] = { .type = NLA_S32 },
> > > > + [RDMA_NLDEV_ATTR_PROVIDER_U32] = { .type = NLA_U32 },
> > > > + [RDMA_NLDEV_ATTR_PROVIDER_X32] = { .type = NLA_U32 },
> > > > + [RDMA_NLDEV_ATTR_PROVIDER_D64] = { .type = NLA_S64 },
> > > > + [RDMA_NLDEV_ATTR_PROVIDER_U64] = { .type = NLA_U64 },
> > > > + [RDMA_NLDEV_ATTR_PROVIDER_X64] = { .type = NLA_U64 },
> > > > + [RDMA_NLDEV_ATTR_PAD] = { },
> > >
> > > RDMA_NLDEV_ATTR_PAD ????
> >
> > This was supposed to have been done earlier..
> >
>
> It wasn't clear whether we needed it. Leon sez 0 is fine.
>
> > Seperate patch please, and change every caller of nla_put_u64_64bit to
> > use the new constant.
> >
>
> Yea ok.
>
> > And I think it has to be 0 for ABI compat, doesn't it?
> >
>
> I guess so, since 0 was used originally.
Yes, we are using 0, which is equal to RDMA_NLDEV_ATTR_UNSPEC.
All zeros can be replaced for clarity, but from ABI perspective we are
fine.
Thanks
>
> Steve.
>
Attachment:
signature.asc
Description: PGP signature
