Hi, Some minor follow up things worth noting:
(1) I discussed with Andres off-list today if there would be a problem with TSC instructions being disabled for user mode, since there is technically a way to restrict them to kernel mode only. In my testing this won't be a practical problem on Linux, since vDSO already directly calls RDTSCP, and so if you disallow TSC through "prctl(PR_SET_TSC, PR_TSC_SIGSEGV)" pg_test_timing does crash - but it crashes in the clock_gettime call, i.e. would do so even without any direct TSC use in the picture. Presumably you could change the Linux clocksource to avoid that. I also did some testing with containers that use seccomp profiles (e.g. Docker), and there are no issues that I can found beyond what would have already been a problem before. If this was a practical problem, we could consider moving the "pg_initialize_timing_tsc" call in pg_set_timing_clock_source to be inside the switch cases (like Andres suggested upthread, but did not change in the commit to be clear), that way a user could start Postgres with timing_clock_source=system and not have any RDTSC(P) instructions run at all. (2) The pg_test_timing documentation has an existing link to this wiki page: https://wiki.postgresql.org/wiki/Pg_test_timing -- it seems that page does talk about certain things that are not covered in the main documentation, but I'm not really sure why that can't be in the main docs. Do we want to keep maintaining that wiki page? (maybe we should just remove that link, and move anything we consider critical to the main docs?) (3) I will move the patch to add ARM support to a new thread sometime before the PG20 branch opens, so we can discuss that further. With that, I've marked the commitfest entry as committed. Thanks, Lukas -- Lukas Fittl
