From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Magnus Hagander <magnus(at)hagander(dot)net> |
Cc: | Andres Freund <andres(at)2ndquadrant(dot)com>, pgsql-hackers(at)postgresql(dot)org, Robert Haas <robertmhaas(at)gmail(dot)com>, Thom Brown <thom(at)linux(dot)com>, Josh Berkus <josh(at)agliodbs(dot)com> |
Subject: | Re: Posix Shared Mem patch |
Date: | 2012-06-28 17:43:16 |
Message-ID: | 24864.1340905396@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Magnus Hagander <magnus(at)hagander(dot)net> writes:
> On Thu, Jun 28, 2012 at 7:27 PM, Andres Freund <andres(at)2ndquadrant(dot)com> wrote:
>> On Thursday, June 28, 2012 07:19:46 PM Magnus Hagander wrote:
>>> What happens if you mlock() it into memory - does that fail quickly?
>>> Is that not something we might want to do *anyway*?
>> You normally can only mlock() mminor amounts of memory without changing
>> settings. Requiring to change that setting (aside that mlocking would be a bad
>> idea imo) would run contrary to the point of the patch, wouldn't it? ;)
> It would. I wasn't aware of that limitation :)
The OSX man page says that mlock should give EAGAIN for a permissions
failure (ie, exceeding the rlimit) but
[ENOMEM] Some portion of the indicated address range is not
allocated. There was an error faulting/mapping a
page.
It might be helpful to try mlock (if available, which it isn't
everywhere) and complain about ENOMEM but not other errors. If course,
if the kernel checks rlimit first, we won't learn anything ...
I think it *would* be a good idea to mlock if we could. Setting shmem
large enough that it swaps has always been horrible for performance,
and in sysv-land there's no way to prevent that. But we can't error
out on permissions failure.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2012-06-28 17:46:30 | Re: Posix Shared Mem patch |
Previous Message | Heikki Linnakangas | 2012-06-28 17:39:18 | Re: Patch-2 (2-move-continuation-record-to-page-header.patch) WAL Format Changes |