Re: Reduce timing overhead of EXPLAIN ANALYZE using rdtsc?
Hannu Krosing <hannuk@google.com>
On Fri, Feb 13, 2026 at 5:11 AM Lukas Fittl <lukas@fittl.com> wrote: > > On Thu, Feb 12, 2026 at 4:41 PM Andres Freund <andres@anarazel.de> wrote: > > I wonder if pg_test_timing should have a small loop with a fixed count to > > determine the timing without all the overhead the existing loop has... > > I agree that using a fixed count in pg_test_timing would be helpful to > measure just the timing gathering itself, vs the translation into > nanoseconds. I haven't looked at the code here yet, but when using plain rdtsc on modern CPUs one sees much more overhead from just the fact that the code is there than from calling the rdtsc instruction, and the overhead can vary by orders of magnitude based on how complex the work is that is timed. I discovered this when I timed the (then-)new dead tid lookups in the Vacuum in Pg 17 and saw significantly larger overhead per lookup when the lookups themselves were slower, i.e. a case where the lookups were done in random order (inded was on created on a column filled with random()) So while just a tight loop of N million rtdsc calls will give you the lower limit, it is likely not very representative of actual overhead.
Commits
GET /api/v1/messages/:b64id/commits
the thread's linked commits as JSON, with link sources.
API reference →
-
pg_test_timing: Show additional TSC clock source debug info
- 5ba34f6dc838 19 (unreleased) landed
-
instrumentation: Avoid CPUID 0x15/0x16 for Hypervisor TSC frequency
- 7fc36c5db550 19 (unreleased) landed
-
pg_test_timing: Also test RDTSC[P] timing, report time source, TSC frequency
- 16fca4825483 19 (unreleased) landed
-
Allow retrieving x86 TSC frequency/flags from CPUID
- bcb2cf41f964 19 (unreleased) landed
-
instrumentation: Standardize ticks to nanosecond conversion method
- 0022622c93d9 19 (unreleased) landed
-
instrumentation: Use Time-Stamp Counter on x86-64 to lower overhead
- 294520c44487 19 (unreleased) landed
-
Check for __cpuidex and __get_cpuid_count separately
- effaa464afd3 19 (unreleased) landed
-
pg_test_timing: Reduce per-loop overhead
- 82c0cb4e672d 19 (unreleased) landed
-
Refactor handling of x86 CPUID instructions
- be6a7494d2e3 19 (unreleased) landed
-
instrumentation: Drop INSTR_TIME_SET_CURRENT_LAZY macro
- 9d6294c09ed0 19 (unreleased) landed
-
Rename pg_crc32c_sse42_choose.c for general purpose
- b9278871f991 19 (unreleased) cited
-
Zero initialize uses of instr_time about to trigger compiler warnings
- 25b2aba0c3a5 16.0 landed
-
instr_time: Represent time as an int64 on all platforms
- 03023a2664f8 16.0 landed
-
Add 250c8ee07ed to git-blame-ignore-revs
- ff23b592ad66 16.0 cited