+ * and the hugepage-related mmap flags to use into *mmap_flags.  If huge pages
+ * is not supported, *hugepagesize and *mmap_flags will be set to 0.

nitpick: *are* not supported, as you say elsewhere.

+                       gettext_noop("Shows the number of huge pages needed for 
the main shared memory area."),

Maybe this was already discussed, but "main" could be misleading.

To me that sounds like there might be huge pages needed for something other
than the "main" area and the reported value might turn out to be inadequate,
(which is exactly the issue these patch are trying to address).

In particular, this sounds like it's just going to report
shared_buffers/huge_page_size (since shared buffers is usually the "main" use
of shared memory) - rather than reporting the size of the entire shared memory
in units of huge_pages.

-- 
Justin


Reply via email to