From mboxrd@z Thu Jan 1 00:00:00 1970 From: Leon Romanovsky Subject: Re: [PATCH iproute2-next 2/2] rdma: Print net device name and index for RDMA device Date: Mon, 26 Mar 2018 20:00:59 +0300 Message-ID: <20180326170059.GK1877@mtr-leonro.local> References: <20180326082829.21214-1-leon@kernel.org> <20180326082829.21214-3-leon@kernel.org> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="7SrMUQONj8Rl9QNG" Cc: David Ahern , netdev , RDMA mailing list , Stephen Hemminger To: Steve Wise Return-path: Received: from mail.kernel.org ([198.145.29.99]:56476 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751964AbeCZRBD (ORCPT ); Mon, 26 Mar 2018 13:01:03 -0400 Content-Disposition: inline In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: --7SrMUQONj8Rl9QNG Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Mon, Mar 26, 2018 at 10:17:12AM -0500, Steve Wise wrote: > > > On 3/26/2018 3:28 AM, Leon Romanovsky wrote: > > From: Leon Romanovsky > > > > The RDMA devices are operated in RoCE and iWARP modes have net device > > underneath. Present their names in regular output and their net index > > in detailed mode. > > > > [root@nps ~]# rdma link show mlx5_3/1 > > 4/1: mlx5_3/1: state ACTIVE physical_state LINK_UP netdev ens7 > > [root@nps ~]# rdma link show mlx5_3/1 -d > > 4/1: mlx5_3/1: state ACTIVE physical_state LINK_UP netdev ens7 netdev_index 7 > > caps: > > > > Signed-off-by: Leon Romanovsky > > --- > > rdma/include/uapi/rdma/rdma_netlink.h | 4 ++++ > > rdma/link.c | 21 +++++++++++++++++++++ > > rdma/utils.c | 2 ++ > > 3 files changed, 27 insertions(+) > > > > diff --git a/rdma/include/uapi/rdma/rdma_netlink.h b/rdma/include/uapi/rdma/rdma_netlink.h > > index 9446a721..45474f13 100644 > > --- a/rdma/include/uapi/rdma/rdma_netlink.h > > +++ b/rdma/include/uapi/rdma/rdma_netlink.h > > @@ -388,6 +388,10 @@ enum rdma_nldev_attr { > > RDMA_NLDEV_ATTR_RES_LOCAL_DMA_LKEY, /* u32 */ > > RDMA_NLDEV_ATTR_RES_UNSAFE_GLOBAL_RKEY, /* u32 */ > > > > + /* Netdev information for relevant protocols, like RoCE and iWARP */ > > + RDMA_NLDEV_ATTR_NDEV_INDEX, /* u32 */ > > + RDMA_NLDEV_ATTR_NDEV_NAME, /* string */ > > + > > RDMA_NLDEV_ATTR_MAX > > }; > > #endif /* _RDMA_NETLINK_H */ > > diff --git a/rdma/link.c b/rdma/link.c > > index 66bcd50e..7e914c87 100644 > > --- a/rdma/link.c > > +++ b/rdma/link.c > > @@ -205,6 +205,26 @@ static void link_print_phys_state(struct rd *rd, struct nlattr **tb) > > pr_out("physical_state %s ", phys_state_to_str(phys_state)); > > } > > > > +static void link_print_netdev(struct rd *rd, struct nlattr **tb) > > +{ > > + const char *netdev_name; > > + uint32_t idx; > > + > > + if (!tb[RDMA_NLDEV_ATTR_NDEV_NAME] || !tb[RDMA_NLDEV_ATTR_NDEV_INDEX]) > > + return; > > + > > + netdev_name = mnl_attr_get_str(tb[RDMA_NLDEV_ATTR_NDEV_NAME]); > > + idx = mnl_attr_get_u32(tb[RDMA_NLDEV_ATTR_NDEV_INDEX]); > > + if (rd->json_output) { > > + jsonw_string_field(rd->jw, "netdev", netdev_name); > > + jsonw_uint_field(rd->jw, "netdev_index", idx); > > + } else { > > + pr_out("netdev %s ", netdev_name); > > + if (rd->show_details) > > + pr_out("netdev_index %u ", idx); > > + } > > +} > > + > > Why is the netdev_index under 'show_details'? Because I think that it is useless for day-to-day usage of average rdmatool user and mostly will be used as pipe to other scripts, if JSON doesn't good fit for any reasons. So I decided do not clutter "general output". > > Reviewed-by: Steve Wise Thanks > > > static int link_parse_cb(const struct nlmsghdr *nlh, void *data) > > { > > struct nlattr *tb[RDMA_NLDEV_ATTR_MAX] = {}; > > @@ -241,6 +261,7 @@ static int link_parse_cb(const struct nlmsghdr *nlh, void *data) > > link_print_lmc(rd, tb); > > link_print_state(rd, tb); > > link_print_phys_state(rd, tb); > > + link_print_netdev(rd, tb); > > if (rd->show_details) > > link_print_caps(rd, tb); > > > > diff --git a/rdma/utils.c b/rdma/utils.c > > index f9460162..4fed80ab 100644 > > --- a/rdma/utils.c > > +++ b/rdma/utils.c > > @@ -375,6 +375,8 @@ static const enum mnl_attr_data_type nldev_policy[RDMA_NLDEV_ATTR_MAX] = { > > [RDMA_NLDEV_ATTR_RES_STATE] = MNL_TYPE_U8, > > [RDMA_NLDEV_ATTR_RES_PID] = MNL_TYPE_U32, > > [RDMA_NLDEV_ATTR_RES_KERN_NAME] = MNL_TYPE_NUL_STRING, > > + [RDMA_NLDEV_ATTR_NDEV_INDEX] = MNL_TYPE_U32, > > + [RDMA_NLDEV_ATTR_NDEV_NAME] = MNL_TYPE_NUL_STRING, > > }; > > > > int rd_attr_cb(const struct nlattr *attr, void *data) > > -- > > 2.14.3 > > > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > > the body of a message to majordomo@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html > --7SrMUQONj8Rl9QNG Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAEBCAAdFiEEkhr/r4Op1/04yqaB5GN7iDZyWKcFAlq5J0sACgkQ5GN7iDZy WKd4Pw/7B9FtyDhrOFl1gNQOva4ank05Zl+8KPIPt0rcG/xwu6vgjnDP++xCBrkN 0Vm+094YbMh0mey1QZkIl54us0PNJVdOZ7SMFY+qqoXgKdga7WtOvLJskIzhlCM6 wI/cBNPDDImiMg4th45xZXamydq157AGktzyQRBMqvKRjGt4jpweY0juzr4IoXju Z526bZEBMhPfN2jqoTZfkK+zzS1qvnVKdmgeVwM4IC4VIcjoJWQ1FHBx9mhYlUUS 9fRgDj0Dv8APrBK+xcPuoNz+Y2DbPf6ZZf9qTf3v/ox1Cmt2yxinbSXLBXSha4H8 jxBG2zHJwhqNlL2Fv8u2ACX3sfGbQNEwTC1NoBS64kLSjzYDGyovBdBm9KnjtghF E7JyT7q9cXHk9M1M11W57s2zZYGOqKplfV/ecDeC3+g13PzHUFJocKijsmg5Gv50 1IxKtn293j3b5F18FQZtMnoKiD3mZt8D3pE2IHbD+PSpP1UTKMAy/lmZeI3Spuck qy8hAwQsCQBk07tC0S1WxBsHYN1Hg1V/OUmNL+vjI9KX4jpex/oV0njhYMwrfM9P xaBJxvqAK/iQrUlqSSobD4sK0BvAxaqWX2rzRsSMeZOJsrpk1LB8v6b0epXcPT1V +q4iHLZ2QKDal1WMeNlPLt7AoPkMX5DFvI754EO+FZUY2jkCGu0= =9/XL -----END PGP SIGNATURE----- --7SrMUQONj8Rl9QNG--