apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Colm MacCarthaigh <c...@stdlib.net>
Subject Re: [PATCH] Configurable read/write buffer sizes
Date Sun, 18 Sep 2005 22:27:32 GMT

Anyone had a change to look at any of the below? (Thanks for committing
the docs nit Garrett).

01-flags-stdio.patch is the one I actually have a dependency on right
now, and is also relatively simple. I'd really appreciate some feedback
on that. 

I can live with arguing a case for variable buffer sizes. If benchmarks
will help; I can run a tonne. I've seeing some pretty nifty speed-ups
when I apply large buffer sizes to parts of httpd (and sometimes even
suprising places, like rotatelogs).

If it helps, I'm usually in #apr, feel free to bug me there. 

On Fri, Sep 09, 2005 at 02:57:40PM +0100, Colm MacCarthaigh wrote:
> O.k., I've done some new patches and a few read throughs to think the
> implications of a large buffer size out, and implemented a flags-capable
> variant of the stdin/out/err openers. 
> 
> I believe apr_size_t is safe, apart from the circumstance where someone
> tries to assign a vlue to the buffer size that would overflow an
> apr_size_t anyway, so not worth worrying about.
> 
> The patches are all at;
> 
>     http://people.apache.org/~colm/apr-IO-patches/
> 
> They are all against trunk, but can be applied in sequence, there's only
> one slight offset change that patch has to figure out.
> 
>     001-flags-stdio.patch 	 Implement apr_file_open_flags_std(err|out|in)
>                                 Nothing major, but I've also set APR_READ
>                                 or APR_WRITE as appropriate, purely for the
>                                 sake of apr_file_flags_get();
> 
>     002-set-buffer.patch        APR-wide changes to implement 
>                                 apr_file_set_buffer() and apr_file_buffer_size_get(),
>                                 includes a testcase
> 
>     003-set-buffer-unix.patch   Unix implementation of same, tested on Ubuntu
> 
>     004-set-buffer-win32.patch  Win32 implementation, tested w2k3. I didn't include
>                                 the changes to the dsp files, because VS2005 to behave.
> 
>     005-set-buffer-netware.patch Netware header changes neccessary, untested
> 
>     006-set-buffer-os2.patch    OS implementation, untested, based on inspection	
> 
> -- 
> Colm MacCárthaigh                        Public Key: colm+pgp@stdlib.net
> 

-- 
Colm MacCárthaigh                        Public Key: colm+pgp@stdlib.net

Mime
View raw message