From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by dpdk.org (Postfix) with ESMTP id 0C80E37A2 for ; Fri, 27 Jul 2018 11:56:43 +0200 (CEST) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga003.jf.intel.com ([10.7.209.27]) by orsmga103.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 27 Jul 2018 02:56:42 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.51,408,1526367600"; d="scan'208";a="70414336" Received: from rnicolau-mobl.ger.corp.intel.com (HELO [10.237.221.82]) ([10.237.221.82]) by orsmga003.jf.intel.com with ESMTP; 27 Jul 2018 02:56:41 -0700 To: "De Lara Guarch, Pablo" , "dev@dpdk.org" Cc: "Hunt, David" , "Yao, Lei A" References: <1531921784-10578-1-git-send-email-radu.nicolau@intel.com> From: Radu Nicolau Message-ID: <76f02afd-a7b3-7f7f-3772-badd050cb6ee@intel.com> Date: Fri, 27 Jul 2018 10:56:40 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.5.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Content-Language: en-US Subject: Re: [dpdk-dev] [PATCH] examples/l3fw-power: do not exit on power lib init failure 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: Fri, 27 Jul 2018 09:56:44 -0000 On 7/26/2018 6:39 PM, De Lara Guarch, Pablo wrote: > Hi Radu, > >> -----Original Message----- >> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Radu Nicolau >> Sent: Wednesday, July 18, 2018 2:50 PM >> To: dev@dpdk.org >> Cc: Hunt, David ; Yao, Lei A ; >> Nicolau, Radu >> Subject: [dpdk-dev] [PATCH] examples/l3fw-power: do not exit on power lib init >> failure >> >> Do not exit the application if power library fails to initialize or high performance >> cores configuration cannot be used. > I would say this deserves more explanation. It doesn't look correct to me to keep log an error that > the power library couldn't be initialized and still the sample application keeps running. The application can also be used without the power library features, and this was also the previous behavior, before high/regular performance core feature was added. > > Actually, the way init_power_library is coded, it returns the output of the call of rte_power_init on the last core > (ret is overwritten for each core), which looks wrong to me. I will update init_power_library to return -1 if rte_power_init fails for at least one core. > > Thanks, > Pablo >> Signed-off-by: Radu Nicolau >> ---