The pthreadpool_atfork handlers don't work correctly when the parent process has idle threads (possibly from messaging_dgm). Have patch, need bugnumber.
Created attachment 13530 [details] git-am fix for 4.7.0. Back-port to 4.6.x might be trickier...
Created attachment 13531 [details] git-am back-port of patch to 4.6.x. Doesn't include test.
Created attachment 13532 [details] git-am full backport for 4.6.x - includes test.
Created attachment 13533 [details] git-am partial backport for 4.5.x - test depends on pthreadpool_pipe_add_job() which doesn't exist in 4.5.x.
Comment on attachment 13533 [details] git-am partial backport for 4.5.x - test depends on pthreadpool_pipe_add_job() which doesn't exist in 4.5.x. While the 4.5 backport patches in precisely what went into master and the others, I would be a bit more careful here. We changed from dependent to detached threads with 4.6, so I could imagine that we need a pthread_join call in the pthreadpool_parent() code. Otherwise we might end up with data structures for zombie threads in the child. I'd need to spend more time than I have right now on this, in particular port the test. Can we talk about that at SDC? For now, no "+" on the 4.5 port until you convince me it's correct :-)
Created attachment 13551 [details] Patch This one is the reason why I did not upload the patches yet. I had wanted to make that in one run, but apparently I was just too slow for our development methodology, sorry for that.
(In reply to Volker Lendecke from comment #6) > This one is the reason why I did not upload the patches yet. Applies cleanly to 4.6 as well.
Let's just target 4.6.x, 4.7.0 with these fixes and not do a 4.5.x fix yet.
Re-assigning to Karolin for inclusion in 4.7.0, 4.6.next.
(In reply to Jeremy Allison from comment #9) Pushed to autobuild-v4-{7,6}-test.
Karolin, don't forget to add attachment #13551 [details] : https://bugzilla.samba.org/attachment.cgi?id=13551 to both 4.7.0 and 4.6.x, that is also needed for the complete bugfix. Thanks ! Jeremy.
(In reply to Jeremy Allison from comment #11) Thanks for the hint, Jeremy! Pushed to autobuild-v4-{6,7}-test.
Pushed to both branches. Closing out bug report. Thanks!