Re: memory fields from getrusage()

From: Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com>
To: Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>
Cc: Justin Pryzby <pryzby(at)telsasoft(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: memory fields from getrusage()
Date: 2017-09-02 02:00:44
Message-ID: CAEepm=1f9k6UYP3fPfv1YQr4q7Y5f7np1m=tykFGuSknSLaizg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Sat, Sep 2, 2017 at 7:46 AM, Peter Eisentraut
<peter(dot)eisentraut(at)2ndquadrant(dot)com> wrote:
> On 6/15/17 10:58, Justin Pryzby wrote:
>> On Thu, Jun 15, 2017 at 10:29:21AM -0400, Robert Haas wrote:
>>> On Wed, Jun 14, 2017 at 6:28 PM, Justin Pryzby <pryzby(at)telsasoft(dot)com> wrote:
>>>> On Tue, Jun 13, 2017 at 12:16:00PM -0400, Robert Haas wrote:
>>>>> It might be worth adding platform-specific code for common platforms.
>>>>
>>>> All I care (which linux happily/happens to support) is maxrss; I was probably
>>>> originally interested in this while digging into an issue with hash agg.
>>>
>>> I don't think it needs to go in a separate file. I'd just patch ShowUsage().
>
> I have committed a patch that shows maxrss, with /1024 adjustment for
> macOS. That should cover all platforms that I could find.(*)

Apparently ru_maxrss is in *pages* on Solaris-derived systems:

https://illumos.org/man/3c/getrusage

AIX seems to be like Linux and FreeBSD (kilobytes):

https://www.ibm.com/support/knowledgecenter/en/ssw_aix_61/com.ibm.aix.basetrf1/getrusage_64.htm

--
Thomas Munro
http://www.enterprisedb.com

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Amit Kapila 2017-09-02 02:11:59 Re: Speed up Clog Access by increasing CLOG buffers
Previous Message Robert Haas 2017-09-02 01:33:17 Re: Adding support for Default partition in partitioning