From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from rcdn-iport-7.cisco.com (rcdn-iport-7.cisco.com [173.37.86.78]) by dpdk.org (Postfix) with ESMTP id EA15F1B1EB for ; Tue, 10 Oct 2017 19:30:41 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=4460; q=dns/txt; s=iport; t=1507656641; x=1508866241; h=from:to:subject:date:message-id:references:in-reply-to: content-transfer-encoding:mime-version; bh=3qDfDkYMBE2ZIzHrI1N0ldEfgTW70hVCb3hsWcMw8F0=; b=iPBNR30+tMUDhR3jaNFk0iWRDa2H7v1F99X03JTGLLuIB3y2NGmlhxWl BcevvKzsWFkD8G0unw5tuzn6GHrqjd02po6uTLdtaaGmDOBmLEmj1q/dZ RSK/Ur30XTa3jlBpuHiVm6M3DC6grH4iAi69ehAcDbDSGBXlY53HJJnEV w=; X-IronPort-Anti-Spam-Filtered: true X-IronPort-Anti-Spam-Result: =?us-ascii?q?A0CdAAD7At1Z/4kNJK1dGQEBAQEBAQEBA?= =?us-ascii?q?QEBBwEBAQEBg1uBUicHjhKPdIF2li+CEgqFOwKETj8YAQIBAQEBAQEBayiFHQE?= =?us-ascii?q?BAQEDJxNLBAIBCA4DBAEBHwkHMhQJCAIEARIIihyqGzqLMQEBAQEBAQEBAQEBA?= =?us-ascii?q?QEBAQEBAQEBAR2DLYICgVGFE4p4BYoMhzWBa44QApRegh2Qe4odixcCERkBgTg?= =?us-ascii?q?BHziBDngVh2Z2iTKBEAEBAQ?= X-IronPort-AV: E=Sophos;i="5.43,359,1503360000"; d="scan'208";a="304180086" Received: from alln-core-4.cisco.com ([173.36.13.137]) by rcdn-iport-7.cisco.com with ESMTP/TLS/DHE-RSA-AES256-SHA; 10 Oct 2017 17:30:40 +0000 Received: from XCH-ALN-007.cisco.com (xch-aln-007.cisco.com [173.36.7.17]) by alln-core-4.cisco.com (8.14.5/8.14.5) with ESMTP id v9AHUe1W001755 (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=FAIL); Tue, 10 Oct 2017 17:30:40 GMT Received: from xch-rcd-007.cisco.com (173.37.102.17) by XCH-ALN-007.cisco.com (173.36.7.17) with Microsoft SMTP Server (TLS) id 15.0.1320.4; Tue, 10 Oct 2017 12:30:39 -0500 Received: from xch-rcd-007.cisco.com ([173.37.102.17]) by XCH-RCD-007.cisco.com ([173.37.102.17]) with mapi id 15.00.1320.000; Tue, 10 Oct 2017 12:30:39 -0500 From: "John Daley (johndale)" To: Aaron Conole , "dev@dpdk.org" Thread-Topic: [dpdk-dev] [RFC 1/4] enic: update format string to match arg types Thread-Index: AQHTNvjP983dKThGpk+7XROYPL5NXaLdazhQ Date: Tue, 10 Oct 2017 17:30:39 +0000 Message-ID: References: <20170926185329.2776-1-aconole@redhat.com> <20170926185329.2776-2-aconole@redhat.com> In-Reply-To: <20170926185329.2776-2-aconole@redhat.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ms-exchange-transport-fromentityheader: Hosted x-originating-ip: [10.19.145.149] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dpdk-dev] [RFC 1/4] enic: update format string to match arg types X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 10 Oct 2017 17:30:42 -0000 > -----Original Message----- > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Aaron Conole > Sent: Tuesday, September 26, 2017 11:53 AM > To: dev@dpdk.org > Subject: [dpdk-dev] [RFC 1/4] enic: update format string to match arg typ= es >=20 > The argument `index` (and unique_id) is unsigned, but the format string t= ype > used was for signed types. >=20 > Signed-off-by: Aaron Conole > --- > drivers/net/enic/base/vnic_cq.c | 4 ++-- drivers/net/enic/base/vnic_de= v.c > | 6 +++--- drivers/net/enic/base/vnic_rq.c | 4 ++-- > drivers/net/enic/base/vnic_wq.c | 2 +- > 4 files changed, 8 insertions(+), 8 deletions(-) >=20 > diff --git a/drivers/net/enic/base/vnic_cq.c > b/drivers/net/enic/base/vnic_cq.c index 2f65f35..33db10f 100644 > --- a/drivers/net/enic/base/vnic_cq.c > +++ b/drivers/net/enic/base/vnic_cq.c > @@ -65,11 +65,11 @@ int vnic_cq_alloc(struct vnic_dev *vdev, struct > vnic_cq *cq, unsigned int index, >=20 > cq->ctrl =3D vnic_dev_get_res(vdev, RES_TYPE_CQ, index); > if (!cq->ctrl) { > - pr_err("Failed to hook CQ[%d] resource\n", index); > + pr_err("Failed to hook CQ[%u] resource\n", index); > return -EINVAL; > } >=20 > - snprintf(res_name, sizeof(res_name), "%d-cq-%d", instance++, > index); > + snprintf(res_name, sizeof(res_name), "%d-cq-%u", instance++, > index); > err =3D vnic_dev_alloc_desc_ring(vdev, &cq->ring, desc_count, > desc_size, > socket_id, res_name); > if (err) > diff --git a/drivers/net/enic/base/vnic_dev.c > b/drivers/net/enic/base/vnic_dev.c > index 49b3655..808f95b 100644 > --- a/drivers/net/enic/base/vnic_dev.c > +++ b/drivers/net/enic/base/vnic_dev.c > @@ -650,7 +650,7 @@ int vnic_dev_stats_dump(struct vnic_dev *vdev, > struct vnic_stats **stats) >=20 > if (!vdev->stats) { > snprintf((char *)name, sizeof(name), > - "vnic_stats-%d", instance++); > + "vnic_stats-%u", instance++); > vdev->stats =3D vdev->alloc_consistent(vdev->priv, > sizeof(struct vnic_stats), &vdev->stats_pa, (u8 > *)name); > if (!vdev->stats) > @@ -900,7 +900,7 @@ int vnic_dev_notify_set(struct vnic_dev *vdev, u16 > intr) > } > if (!vnic_dev_in_reset(vdev)) { > snprintf((char *)name, sizeof(name), > - "vnic_notify-%d", instance++); > + "vnic_notify-%u", instance++); > notify_addr =3D vdev->alloc_consistent(vdev->priv, > sizeof(struct vnic_devcmd_notify), > ¬ify_pa, (u8 *)name); > @@ -1150,7 +1150,7 @@ int vnic_dev_classifier(struct vnic_dev *vdev, u8 > cmd, u16 *entry, > tlv_size =3D filter_size + action_size + > 2*sizeof(struct filter_tlv); > snprintf((char *)z_name, sizeof(z_name), > - "vnic_clsf_%d", unique_id++); > + "vnic_clsf_%u", unique_id++); > tlv_va =3D vdev->alloc_consistent(vdev->priv, > tlv_size, &tlv_pa, (u8 *)z_name); > if (!tlv_va) > diff --git a/drivers/net/enic/base/vnic_rq.c > b/drivers/net/enic/base/vnic_rq.c index 10a40c1..776a6f7 100644 > --- a/drivers/net/enic/base/vnic_rq.c > +++ b/drivers/net/enic/base/vnic_rq.c > @@ -58,13 +58,13 @@ int vnic_rq_alloc(struct vnic_dev *vdev, struct vnic_= rq > *rq, unsigned int index, >=20 > rq->ctrl =3D vnic_dev_get_res(vdev, RES_TYPE_RQ, index); > if (!rq->ctrl) { > - pr_err("Failed to hook RQ[%d] resource\n", index); > + pr_err("Failed to hook RQ[%u] resource\n", index); > return -EINVAL; > } >=20 > vnic_rq_disable(rq); >=20 > - snprintf(res_name, sizeof(res_name), "%d-rq-%d", instance++, > index); > + snprintf(res_name, sizeof(res_name), "%d-rq-%u", instance++, > index); > rc =3D vnic_dev_alloc_desc_ring(vdev, &rq->ring, desc_count, > desc_size, > rq->socket_id, res_name); > return rc; > diff --git a/drivers/net/enic/base/vnic_wq.c > b/drivers/net/enic/base/vnic_wq.c index 7c4119c..273e3db 100644 > --- a/drivers/net/enic/base/vnic_wq.c > +++ b/drivers/net/enic/base/vnic_wq.c > @@ -52,7 +52,7 @@ int vnic_wq_alloc_ring(struct vnic_dev *vdev, struct > vnic_wq *wq, > char res_name[NAME_MAX]; > static int instance; >=20 > - snprintf(res_name, sizeof(res_name), "%d-wq-%d", instance++, wq- > >index); > + snprintf(res_name, sizeof(res_name), "%d-wq-%u", instance++, > +wq->index); > return vnic_dev_alloc_desc_ring(vdev, &wq->ring, desc_count, > desc_size, > wq->socket_id, res_name); > } > -- > 2.9.5 Reviewed-by: John Daley Thanks, Johnd