Re: Speed up Clog Access by increasing CLOG buffers
Tomas Vondra <tomas.vondra@2ndquadrant.com>
On 09/07/2016 01:13 PM, Amit Kapila wrote: > On Wed, Sep 7, 2016 at 1:08 AM, Tomas Vondra > <tomas.vondra@2ndquadrant.com> wrote: >> On 09/06/2016 04:49 AM, Amit Kapila wrote: >>> On Mon, Sep 5, 2016 at 11:34 PM, Tomas Vondra >>> <tomas.vondra@2ndquadrant.com> wrote: >>>> >>>> >>>> On 09/05/2016 06:03 AM, Amit Kapila wrote: >>>>> So, in short we have to compare three >>>>> approaches here. >>>>> >>>>> 1) Group mode to reduce CLOGControlLock contention >>>>> 2) Use granular locking model >>>>> 3) Use atomic operations >>>>> >>>>> For approach-1, you can use patch [1]. For approach-2, you can use >>>>> 0001-Improve-64bit-atomics-support patch[2] and the patch attached >>>>> with this mail. For approach-3, you can use >>>>> 0001-Improve-64bit-atomics-support patch[2] and the patch attached >>>>> with this mail by commenting USE_CONTENT_LOCK. If the third doesn't >>>>> work for you then for now we can compare approach-1 and approach-2. >>>>> >>>> >>>> OK, I can compile all three cases - but onl with gcc 4.7 or newer. Sadly >>>> the 4-socket 64-core machine runs Debian Jessie with just gcc 4.6 and my >>>> attempts to update to a newer version were unsuccessful so far. >>>> >>> >>> So which all patches your are able to compile on 4-socket m/c? I >>> think it is better to measure the performance on bigger machine. >> >> Oh, sorry - I forgot to mention that only the last test (with >> USE_CONTENT_LOCK commented out) fails to compile, because the functions >> for atomics were added in gcc-4.7. >> > > No issues, in that case we can leave the last test for now and do it later. > FWIW I've managed to compile a new GCC on the system (all I had to do was to actually read the damn manual), so I'm ready to do the test once I get a bit of time. regards -- Tomas Vondra http://www.2ndQuadrant.com PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
Commits
-
Use group updates when setting transaction status in clog.
- baaf272ac908 11.0 landed
- ccce90b39867 10.0 landed
-
Improve 64bit atomics support.
- e8fdbd58fe56 10.0 landed
-
Add ProcArrayGroupUpdate wait event.
- d4116a771925 10.0 landed
-
Make the different Unix-y semaphore implementations ABI-compatible.
- be7b2848c6d8 10.0 cited
-
Fix broken ALTER INDEX documentation
- 2143f5e12790 9.6.0 cited
-
Code and docs review for commit 3187d6de0e5a9e805b27c48437897e8c39071d45.
- d12e5bb79bb5 9.6.0 cited
-
Partition the freelist for shared dynahash tables.
- 44ca4022f3f9 9.6.0 cited
-
Correct StartupSUBTRANS for page wraparound
- 481725c0ba73 9.6.0 cited
-
Make idle backends exit if the postmaster dies.
- ac1d7945f866 9.6.0 cited
-
contrib/sslinfo: add ssl_extension_info SRF
- 49124613f134 9.6.0 cited
-
Reduce ProcArrayLock contention by removing backends in batches.
- 0e141c0fbb21 9.6.0 cited
-
Fix `make installcheck` for serializable transactions.
- 253de7e1eb9a 9.6.0 cited
-
Lockless StrategyGetBuffer clock sweep hot path.
- d72731a70450 9.5.0 cited
-
Reduce sinval synchronization overhead.
- b4fbe392f8ff 9.2.0 cited