From mboxrd@z Thu Jan 1 00:00:00 1970 From: Aaron Conole Subject: Re: [PATCH v2 00/25] linux/eal: Remove most causes of panic on init Date: Wed, 08 Feb 2017 14:11:36 -0500 Message-ID: References: <20170208185142.28678-1-aconole@redhat.com> Mime-Version: 1.0 Content-Type: text/plain Cc: Stephen Hemminger , Bruce Richardson To: dev@dpdk.org Return-path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id 30C32952 for ; Wed, 8 Feb 2017 20:11:37 +0100 (CET) In-Reply-To: <20170208185142.28678-1-aconole@redhat.com> (Aaron Conole's message of "Wed, 8 Feb 2017 13:51:17 -0500") List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Aaron Conole writes: > In many cases, it's enough to simply let the application know that the > call to initialize DPDK has failed. A complete halt can then be > decided by the application based on error returned (and the app could > even attempt a possible re-attempt after some corrective action by the > user or application). > > Changes ->v2: > - Audited all "RTE_LOG (" calls that were introduced, and converted > to "RTE_LOG(" > - Added some fprintf(stderr, "") lines to indicate errors before logging > is initialized > - Removed assignments to errno. > - Changed patch 14/25 to reflect EFAULT, and document in 25/25 > > I kept the rte_errno reflection, since this is control-path code and the > init function returns a sentinel value of -1. > I got 3 new checkpatch warnings that seem to have missed my local commit hook. I'll fix them up and send a new series in-reply to this one. Sorry for the noise.