From: | Peter Eisentraut <peter(at)eisentraut(dot)org> |
---|---|
To: | Thomas Munro <thomas(dot)munro(at)gmail(dot)com> |
Cc: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: macOS prefetching support |
Date: | 2024-08-14 14:39:31 |
Message-ID: | 4c060690-882b-4129-954e-1249bdceb5e3@eisentraut.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 14.08.24 14:36, Thomas Munro wrote:
> On Wed, Aug 14, 2024 at 7:04 PM Peter Eisentraut <peter(at)eisentraut(dot)org> wrote:
>> Attached is a patch to implement this. It seems to work, but of course
>> it's kind of hard to tell whether it actually does anything useful.
>
> Header order problem: pg_config_os.h defines __darwin__, but
> pg_config_manual.h is included first, and tests __darwin__. I hacked
> my way around that, and then made a table of 40,000,000 integers in a
> 2GB buffer pool. I used "select count(pg_buffercache_evict(buffered))
> from pg_buffer_cache", and "sudo purge", to clear the two layers of
> cache for each test, and then measured:
>
> maintenance_io_concurrency=0, ANALYZE: 2311ms
> maintenance_io_concurrency=10, ANALYZE: 652ms
> maintenance_io_concurrency=25, ANALYZE: 389ms
>
> It works!
Cool! I'll work on a more polished patch.
From | Date | Subject | |
---|---|---|---|
Next Message | David Fetter | 2024-08-14 14:43:51 | Re: [PATCH] Add get_bytes() and set_bytes() functions |
Previous Message | SAMEER KUMAR | 2024-08-14 14:38:49 | Re: Adding clarification to description of IPC wait events XactGroupUpdate and ProcArrayGroupUpdate |