From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id DF5E2A04B9; Mon, 7 Sep 2020 11:15:00 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 517191C0BE; Mon, 7 Sep 2020 11:15:00 +0200 (CEST) Received: from huawei.com (szxga07-in.huawei.com [45.249.212.35]) by dpdk.org (Postfix) with ESMTP id F0FB21BEE1 for ; Mon, 7 Sep 2020 11:14:58 +0200 (CEST) Received: from DGGEMS412-HUB.china.huawei.com (unknown [172.30.72.58]) by Forcepoint Email with ESMTP id AE8124E08F2C5F4F3388; Mon, 7 Sep 2020 17:14:57 +0800 (CST) Received: from [127.0.0.1] (10.65.81.238) by DGGEMS412-HUB.china.huawei.com (10.3.19.212) with Microsoft SMTP Server id 14.3.487.0; Mon, 7 Sep 2020 17:14:49 +0800 To: Stephen Hemminger References: <1592483709-7076-1-git-send-email-tangchengchang@huawei.com> <1599296854-56620-1-git-send-email-tangchengchang@huawei.com> <1599296854-56620-4-git-send-email-tangchengchang@huawei.com> <20200905095859.743243bb@hermes.lan> CC: , , , , , , From: Chengchang Tang Message-ID: <975a84de-1455-737d-25a0-cc5f37cf31d8@huawei.com> Date: Mon, 7 Sep 2020 17:14:48 +0800 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.7.1 MIME-Version: 1.0 In-Reply-To: <20200905095859.743243bb@hermes.lan> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit X-Originating-IP: [10.65.81.238] X-CFilter-Loop: Reflected Subject: Re: [dpdk-dev] [PATCH v4 3/5] app/procinfo: add Rx buffer size to --show-port 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: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 2020/9/6 0:59, Stephen Hemminger wrote: > On Sat, 5 Sep 2020 17:07:32 +0800 > Chengchang Tang wrote: > >> printf("\t -- queue %d rx scatter %d" >> " descriptors %d" >> + " rx buffer size %d" >> " offloads 0x%"PRIx64 >> " mempool socket %d\n", >> j, >> queue_info.scattered_rx, >> queue_info.nb_desc, >> + queue_info.rx_buf_size, >> queue_info.conf.offloads, >> queue_info.mp->socket_id); >> } > > These should be using %u and need space after " for PRIx64 > Why not: > printf("\t -- queue %u rx scatter %u" > " descriptors %u" > " rx buffer size %u" > " offloads %#" PRIx64 > " mempool socket %d\n", > OK, I will fix these in next version. Thanks