[PATCH] Avoid a pipe read in aio result scheduling
Jeremy Allison
jra at samba.org
Mon Aug 22 23:38:26 UTC 2016
On Mon, Aug 22, 2016 at 12:09:29PM -0700, Jeremy Allison wrote:
> On Mon, Aug 22, 2016 at 05:27:35PM +0200, Volker Lendecke wrote:
> > Hi!
> >
> > Attached find a patchset that is supposed to improve our async I/O
> > result scheduling. For a more verbose explanation I'd recommend the
> > commit message for [PATCH 04/23].
> >
> > Jeremy, I did not look deeply enough into your threaded messaging
> > tevent code to see whether the tevent_thread_proxy_schedule()
> > implementation can be based upon tevent_threaded_schedule_immediate().
> >
> > Thanks metze for very deep review and insights.
> >
> > Comments?
>
> Oh it removes the asys stuff ! Very cool (deleted code is always
> good :-).
>
> I'll take some time and go over this really carefully asap (next
> day or so).
Just got through the tevent core changes.
Still reviewing - but the tevent changes are *amazingly* small
and clean for the change they make - great job !
Solves a different problem than the tevent_thread_proxy_schedule()
code does though I think - this new code changes the core
tevent loop scheduling whereas the tevent_thread_proxy_schedule()
stuff is essentially built on top of existing tevent code as
a utility function to pass memory back and forth.
So I don't feel like the older code is obsoleted :-) :-).
Once I've finished going through and pushed I'll see if
I can build the tevent_thread_proxy_schedule() on top
of the new core functionality to save having the extra
pipe().
More information about the samba-technical
mailing list