Re: [dpdk-dev] [PATCH v9 02/10] bond: replace rte_panic instances in bonding driver

2018-04-26 Thread Thomas Monjalon
26/04/2018 23:06, Arnon Warshavsky: > By returning from this function in this patch I also aim to terminate the > process, but through the door rather than out the window, :D Thanks for the laugh. Where is the button "Like"?

Re: [dpdk-dev] [PATCH v9 02/10] bond: replace rte_panic instances in bonding driver

2018-04-26 Thread Arnon Warshavsky
Hi Kevin > /* Any memory allocation failure in initialization is critical > because > >* resources can't be free, so reinitialization is impossible. */ > > What about this comment? Not sure if it is just stale or if there is > something else you need to do. > This comment was origin

Re: [dpdk-dev] [PATCH v9 02/10] bond: replace rte_panic instances in bonding driver

2018-04-26 Thread Kevin Traynor
On 04/26/2018 07:20 AM, Arnon Warshavsky wrote: > Replace panic calls with log and return value. > Local functions to this file, > changing from void to int are non-abi-breaking > > Signed-off-by: Arnon Warshavsky > --- > drivers/net/bonding/rte_eth_bond_8023ad.c | 29 > ++--

[dpdk-dev] [PATCH v9 02/10] bond: replace rte_panic instances in bonding driver

2018-04-25 Thread Arnon Warshavsky
Replace panic calls with log and return value. Local functions to this file, changing from void to int are non-abi-breaking Signed-off-by: Arnon Warshavsky --- drivers/net/bonding/rte_eth_bond_8023ad.c | 29 ++- drivers/net/bonding/rte_eth_bond_8023ad_private.h | 2 +