Re: Estimating HugePages Requirements?
Michael Paquier <michael@paquier.xyz>
From: Michael Paquier <michael@paquier.xyz>
To: Robert Haas <robertmhaas@gmail.com>
Cc: "Bossart, Nathan" <bossartn@amazon.com>, Fujii Masao <masao.fujii@oss.nttdata.com>, Justin Pryzby <pryzby@telsasoft.com>, Andres Freund <andres@anarazel.de>, Magnus Hagander <magnus@hagander.net>, Mark Dilger <mark.dilger@enterprisedb.com>, Don Seiler <don@seiler.us>, "pgsql-hackers@postgresql.org" <pgsql-hackers@postgresql.org>
Date: 2021-09-14T00:08:47Z
Lists: pgsql-hackers
On Mon, Sep 13, 2021 at 04:20:00PM -0400, Robert Haas wrote: > On Mon, Sep 13, 2021 at 2:49 PM Bossart, Nathan <bossartn@amazon.com> wrote: >> Yeah, I agree. What about >> huge_pages_needed_for_shared_memory_size or >> huge_pages_needed_for_main_shared_memory? I'm still not stoked about >> using "required" or "needed" in the name, as it sounds like huge pages >> must be allocated for the server to run, which is only true if >> huge_pages=on. I haven't thought of a better word to use, though. > > I prefer the first of those to the second. I don't find it > particularly better or worse than my previous suggestion of > shared_memory_size_in_huge_pages. I am not particularly fond of the use "needed" in this context, so I'd be fine with your suggestion of "shared_memory_size_in_huge_pages. Some other ideas I could think of: - shared_memory_size_as_huge_pages - huge_pages_for_shared_memory_size Having shared_memory_size in the GUC name is kind of appealing though in terms of grepping, and one gets the relationship with shared_memory_size immediately. If the consensus is huge_pages_needed_for_shared_memory_size, I won't fight it, but IMO that's too long. -- Michael
Commits
-
Silence extra logging when using "postgres -C" on runtime-computed GUCs
- 8bbf8461a3a2 15.0 landed
-
doc: Improve postgres command for shared_memory_size_in_huge_pages
- bbd4951b73ec 15.0 landed
-
Introduce GUC shared_memory_size_in_huge_pages
- 43c1c4f65eab 15.0 landed
-
Support "postgres -C" with runtime-computed GUCs
- 0c39c292077e 15.0 landed
-
Make shared_memory_size a preset option
- 3b231596ccfc 15.0 landed
-
Introduce GUC shared_memory_size
- bd1788051b02 15.0 landed
-
Move the shared memory size calculation to its own function
- 0bd305ee1d42 15.0 landed
-
Add new GUC, max_worker_processes, limiting number of bgworkers.
- 6bc8ef0b7f1f 9.4.0 cited