[dpdk-dev] [PATCH v4 10/11] eal: replace rte_panic instances in init sequence

Burakov, Anatoly anatoly.burakov at intel.com
Thu Apr 19 16:57:43 CEST 2018


On 19-Apr-18 3:48 PM, Arnon Warshavsky wrote:
> Copy on the commit message  and volatile.
> 
> Regarding the new function defunct_and_remain_in_endless_loop ()
> I don't think I can put that in a separate patch without breaking the 
> current patch independence.

How so?

Just leave some panic instances in there for thread-related stuff and 
fix them up in the next patch.

Also, i'm not sure sending threads into an infinite loop on panic is 
such a good idea. You might want to look at Olivier's approach [1] to 
creating threads, using pthread_barriers and pthread_kill/cancel.

This does warrant a separate patch now :)

-- 
Thanks,
Anatoly


More information about the dev mailing list