Re: [dpdk-dev] [PATCH v3] eal: replace rte_panic instances to return an error value

2018-03-27 Thread Arnon Warshavsky
I now have a set of several patches which pass the abi-validation , 1 patch of checkpatches.sh to prevent future panic instances and the set of abi-breaking changes which are included the init sequence. 1. I assume it is best to bundle all to a single patchset. Please correct me otherwise 2. Tryi

Re: [dpdk-dev] [PATCH v3] eal: replace rte_panic instances to return an error value

2018-03-21 Thread Arnon Warshavsky
I'm a fan of such 1:1 baby steps, but I guess I will stick with the flavor of majority here. Will shift those to explicit log and return. On Wed, Mar 21, 2018 at 11:11 AM, Bruce Richardson < bruce.richard...@intel.com> wrote: > On Tue, Mar 20, 2018 at 11:53:08PM +0200, Arnon Warshavsky wrote: > >

Re: [dpdk-dev] [PATCH v3] eal: replace rte_panic instances to return an error value

2018-03-21 Thread Bruce Richardson
On Tue, Mar 20, 2018 at 11:53:08PM +0200, Arnon Warshavsky wrote: > The purpose of this patch is to cleanup the library code > from paths that end up aborting the process, > and move to checking error values, in order to allow the running process > perform an orderly tear-down or other mitigation o

[dpdk-dev] [PATCH v3] eal: replace rte_panic instances to return an error value

2018-03-20 Thread Arnon Warshavsky
The purpose of this patch is to cleanup the library code from paths that end up aborting the process, and move to checking error values, in order to allow the running process perform an orderly tear-down or other mitigation of the event. This patch modifies the majority of rte_panic calls under li