Re: [HACKERS] PATCH : Generational memory allocator (was PATCH: two slab-like memory allocators)

2017-09-24 Thread Simon Riggs
On 24 September 2017 at 21:32, Tomas Vondra wrote: > Attached is an updated version of the patch, tweaking the comments. That looks good, thanks. Marking Ready for Committer to give notice before commit. -- Simon Riggshttp://www.2ndQuadrant.com/

Re: [HACKERS] PATCH : Generational memory allocator (was PATCH: two slab-like memory allocators)

2017-09-24 Thread Tomas Vondra
Hi, Attached is an updated version of the patch, tweaking the comments. 1) I've added a section at the end of src/backend/utils/mmgr/README, briefly explaining the alternative memory allocators we have. I don't think we should get into too much low-level detail here, that's more appropriate for

Re: [HACKERS] PATCH : Generational memory allocator (was PATCH: two slab-like memory allocators)

2017-09-14 Thread Tomas Vondra
On 09/14/2017 04:21 PM, Simon Riggs wrote: > On 14 August 2017 at 01:35, Tomas Vondra wrote: >> Hi, >> >> Attached is a rebased version of the Generational context, originally >> submitted with SlabContext (which was already committed into Pg 10). >> >> The main

Re: [HACKERS] PATCH : Generational memory allocator (was PATCH: two slab-like memory allocators)

2017-09-14 Thread Simon Riggs
On 14 August 2017 at 01:35, Tomas Vondra wrote: > Hi, > > Attached is a rebased version of the Generational context, originally > submitted with SlabContext (which was already committed into Pg 10). > > The main change is that I've abandoned the pattern of defining a

[HACKERS] PATCH : Generational memory allocator (was PATCH: two slab-like memory allocators)

2017-08-13 Thread Tomas Vondra
Hi, Attached is a rebased version of the Generational context, originally submitted with SlabContext (which was already committed into Pg 10). The main change is that I've abandoned the pattern of defining a Data structure and then a pointer typedef, i.e. typedef struct