RFR: JDK-8257588: Make os::_page_sizes a set [v2]

Thomas Stuefe stuefe at openjdk.java.net
Thu Dec 3 13:01:55 UTC 2020


On Thu, 3 Dec 2020 12:26:04 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

>> Not sure what you mean.
>> 
>> PagesizeSet::smallest() returns the smallest page in the page size set, which may be any size, or 0 if empty. 
>> 
>> os::_page_sizes should always contain os::vm_page_size(), but that too is not the same as os::min_page_size(). os::min_page_size() is just a constant, equal or smaller than the smallest page size across all platforms.
>> 
>> The naming is confusing, and probably it could be just a static const instead of a function. But I don't want to touch that with this change. But there are probably a number of misuses of os::min_page_size() where the author actually meant os::vm_page_size().
>
> Okay, apologies for falling into the mentioned trap.

No problem. We should probably improve this.

-------------

PR: https://git.openjdk.java.net/jdk/pull/1522


More information about the hotspot-dev mailing list