[PATCH] Improve talloc performance
Jeremy Allison
jra at samba.org
Wed Jun 29 23:55:23 UTC 2016
On Fri, Jun 17, 2016 at 05:12:12PM -0700, Jeremy Allison wrote:
> On Fri, Jun 17, 2016 at 09:50:27PM +1200, Andrew Bartlett wrote:
> > Attached is a patch to improve talloc performance, by calling
> > talloc_chunk_from_ptr() less often. I'm impressed by how much can be
> > gained!
> >
> > Please carefully review!
>
> I'm going through this right now, but one thing
> that has to be done is to split this into a set of smaller
> changesets as the chunks are too large to carefully parse
> mentally (at least for me). Too much change per patchset.
OK, split this into an 11-patch set, with each change
being somewhat clearer and smaller (IMHO of course :-).
Your original patchset had one error (missing setting
the .name in the replacement for talloc_vasprintf(),
and a couple of places where you added code, then removed
it again.
I'm still running tests on it at the moment, but at
least it compiles :-).
I'll post for your review once I've gotten it past
testing !
Cheers,
Jeremy.
More information about the samba-technical
mailing list