From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.7 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_PASS,USER_AGENT_MUTT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id E182BC43381 for ; Thu, 21 Feb 2019 15:39:46 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id A301C2084D for ; Thu, 21 Feb 2019 15:39:46 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1550763586; bh=SQu27S0GWPZzfJgN39kBSJFVqJhj9piaktnnjLouXS8=; h=Date:From:To:Cc:Subject:References:In-Reply-To:List-ID:From; b=YZPKNBW6KmWdTCXyeEARjyl9FjnLwA+oTFNjLv3qQXlN5ibIhLrCuwJ+JnZYvnL+/ dnCwIhL00lciawIicNSbkwWvuVIzmZAyxpvlthBwMiJU5bn3bx4WKsgbv5p30R9bkz cU+5KI+6ROY1fg0HeZbJhN/WcPXG1117gAtmJy1U= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727295AbfBUPjp (ORCPT ); Thu, 21 Feb 2019 10:39:45 -0500 Received: from mail.kernel.org ([198.145.29.99]:39748 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725866AbfBUPjp (ORCPT ); Thu, 21 Feb 2019 10:39:45 -0500 Received: from localhost (unknown [193.47.165.251]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id E317A2075A; Thu, 21 Feb 2019 15:39:43 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1550763584; bh=SQu27S0GWPZzfJgN39kBSJFVqJhj9piaktnnjLouXS8=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=iI0+kpYk4pFKWiy3YYsP69R4VoZMQsJttYYNzBHKO3sdkxr2r8UFzBfMULzxukeyQ MC+ELTdQJC9DGQO99uUZP3O4iUmOMoa7U0NXfcDJcIi1sDtrQWXa52St3YVasCY7G3 ZJQzr5c4PTKwb+RfTHHiJlXcz5X0HlGkmk+2aD90= Date: Thu, 21 Feb 2019 17:39:38 +0200 From: Leon Romanovsky To: David Ahern Cc: netdev , RDMA mailing list , Stephen Hemminger Subject: Re: [PATCH iproute2-next v1 02/19] rdma: Remove duplicated print code Message-ID: <20190221153938.GC23561@mtr-leonro.mtl.com> References: <20190220072136.3840-1-leon@kernel.org> <20190220072136.3840-3-leon@kernel.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="Pd0ReVV5GZGQvF3a" Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.10.1 (2018-07-13) Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org --Pd0ReVV5GZGQvF3a Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Thu, Feb 21, 2019 at 10:23:01AM -0500, David Ahern wrote: > On 2/20/19 2:21 AM, Leon Romanovsky wrote: > > From: Leon Romanovsky > > > > There is no need to keep same print functions for > > uint32_t and uint64_t, unify them into one function. > > > > Signed-off-by: Leon Romanovsky > > --- > > rdma/res.c | 22 +++++++--------------- > > 1 file changed, 7 insertions(+), 15 deletions(-) > > > > diff --git a/rdma/res.c b/rdma/res.c > > index 6b0f5fe3..87865ec8 100644 > > --- a/rdma/res.c > > +++ b/rdma/res.c > > @@ -808,28 +808,20 @@ static int res_cq_parse_cb(const struct nlmsghdr *nlh, void *data) > > return MNL_CB_OK; > > } > > > > -static void print_key(struct rd *rd, const char *name, uint32_t val) > > +static void print_key(struct rd *rd, const char *name, uint64_t val) > > { > > if (rd->json_output) > > jsonw_xint_field(rd->jw, name, val); > > else > > - pr_out("%s 0x%x ", name, val); > > + pr_out("%s 0x%" PRIx64 " ", name, val); > > } > > what's the plan to move rdma tool to iproute2's json functions? > > I realize rdma tool inherited this from devlink, but this command does > not need to continue managing this. I can work on this next cycle, after I'll finish to code some obligations in rdma-core. Thanks --Pd0ReVV5GZGQvF3a Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIcBAEBAgAGBQJcbsY6AAoJEORje4g2clinlPMP/1JmbbUOh0rG7tpii5bN4MxT rXMBrNsECetzSFc4fG61aTp2IXbPTkrcH5MnxFNyfU6DoFWhK+bQLj2fUOY2EXv8 4EZ2X9PJ2h9OUf1w1ftqkfDpwBrNvcNOt8gVgbi0bWpe5A+gdTyPNrSx5m6/9x6f deEbvTJvN3JpIKTBRLGt/5lTy8lKgPDbp94nKNnWLIFZRT6Q4fAoXOSLllbtTL/T rAfGMIDkdjwznjB2X8NrkUxX1f3AIwblS6Bd55Y8oKkCAxtNbPJmkUeX02He4KWq lxojt1X+IHYQMsEXX42qEh63gNO7D/ojhKopGbLv89dZ+ykiEXXyxS7hkx1u/jj+ 9Lx3YPW4QIwPJlT93WixgBegjNJ4Y7NRjfHTw2/1lE4/e/TnXSIckde4AFy1T17s Em8O9cNNiroedbnnaOHAQ356Q/xuGWywPDEnvQzp9FZD5Of5CWFAWRdAWg905qA9 +xtQdmWBGhQjRglhqn1CPKcDxRHMpPdYwxyc/LAHXp/nbQKT9KBysaMuSD6YuzUb T5ok7b95JHJ0NdUUtt089YLxQMDVBfIMZAyarkBCR7Hsi8SZn0xYrJrDMPOr+uJK EZhJPnVVLz/iK8WC0O3smJdC/efxrdwgTkMRYrXfpfP/+qT6Q7d9GWptCpNWkrhK +ldSqF7frULxIHAtYy/2 =ei2+ -----END PGP SIGNATURE----- --Pd0ReVV5GZGQvF3a--