Hi,
On 2/7/23 19:12, Andres Freund wrote:
> This fails to build on several platforms:
>
> https://cirrus-ci.com/github/postgresql-cfbot/postgresql/commitfest%2F42%2F3751
I think I fixed the compilation errors. It was due to a few variables
being declared under
#if defined(__x86_64__) && defined(__linux__)
while being used also under non x86 Linux.
I also removed again the code to obtain the TSC frequency under
hypervisors because the TSC is usually emulated and therefore no faster
than clock_gettime() anyways. So we now simply fallback to
clock_gettime() on hypervisors when we cannot obtain the frequency via
leaf 0x16.
Beyond that I reviewed the first two patches a while ago in [1]. I hope
we can progress with them to further reduce the size of this patch set.
[1]
https://www.postgresql.org/message-id/3ac157f7-085d-e071-45fc-b87cd306360c%40gmail.com
--
David Geier
(ServiceNow)