RFR: 8049536: os::commit_memory on Solaris uses aligment_hint as page size
Erik Helin
erik.helin at oracle.com
Mon Sep 15 08:14:27 UTC 2014
Thanks Thomas!
Erik
On 2014-09-15 10:12, Thomas Schatzl wrote:
> Hi again,
>
> On Mon, 2014-09-15 at 10:06 +0200, Thomas Schatzl wrote:
>> Hi,
>>
>> On Fri, 2014-09-12 at 15:08 +0200, Erik Helin wrote:
>>> All,
>>>
>>> quick feedback from StefanK:
>>> - forgot to cast vm_page_size() to size_t in one place
>>> - print caddr_t with PRT_FORMAT in error message
>>>
>>> Webrevs:
>>> - full: http://cr.openjdk.java.net/~ehelin/8049536/webrev.03/
>>> - incremental: http://cr.openjdk.java.net/~ehelin/8049536/webrev.02-03/
>>
>> os_solaris.cpp, line 3093:
>>
>> assert(is_size_aligned((size_t) start, align), ...
>>
>> I think it is invalid to cast an address to a size_t. At least the C99
>> standard does not ensure that this is a valid cast.
>> (In practice it is the case for all by Oracle supported platforms I
>> think)
>>
>> The change should use is_ptr_aligned() here and in line 3096 I think,
>> and fix the err_msg() format parameters.
>
> It's good in the HTML webrev. The udiff still contained the 02 code it
> seems.
>
> So, looks good.
>
> Sorry,
> Thomas
>
>
More information about the hotspot-gc-dev
mailing list