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 45B78A0541; Fri, 7 Feb 2020 04:45:27 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 7D8C41C07F; Fri, 7 Feb 2020 04:45:26 +0100 (CET) Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by dpdk.org (Postfix) with ESMTP id 8924D1C07B for ; Fri, 7 Feb 2020 04:45:24 +0100 (CET) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga105.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 06 Feb 2020 19:45:23 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.70,411,1574150400"; d="scan'208,217";a="311909182" Received: from pkadam-mobl.amr.corp.intel.com (HELO [10.251.146.214]) ([10.251.146.214]) by orsmga001.jf.intel.com with ESMTP; 06 Feb 2020 19:45:22 -0800 To: Dmitry Kozlyuk Cc: dev@dpdk.org, thomas@monjalon.net, Harini.Ramakrishnan@microsoft.com, keith.wiles@intel.com, bruce.richardson@intel.com, david.marchand@redhat.com, jerinjacobk@gmail.com, ranjit.menon@intel.com, antara.ganesh.kolar@intel.com References: <20200131000307.10608-1-pallavi.kadam@intel.com> <20200201000406.11060-1-pallavi.kadam@intel.com> <20200201000406.11060-10-pallavi.kadam@intel.com> <20200205225441.3d833bae@Sovereign> <63c486b2-b53a-ffe8-fcdf-4fd207132df2@intel.com> <50a4ad5d-d1f2-52a7-5265-e3f0c88114c5@intel.com> <20200206094137.66a3bc7e@Sovereign> From: Pallavi Kadam Message-ID: <6966c67f-70cb-8b65-8036-99e7ef566f9c@intel.com> Date: Thu, 6 Feb 2020 19:45:22 -0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:60.0) Gecko/20100101 Thunderbird/60.9.1 MIME-Version: 1.0 In-Reply-To: <20200206094137.66a3bc7e@Sovereign> Content-Language: en-US Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit X-Content-Filtered-By: Mailman/MimeDel 2.1.15 Subject: Re: [dpdk-dev] [PATCH v7 9/9] eal: add minimum viable code to support parsing 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" Hi Dmitry, On 2/5/2020 10:41 PM, Dmitry Kozlyuk wrote: > >> On 2/5/2020 4:39 PM, Pallavi Kadam wrote: >>> >>> On 2/5/2020 11:54 AM, Dmitry Kozlyuk wrote: >>>> Crashes at argument parsing, WinDbg log attached. >>> This patch works fine with meson=0.49. >>> I was able to execute the 'helloworld' app. >>> >>> When I tried to build with meson=0.52 it failed with the error: >>> [9/25] Linking target lib/librte_kvargs-20.0.dll. >>> FAILED: lib/librte_kvargs-20.0.dll >>> clang @lib/librte_kvargs-20.0.dll.rsp >>> clang: error: no such file or directory: '/OPT:REF' >> Fixed this error with the patch you sent before. >> were able to execute parsing: >> >> $ ./build/examples/dpdk-helloworld.exe -l 4-8 >> EAL: Detected 20 lcore(s) >> EAL: Detected 2 NUMA nodes >> hello from core 5 >> hello from core 6 >> hello from core 7 >> hello from core 8 >> hello from core 4 >> >>> Not completely sure, if this could be the reason for the crash. > The crash does not happen with any arguments, "-l 4-8" works fine as do > "-cf" and "-v", but "--log-level=eal:8" crashes reproducibly. I assume > application should not crash on invalid options, but report an error and exit. > For the reference, I applied your patchset to the latest dpdk:master. > > Z:\>build\native\clang\examples\dpdk-helloworld.exe > EAL: Detected 4 lcore(s) > EAL: Detected 1 NUMA nodes > > OK, no lcores specified. > > Z:\>build\native\clang\examples\dpdk-helloworld.exe -cf > EAL: Detected 4 lcore(s) > EAL: Detected 1 NUMA nodes > hello from core 1 > hello from core 2 > hello from core 3 > hello from core 0 > > OK, 4 lcores enabled. > > Z:\>build\native\clang\examples\dpdk-helloworld.exe -cf --log-level=eal:8 > > Crash, WinDbg log attached to bug report. > > Z:\>echo %errorlevel% > -1073741819 > > Z:\> > > Attaching the build log, but I don't think it matters, it's something with > getopt implementation or it's use, judging by stack trace. Thanks for the detailed log. There was bit size issue in getopt file. We fixed the issue. Please try v8 and let us know. > > P.S. I can also see "--syslog" option in the help message, you should > probably hide it via #ifndef, as you did with said option handling. Incorporated in v8. Thanks >