Re: Possible performance regression in version 10.1 with pgbench read-write tests.

From: Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Andres Freund <andres(at)anarazel(dot)de>, Mithun Cy <mithun(dot)cy(at)enterprisedb(dot)com>, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Possible performance regression in version 10.1 with pgbench read-write tests.
Date: 2018-07-23 10:06:58
Message-ID: CAEepm=2LHBr1mzei57QS3YjvKEaRO9P9Ms6E8wvpbq3fjW1pwQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, Jul 23, 2018 at 3:40 PM, Thomas Munro
<thomas(dot)munro(at)enterprisedb(dot)com> wrote:
> On Sun, Jul 22, 2018 at 8:19 AM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>> 8 clients 72 clients
>>
>> unmodified HEAD 16112 16284
>> with padding patch 16096 16283
>> with SysV semas 15926 16064
>> with padding+SysV 15949 16085
>>
>> This is on RHEL6 (kernel 2.6.32-754.2.1.el6.x86_64), hardware is dual
>> 4-core Intel E5-2609 (Sandy Bridge era). This hardware does show NUMA
>> effects, although no doubt less strongly than Mithun's machine.
>>
>> I would like to see some other results with a newer kernel. I tried to
>> repeat this test on a laptop running Fedora 28, but soon concluded that
>> anything beyond very short runs was mainly going to tell me about thermal
>> throttling :-(. I could possibly get repeatable numbers from, say,
>> 1-minute SELECT-only runs, but that would be a different test scenario,
>> likely one with a lot less lock contention.
>
> I did some testing on 2-node, 4-node and 8-node systems running Linux
> 3.10.something (slightly newer but still ancient). Only the 8-node
> box (= same one Mithun used) shows the large effect (the 2-node box
> may be a tiny bit faster patched but I'm calling that noise for now...
> it's not slower, anyway).

Here's an attempt to use existing style better: a union, like
LWLockPadded and WALInsertLockPadded. I think we should back-patch to
10. Thoughts?

--
Thomas Munro
http://www.enterprisedb.com

Attachment Content-Type Size
0001-Pad-semaphores-to-avoid-false-sharing.patch application/octet-stream 1.6 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Ibrar Ahmed 2018-07-23 10:19:24 Re: Log query parameters for terminated execute
Previous Message Sergei Kornilov 2018-07-23 10:05:51 Re: Log query parameters for terminated execute