| Message ID | 20230503210342.66155-1-kheib@redhat.com (mailing list archive) |
|---|---|
| State | Superseded |
| Delegated to: | David Ahern |
| Headers | show |
| Series | [iproute2-next] rdma: Report device protocol | expand |
| Context | Check | Description |
|---|---|---|
| netdev/tree_selection | success | Not a local patch |
On Wed, May 03, 2023 at 05:03:42PM -0400, Kamal Heib wrote: > Add support for reporting the device protocol. > > $ rdma dev > 11: mlx5_0: node_type ca proto roce fw 12.28.2006 > node_guid 248a:0703:004b:f094 sys_image_guid 248a:0703:004b:f094 > 12: mlx5_1: node_type ca proto ib fw 12.28.2006 > node_guid 248a:0703:0049:d4f0 sys_image_guid 248a:0703:0049:d4f0 > 13: mlx5_2: node_type ca proto ib fw 12.28.2006 > node_guid 248a:0703:0049:d4f1 sys_image_guid 248a:0703:0049:d4f0 > 17: siw0: node_type rnic proto iw node_guid > 0200:00ff:fe00:0000 sys_image_guid 0200:00ff:fe00:0000 > > Signed-off-by: Kamal Heib <kheib@redhat.com> > --- > rdma/dev.c | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/rdma/dev.c b/rdma/dev.c > index c684dde4a56f..04c2a574405c 100644 > --- a/rdma/dev.c > +++ b/rdma/dev.c > @@ -189,6 +189,16 @@ static void dev_print_node_type(struct rd *rd, struct nlattr **tb) > node_str); > } > > +static void dev_print_dev_proto(struct rd *rd, struct nlattr **tb) > +{ > + const char *str; > + if (!tb[RDMA_NLDEV_ATTR_DEV_PROTOCOL]) > + return; > + > + str = mnl_attr_get_str(tb[RDMA_NLDEV_ATTR_DEV_PROTOCOL]); > + print_color_string(PRINT_ANY, COLOR_NONE, "proto", "proto %s ", str); Please, let's use full word "protocol" and not "proto". Other than that, Acked-by: Leon Romanovsky <leonro@nvidia.com>
On 2023-05-04 03:48, Leon Romanovsky wrote: > On Wed, May 03, 2023 at 05:03:42PM -0400, Kamal Heib wrote: >> Add support for reporting the device protocol. >> >> $ rdma dev >> 11: mlx5_0: node_type ca proto roce fw 12.28.2006 >> node_guid 248a:0703:004b:f094 sys_image_guid 248a:0703:004b:f094 >> 12: mlx5_1: node_type ca proto ib fw 12.28.2006 >> node_guid 248a:0703:0049:d4f0 sys_image_guid 248a:0703:0049:d4f0 >> 13: mlx5_2: node_type ca proto ib fw 12.28.2006 >> node_guid 248a:0703:0049:d4f1 sys_image_guid 248a:0703:0049:d4f0 >> 17: siw0: node_type rnic proto iw node_guid >> 0200:00ff:fe00:0000 sys_image_guid 0200:00ff:fe00:0000 >> >> Signed-off-by: Kamal Heib <kheib@redhat.com> >> --- >> rdma/dev.c | 11 +++++++++++ >> 1 file changed, 11 insertions(+) >> >> diff --git a/rdma/dev.c b/rdma/dev.c >> index c684dde4a56f..04c2a574405c 100644 >> --- a/rdma/dev.c >> +++ b/rdma/dev.c >> @@ -189,6 +189,16 @@ static void dev_print_node_type(struct rd *rd, struct nlattr **tb) >> node_str); >> } >> >> +static void dev_print_dev_proto(struct rd *rd, struct nlattr **tb) >> +{ >> + const char *str; >> + if (!tb[RDMA_NLDEV_ATTR_DEV_PROTOCOL]) >> + return; >> + >> + str = mnl_attr_get_str(tb[RDMA_NLDEV_ATTR_DEV_PROTOCOL]); >> + print_color_string(PRINT_ANY, COLOR_NONE, "proto", "proto %s ", str); > > Please, let's use full word "protocol" and not "proto". > > Other than that, > Acked-by: Leon Romanovsky <leonro@nvidia.com> > Thank you for reviewing the patch, I'll fix it in v2.
diff --git a/rdma/dev.c b/rdma/dev.c index c684dde4a56f..04c2a574405c 100644 --- a/rdma/dev.c +++ b/rdma/dev.c @@ -189,6 +189,16 @@ static void dev_print_node_type(struct rd *rd, struct nlattr **tb) node_str); } +static void dev_print_dev_proto(struct rd *rd, struct nlattr **tb) +{ + const char *str; + if (!tb[RDMA_NLDEV_ATTR_DEV_PROTOCOL]) + return; + + str = mnl_attr_get_str(tb[RDMA_NLDEV_ATTR_DEV_PROTOCOL]); + print_color_string(PRINT_ANY, COLOR_NONE, "proto", "proto %s ", str); +} + static int dev_parse_cb(const struct nlmsghdr *nlh, void *data) { struct nlattr *tb[RDMA_NLDEV_ATTR_MAX] = {}; @@ -206,6 +216,7 @@ static int dev_parse_cb(const struct nlmsghdr *nlh, void *data) print_color_string(PRINT_ANY, COLOR_NONE, "ifname", "%s: ", name); dev_print_node_type(rd, tb); + dev_print_dev_proto(rd, tb); dev_print_fw(rd, tb); dev_print_node_guid(rd, tb); dev_print_sys_image_guid(rd, tb);
Add support for reporting the device protocol. $ rdma dev 11: mlx5_0: node_type ca proto roce fw 12.28.2006 node_guid 248a:0703:004b:f094 sys_image_guid 248a:0703:004b:f094 12: mlx5_1: node_type ca proto ib fw 12.28.2006 node_guid 248a:0703:0049:d4f0 sys_image_guid 248a:0703:0049:d4f0 13: mlx5_2: node_type ca proto ib fw 12.28.2006 node_guid 248a:0703:0049:d4f1 sys_image_guid 248a:0703:0049:d4f0 17: siw0: node_type rnic proto iw node_guid 0200:00ff:fe00:0000 sys_image_guid 0200:00ff:fe00:0000 Signed-off-by: Kamal Heib <kheib@redhat.com> --- rdma/dev.c | 11 +++++++++++ 1 file changed, 11 insertions(+)