[dpdk-dev] [PATCH v3] eal: replace rte_panic instances to return an error value
Arnon Warshavsky
arnon at qwilt.com
Tue Mar 27 16:06:24 CEST 2018
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.
Trying to work around the abi in this phase
What is your take regarding adding a state to the init phase:
int rte_get_legacy_panic_state();
void rte_move_to_legacy_panic_state();
This will allow preserving abi for these few functions, replacing for
example the currently abi-breaking
if (eal_thread_init_master())
return -1
with
eal_thread_init_master()
if ( rte_get_legacy_panic_state())
return -1
while calling rte_move_to_legacy_panic_state() from within these void
functions where panic takes place today.
This can also partially serve the cases where panic is called from within
an interrupt handler, and have no applicative context to return to.
thanks
/Arnon
More information about the dev
mailing list