Nathan Bossart <nathandbossart(at)gmail(dot)com> writes:
> On Fri, Jul 29, 2022 at 12:43:45PM -0400, Reid Thompson wrote:
>> slab.c
>> does not in SlabContextCreate(). Is this intentional, it seems to be an
>> oversight to me.
> IIUC this is because the header is tracked separately from the first
> regular block, unlike aset.c.
That doesn't make it not an oversight, though. It looks like aset.c
thinks that mem_allocated includes all the context's overhead, whereas
this implementation doesn't seem to have that result. The comments
associated with mem_allocated are sufficiently vague that it's impossible
to tell which implementation is correct. Maybe we don't really care,
but ...
regards, tom lane