From patchwork Thu Feb 5 09:26:43 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: xuelin.shi@freescale.com X-Patchwork-Id: 3005 Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [IPv6:::1]) by dpdk.org (Postfix) with ESMTP id 8B1C6B425; Thu, 5 Feb 2015 10:26:46 +0100 (CET) Received: from na01-bn1-obe.outbound.protection.outlook.com (mail-bn1bn0101.outbound.protection.outlook.com [157.56.110.101]) by dpdk.org (Postfix) with ESMTP id 5E1FCB41E for ; Thu, 5 Feb 2015 10:26:45 +0100 (CET) Received: from CY1PR0301MB0684.namprd03.prod.outlook.com (25.160.158.155) by CY1PR0301MB0682.namprd03.prod.outlook.com (25.160.158.152) with Microsoft SMTP Server (TLS) id 15.1.75.20; Thu, 5 Feb 2015 09:26:44 +0000 Received: from CY1PR0301MB0684.namprd03.prod.outlook.com ([25.160.158.155]) by CY1PR0301MB0684.namprd03.prod.outlook.com ([25.160.158.155]) with mapi id 15.01.0075.002; Thu, 5 Feb 2015 09:26:44 +0000 From: Xuelin Shi To: "dev@dpdk.org" Thread-Topic: [PATCH] fix testpmd show port info error Thread-Index: AQHQPr2TY/Z2hG6+xk2PO/Fhu1pFsJzhzS1A Date: Thu, 5 Feb 2015 09:26:43 +0000 Message-ID: References: <1422859901-3896-1-git-send-email-xuelin.shi@freescale.com> In-Reply-To: <1422859901-3896-1-git-send-email-xuelin.shi@freescale.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [123.151.195.51] authentication-results: dpdk.org; dkim=none (message not signed) header.d=none; x-microsoft-antispam: BCL:0;PCL:0;RULEID:;SRVR:CY1PR0301MB0682; x-exchange-antispam-report-test: UriScan:; x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:; SRVR:CY1PR0301MB0682; x-forefront-prvs: 0478C23FE0 x-forefront-antispam-report: SFV:NSPM; SFS:(10019020)(6009001)(13464003)(164054003)(2900100001)(92566002)(110136001)(46102003)(2656002)(2501002)(50986999)(106116001)(76176999)(2950100001)(102836002)(99286002)(87936001)(33656002)(76576001)(122556002)(62966003)(77156002)(450100001)(40100003)(107886001)(74316001)(2351001)(19580405001)(19580395003)(86362001)(66066001)(54356999); DIR:OUT; SFP:1102; SCL:1; SRVR:CY1PR0301MB0682; H:CY1PR0301MB0684.namprd03.prod.outlook.com; FPR:; SPF:None; MLV:sfv; LANG:en; MIME-Version: 1.0 X-OriginatorOrg: freescale.com X-MS-Exchange-CrossTenant-originalarrivaltime: 05 Feb 2015 09:26:43.5566 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 710a03f5-10f6-4d38-9ff4-a80b81da590d X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY1PR0301MB0682 Subject: Re: [dpdk-dev] [PATCH] fix testpmd show port info error X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Hi, Anybody interested in this patch could have a review or comment on it? I'm new here. Should I send this patch to some specific maintainer to make this more efficient? Thanks, Shi Xuelin -----Original Message----- From: xuelin.shi@freescale.com [mailto:xuelin.shi@freescale.com] Sent: Monday, February 02, 2015 14:52 To: dev@dpdk.org Cc: Shi Xuelin-B29237 Subject: [PATCH] fix testpmd show port info error From: Xuelin Shi the port number type should be consistent with librte_cmdline, else there is potential endian issue. Signed-off-by: Xuelin Shi Acked-by: Olivier Matz --- app/test-pmd/cmdline.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 1.9.1 diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c index 4beb404..488ac63 100644 --- a/app/test-pmd/cmdline.c +++ b/app/test-pmd/cmdline.c @@ -5568,7 +5568,7 @@ cmdline_parse_token_string_t cmd_showport_what = TOKEN_STRING_INITIALIZER(struct cmd_showport_result, what, "info#stats#xstats#fdir#stat_qmap"); cmdline_parse_token_num_t cmd_showport_portnum = - TOKEN_NUM_INITIALIZER(struct cmd_showport_result, portnum, INT32); + TOKEN_NUM_INITIALIZER(struct cmd_showport_result, portnum, INT8); cmdline_parse_inst_t cmd_showport = { .f = cmd_showport_parsed,