Re: [PATCH] Core: merge adjacent free slab pages to ameliorate fragmentation from multi-page blocks (Was Re: Help with shared memory usage)

2014-06-25 Thread idina
Sent by Bird Mail___ nginx-devel mailing list nginx-devel@nginx.org http://mailman.nginx.org/mailman/listinfo/nginx-devel

Re: [PATCH] Core: merge adjacent free slab pages to ameliorate fragmentation from multi-page blocks (Was Re: Help with shared memory usage)

2014-06-03 Thread Wandenberg Peixoto
Thanks agentzh, Maxim and Igor On Tue, Jun 3, 2014 at 11:00 AM, Maxim Dounin wrote: > Hello! > > On Mon, Jun 02, 2014 at 08:42:50PM +0400, Maxim Dounin wrote: > > > Hello! > > > > On Sat, May 31, 2014 at 11:46:28PM -0300, Wandenberg Peixoto wrote: > > > > > Hello Maxim, > > > > > > I executed m

Re: [PATCH] Core: merge adjacent free slab pages to ameliorate fragmentation from multi-page blocks (Was Re: Help with shared memory usage)

2014-06-03 Thread Maxim Dounin
Hello! On Mon, Jun 02, 2014 at 08:42:50PM +0400, Maxim Dounin wrote: > Hello! > > On Sat, May 31, 2014 at 11:46:28PM -0300, Wandenberg Peixoto wrote: > > > Hello Maxim, > > > > I executed my tests again and seems that your improved patch version is > > working fine too. > > Good, thanks for t

Re: [PATCH] Core: merge adjacent free slab pages to ameliorate fragmentation from multi-page blocks (Was Re: Help with shared memory usage)

2014-06-02 Thread Maxim Dounin
Hello! On Sat, May 31, 2014 at 11:46:28PM -0300, Wandenberg Peixoto wrote: > Hello Maxim, > > I executed my tests again and seems that your improved patch version is > working fine too. Good, thanks for testing. > Did you plan to merge it on nginx core soon? It's currently waiting for Igor's

Re: [PATCH] Core: merge adjacent free slab pages to ameliorate fragmentation from multi-page blocks (Was Re: Help with shared memory usage)

2014-05-31 Thread Yichun Zhang (agentzh)
Hi Maxim! On Wed, May 28, 2014 at 11:38 AM, Maxim Dounin wrote: > > Below is a patch which does mostly the same without introducing > any additional per-page fields. Please take a look if it works > for you. > Thank you for looking into this! I've run my local test suite for this issue against

Re: [PATCH] Core: merge adjacent free slab pages to ameliorate fragmentation from multi-page blocks (Was Re: Help with shared memory usage)

2014-05-31 Thread Wandenberg Peixoto
Hello Maxim, I executed my tests again and seems that your improved patch version is working fine too. Did you plan to merge it on nginx core soon? -agentzh Did you have opportunity to check if it works for you? Regards On Wed, May 28, 2014 at 3:38 PM, Maxim Dounin wrote: > Hello! > > On

Re: [PATCH] Core: merge adjacent free slab pages to ameliorate fragmentation from multi-page blocks (Was Re: Help with shared memory usage)

2014-05-28 Thread Maxim Dounin
Hello! On Sun, May 11, 2014 at 10:13:52PM -0700, Yichun Zhang (agentzh) wrote: > Hello! > > On Mon, Jul 29, 2013 at 10:11 AM, Maxim Dounin wrote: > > Additionally, doing a full merge of all free blocks on a free > > operation looks too much. It might be something we want to do on > > allocation

[PATCH] Core: merge adjacent free slab pages to ameliorate fragmentation from multi-page blocks (Was Re: Help with shared memory usage)

2014-05-11 Thread Yichun Zhang (agentzh)
Hello! On Mon, Jul 29, 2013 at 10:11 AM, Maxim Dounin wrote: > Additionally, doing a full merge of all free blocks on a free > operation looks too much. It might be something we want to do on > allocation failure, but not on a normal path in > ngx_slab_free_pages(). And/or something lightweight