[PATCH] pthreadpool: create a tevent_threaded_context per registered event context

Jeremy Allison jra at samba.org
Thu Nov 16 18:14:27 UTC 2017


On Tue, Nov 14, 2017 at 04:35:40PM +0100, Ralph Böhme wrote:
> Hi!
> 
> Attached is a patch that enhances pthreadpool_tevent to create one
> tevent_threaded_context per registered event context instead of per
> pthreadpool_tevent_job_send request.
> 
> Please review carefully -- the rundown is a bit tricky -- and push if
> happy. Thanks!
> 
> Fwiw: it passed two private autobuilds and it survives smbtorture
> local.messaging tests with a modified messaging subsystem that sends out all
> messages via helper threads.

OK, how about this version. That actual code is exactly as you
posted, all I've done is thoroughly commented it such that if
I had to maintain this I'd know immediately what it was doing :-).

Let me know if this is OK !

Jeremy.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-pthreadpool-create-a-tevent_threaded_context-per-reg.patch
Type: text/x-diff
Size: 8309 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20171116/d49d048e/0001-pthreadpool-create-a-tevent_threaded_context-per-reg.diff>


More information about the samba-technical mailing list