On Mon, Jun 10, 2019 at 05:13:34PM +0300, Leon Romanovsky wrote:
> On Wed, Jun 05, 2019 at 03:32:50PM -0300, Jason Gunthorpe wrote:
> > From: Jason Gunthorpe <jgg@xxxxxxxxxxxx>
> >
> > This enum is exposed over the sysfs file 'node_type' and over netlink via
> > RDMA_NLDEV_ATTR_DEV_NODE_TYPE, so declare it in the uapi headers.
> >
> > Signed-off-by: Jason Gunthorpe <jgg@xxxxxxxxxxxx>
> > drivers/infiniband/core/verbs.c | 2 +-
> > include/rdma/ib_verbs.h | 13 +------------
> > include/uapi/rdma/rdma_netlink.h | 12 ++++++++++++
> > 3 files changed, 14 insertions(+), 13 deletions(-)
> >
> > diff --git a/drivers/infiniband/core/verbs.c b/drivers/infiniband/core/verbs.c
> > index e666a1f7608d86..56af18456ba776 100644
> > +++ b/drivers/infiniband/core/verbs.c
> > @@ -209,7 +209,7 @@ __attribute_const__ int ib_rate_to_mbps(enum ib_rate rate)
> > EXPORT_SYMBOL(ib_rate_to_mbps);
> >
> > __attribute_const__ enum rdma_transport_type
> > -rdma_node_get_transport(enum rdma_node_type node_type)
> > +rdma_node_get_transport(unsigned int node_type)
> > {
> >
> > if (node_type == RDMA_NODE_USNIC)
> > diff --git a/include/rdma/ib_verbs.h b/include/rdma/ib_verbs.h
> > index cdfeeda1db7f31..d5dd3cb7fcf702 100644
> > +++ b/include/rdma/ib_verbs.h
> > @@ -132,17 +132,6 @@ struct ib_gid_attr {
> > u8 port_num;
> > };
> >
> > -enum rdma_node_type {
>
> Why did you drop "enum rdma_node_type" and changed to be anonymous enum?
To avoid namespace pollution in a user header
Jason
