Re: WIP: bufmgr rewrite per recent discussions

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: "Mark Cave-Ayland" <m(dot)cave-ayland(at)webbased(dot)co(dot)uk>
Cc: pgsql-patches(at)postgresql(dot)org
Subject: Re: WIP: bufmgr rewrite per recent discussions
Date: 2005-02-17 15:46:24
Message-ID: 7346.1108655184@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

"Mark Cave-Ayland" <m(dot)cave-ayland(at)webbased(dot)co(dot)uk> writes:
> As I see it, there is not much noticeable performance gain (and maybe even a
> small loss)with the padding included.

Looks that way. Of course we should never trust a single test case very
far, but this suggests that there's not a whole lot of gold to be mined
by padding out the buffer headers.

>> 3. Pad the LWLock struct (in
>> src/backend/storage/lmgr/lwlock.c) to some power of 2 up to
>> 128 bytes --- same issue of space wasted versus cross-lock contention.

> Having seen the results above, is it still worth looking at this?

Yeah, probably, because there are other possible contention sources
besides buffers that might be alleviated by padding LWLocks. For
instance the buffer manager global locks and the LockMgrLock are
probably all in the same cache line at the moment.

Thanks for running these tests.

regards, tom lane

In response to

Responses

Browse pgsql-patches by date

  From Date Subject
Next Message Zhenbang Wei 2005-02-18 06:14:31 Update Traditional Chinese translations for 8.0 branch
Previous Message Mark Cave-Ayland 2005-02-17 15:37:10 Re: WIP: bufmgr rewrite per recent discussions