On Fri, Feb 5, 2016 at 3:17 AM, Robert Haas <robertmh...@gmail.com> wrote: > > > I think we ought to move the buffer mapping, lock manager, and > predicate lock manager locks into their own tranches also, perhaps > using this new named-tranche facility. >
Makes sense and attached patch implements it using new named tranches facility. One thing to note is that to make it work on EXEC_BACKEND builds, I have passed the individual LWLock array via save-restore backendparams mechanism. Now instead we could have initialised the LWLock arrays in each backend at start-up and I have tried to analyse that way as well, but we need to use NamedLWLockTrancheArray instead of NamedLWLockTrancheRequestArray in GetNamedLWLockTranche() and not only that, we also need to store number of locks information in NamedLWLockTrancheArray as well. So it seems it is better to use save-restore backendparams mechanism for passing LWLock arrays. > In addition, I think we should > get rid of NumLWLocks(). It's no longer calculating anything; it just > returns NUM_FIXED_LWLOCKS, and with the changes described in the first > sentence of this paragraph, it will simply return > NUM_INDIVIDUAL_LWLOCKS. We don't need a function just to do that; the > assignment it does can be moved to the relevant caller. > Agreed and we don't even need an array of LWLockCounter's, just one counter is sufficient for tranches. Apart from this function LWLockAssign() seems redundant to me after new implementation of named tranches, so I have removed that as well. With Regards, Amit Kapila. EnterpriseDB: http://www.enterprisedb.com
fixed_locks_tranche_v1.patch
Description: Binary data
-- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
