Re: [COMMITTERS] pgsql: Correctly align page's images in generic wal API - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: [COMMITTERS] pgsql: Correctly align page's images in generic wal API
Date
Msg-id CAB7nPqShY5XdW5CrDxsN5OUag+zb+TM-J4nYpMjisa+hQbLaCw@mail.gmail.com
Whole thread Raw
Responses Re: [COMMITTERS] pgsql: Correctly align page's images in generic wal API  (Teodor Sigaev <teodor@sigaev.ru>)
List pgsql-hackers
On Tue, May 17, 2016 at 6:01 AM, Teodor Sigaev <teodor@sigaev.ru> wrote:
> Correctly align page's images in generic wal API
>
> Page image should be MAXALIGN'ed because existing code could directly align
> pointers in page instead of align offset from beginning of page.
>
> Found during play with indexes as extenstion, Alexander Korotkov and me

Instead of allocating this memory unconditionally for each buffer,
wouldn't it be better to set all the page pointers to NULL in
GenericXLogStart and allocate memory only once a buffer is registered
in GenericXLogRegisterBuffer when finding a free slot? This patch is
wasting many cycles.
-- 
Michael



pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: pg_upgrade fails in 9.6 beta1
Next
From: Teodor Sigaev
Date:
Subject: Re: [COMMITTERS] pgsql: Correctly align page's images in generic wal API