From mboxrd@z Thu Jan 1 00:00:00 1970 From: "John Daley (johndale)" Subject: Re: [RFC 1/4] enic: update format string to match arg types Date: Tue, 10 Oct 2017 17:30:39 +0000 Message-ID: References: <20170926185329.2776-1-aconole@redhat.com> <20170926185329.2776-2-aconole@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable To: Aaron Conole , "dev@dpdk.org" 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) In-Reply-To: <20170926185329.2776-2-aconole@redhat.com> Content-Language: en-US List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" > -----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