From: | "Andrey M(dot) Borodin" <x4mmm(at)yandex-team(dot)ru> |
---|---|
To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | Dilip Kumar <dilipbalaut(at)gmail(dot)com>, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>, tender wang <tndrwang(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: SLRU optimization - configurable buffer pool and partitioning the SLRU lock |
Date: | 2024-01-02 18:10:04 |
Message-ID: | 4BF5A925-0D32-40EB-9ED8-D03909655B1B@yandex-team.ru |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
> On 2 Jan 2024, at 19:23, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
>
>>
>> And it would be even better if page for transaction statuses would be determined by backend id somehow. Or at least cache line. Can we allocate a range (sizeof(cacheline)) of xids\subxids\multixacts\whatever for each backend?
>
> I don't understand how this could work. We need to be able to look up
> transaction status by XID, not backend ID.
When GetNewTransactionId() is called we can reserve 256 xids in backend local memory. This values will be reused by transactions or subtransactions of this backend. Here 256 == sizeof(CacheLine).
This would ensure that different backends touch different cache lines.
But this approach would dramatically increase xid consumption speed on patterns where client reconnects after several transactions. So we can keep unused xids in procarray for future reuse.
I doubt we can find significant performance improvement here, because false cache line sharing cannot be _that_ bad.
Best regards, Andrey Borodin.
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2024-01-02 18:13:16 | Re: verify predefined LWLocks have entries in wait_event_names.txt |
Previous Message | Tom Lane | 2024-01-02 18:06:18 | Re: pg_upgrade failing for 200+ million Large Objects |