From: | Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com> |
---|---|
To: | "Tsunakawa, Takayuki" <tsunakawa(dot)takay(at)jp(dot)fujitsu(dot)com> |
Cc: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, Magnus Hagander <magnus(at)hagander(dot)net>, Robert Haas <robertmhaas(at)gmail(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Supporting huge pages on Windows |
Date: | 2017-01-05 21:42:58 |
Message-ID: | CAEepm=2Bv+098ED-nJd-5=oa5niMnbC_Hr1iuHr1ciDLvoXdTQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Thu, Jan 5, 2017 at 4:12 PM, Tsunakawa, Takayuki
<tsunakawa(dot)takay(at)jp(dot)fujitsu(dot)com> wrote:
> [win_large_pages_v4.patch]
Just a small suggestion about the wording in this patch:
+ This feature uses the large-page support on Windows. To use
the large-page
+ support, you need to assign Lock page in memory user right to
the Windows
+ user account which runs <productname>PostgreSQL</productname>.
In the Microsoft documentation I've seen, the privilege's name is
always written as "Lock Pages in Memory" (note: "Pages" plural, and
with initial capital letters). It's quite hard to parse the sentence
otherwise! How about this?
Huge pages are known as large pages on Windows. To use them,
you need to
assign the user right Lock Pages in Memory to the Windows user account
that runs <productname>PostgreSQL</productname>.
+ ereport(elevel,
+ (errmsg("could not enable Lock pages in memory user right: error
code %lu", GetLastError()),
+ errdetail("Failed system call was OpenProcessToken.")));
Same comment about capitalisation of the privilege name in this and
other error messages.
--
Thomas Munro
http://www.enterprisedb.com
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2017-01-05 21:48:44 | Re: Group clear xid can leak semaphore count |
Previous Message | Tom Lane | 2017-01-05 21:39:34 | Re: Re: [BUGS][PATCH] BUG #14486: Inserting and selecting interval have different constraints |