RE: [PATCH v3] lib/eal: fix segfaults due to thread exit order

2022-06-07 Thread Zeng, ZhichaoX
Hi David, Harman Please review this patch at your convenience. Thanks! In addition, I tried to figure out the reason for the failure of the CI test. There is a saying that this is a problem with Asan when cancelling a thread that is waiting on epoll. After adding the '-fstack-protector-all' pa

RE: [PATCH v3] lib/eal: fix segfaults due to thread exit order

2022-06-02 Thread Zeng, ZhichaoX
Olivier > > ; > tho...@monjalon.net; sta...@dpdk.org > Subject: Re: [PATCH v3] lib/eal: fix segfaults due to thread exit order > > On Mon, 30 May 2022 13:47:38 + > zhichaox.z...@intel.com wrote: > > @@ -883,6 +896,8 @@ rte_eal_init(int argc, char **argv) > > > >

Re: [PATCH v3] lib/eal: fix segfaults due to thread exit order

2022-05-30 Thread Stephen Hemminger
On Mon, 30 May 2022 13:47:38 + zhichaox.z...@intel.com wrote: > @@ -883,6 +896,8 @@ rte_eal_init(int argc, char **argv) > > eal_mcfg_complete(); > > + pthread_atfork(NULL, warn_parent, scratch_child); > + > return fctret; > } There are lots of other cases where DPDK will

Re: [PATCH v3] lib/eal: fix segfaults due to thread exit order

2022-05-30 Thread Stephen Hemminger
On Mon, 30 May 2022 13:47:38 + zhichaox.z...@intel.com wrote: > +static void warn_parent(void) > +{ > + RTE_LOG(WARNING, EAL, "fork() was called, DPDK won't work in the child " > + "process unless it calls rte_eal_init()\n"); > +} > + Too wordy. Don't break messages across lin