From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | Andres Freund <andres(at)2ndquadrant(dot)com>, Sean Chittenden <sean(at)chittenden(dot)org>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [PATCH] PostgreSQL 9.4 mmap(2) performance regression on FreeBSD... |
Date: | 2014-10-12 00:33:57 |
Message-ID: | 20141012003357.GS21267@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Tue, Aug 12, 2014 at 07:08:06PM -0400, Robert Haas wrote:
> On Tue, Aug 12, 2014 at 12:59 PM, Andres Freund <andres(at)2ndquadrant(dot)com> wrote:
> > On 2014-08-12 09:42:30 -0700, Sean Chittenden wrote:
> >> One of the patches that I've been sitting on and am derelict in punting
> >> upstream is the attached mmap(2) flags patch for the BSDs. Is there any
> >> chance this can be squeezed in to the PostreSQL 9.4 release?
> >>
> >> The patch is trivial in size and is used to add one flag to mmap(2) calls in
> >> dsm_impl.c. Alan Cox (FreeBSD alc, not Linux) and I went back and forth
> >> regarding PostgreSQL's use of mmap(2) and determined that the following is
> >> correct and will prevent a likely performance regression in PostgreSQL 9.4.
> >> In PostgreSQL 9.3, all mmap(2) calls were called with the flags MAP_ANON |
> >> MAP_SHARED, whereas in PostgreSQL 9.4 this is not the case.
> >
> > The performancewise important call to mmap will still use that set of
> > flags, no? That's the one backing shared_buffers.
> >
> > The mmap backend for *dynamic* shared memory (aka dsm) is *NOT* supposed
> > to be used on common platforms. Both posix and sysv shared memory will
> > be used before falling back to the mmap() backend.
>
> Hmm, yeah. This might still be a good thing to do (because what do we
> lose?) but it shouldn't really be an issue in practice.
Is there a reason this was not applied?
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ Everyone has their own god. +
From | Date | Subject | |
---|---|---|---|
Next Message | Alvaro Herrera | 2014-10-12 01:15:22 | Re: pg_dump refactor patch to remove global variables |
Previous Message | Bruce Momjian | 2014-10-12 00:32:09 | Re: [REVIEW] pg_last_xact_insert_timestamp |