From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH] eal/service: improve error checking of coremasks Date: Tue, 15 May 2018 17:38:01 +0200 Message-ID: <2371860.oF3rqsruhg@xps> References: <1526395965-133647-1-git-send-email-harry.van.haaren@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org, vipin.varghese@intel.com To: Harry van Haaren Return-path: Received: from out2-smtp.messagingengine.com (out2-smtp.messagingengine.com [66.111.4.26]) by dpdk.org (Postfix) with ESMTP id ED3CD1B74D for ; Tue, 15 May 2018 17:38:03 +0200 (CEST) In-Reply-To: <1526395965-133647-1-git-send-email-harry.van.haaren@intel.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 15/05/2018 16:52, Harry van Haaren: > + if (eal_service_cores_parsed()) > + RTE_LOG(ERR, EAL, > + "Warning: Service cores parsed before dataplane cores. Ensure -c is before -s or -S.\n"); > + [...] > + if (core_parsed && taken_lcore_count != count) { > + RTE_LOG(ERR, EAL, > + "Warning: not all service cores were in the coremask - ensure -c or -l includes service cores\n"); > + } Sometimes, messages are split with a dot, sometimes with a dash. Please choose only one for consistency (I prefer dot). I think you can split the string and wrap the line at the end of the sentence.