[PATCH] optimize aio rw path

Jeremy Allison jra at samba.org
Wed Mar 26 22:32:37 MDT 2014


On Wed, Mar 26, 2014 at 01:39:53PM +0100, Volker Lendecke wrote:
> Hi!
> 
> Attached find a few patches that optimize our async I/O
> path. It might be a bit controversial, because it uses C99
> variable arrays. Here in this situation I think it really
> reduces code complexity and is important for performance.
> 
> Review would be appreciated!

LGTM, pushed to autobuild. The only
thing that made me think a bit is
when you create the initial jobs
array you zero it via calloc, but
after you expand it via realloc/memcpy
the expanded area isn't zeroed.

Then I remembered the reason for
this patch is it's performance
critical code, and that if that
memory ever got read before
being written it's a horrible
logic bug anyway :-).

Cheers,

	Jeremy.


More information about the samba-technical mailing list