RFR 7014526: "java -version" crash on sparc with some values of HeapBaseMinAddress

[Yumin Qi]

Hi,

   I joined late for this discussion. We parse 'G', 'M', 'K' and if 
there is no appendix to numbers, it is Byte,  is this right? If so, 
using a number as default (G) will cause more confuse I think.

Thanks
Yumin

On 2/18/2014 9:41 AM, Coleen Phillimore wrote:
>
> On 2/18/14 11:13 AM, Thomas Schatzl wrote:
>> Hi Coleen,
>>
>> On Tue, 2014-02-18 at 09:54 -0500, Coleen Phillimore wrote:
>>> I fixed the indentation and I changed it to %d because it's %dM.  The
>>> %x was temporary for my debugging.
>>    iirc there is a UINTX_FORMAT constant specifically to be used as the
>> format string for uintx values.
>>
>> Is it possible to also write the value selected by the user to make
>> the problem more clear? I.e. "HeapBaseMinAddress must be at least 
>> ..., but
>> is ...".
>>
>> Also I am not sure if dividing by M does not cut significant digits too
>> much, but I guess people are mostly going to use MB here anyway.
>
> I divided by M to avoid this sort of output, which looks like a 
> meaningless bunch of digits to me.
>
> % java -XX:+PrintCommandLineFlags -version
> -XX:InitialHeapSize=1035993088 -XX:MaxHeapSize=16575889408 
> -XX:+PrintCommandLineFlags -XX:+UseCompressedClassPointers 
> -XX:+UseCompressedOops -XX:+UseParallelGC
>
> How about:
>
> java -d64 -XX:HeapBaseMinAddress=128 -XX:+PrintMiscellaneous 
> -XX:+Verbose -version
> ...
> HeapBaseMinAddress must be at least 6442450944 (6G) which is greater 
> than value given 128
> ...
>
> All the defaults are in G.  If that changes we could make the message 
> divide by M.
>
> Thanks,
> Coleen
>
>>
>> Thanks,
>>    Thomas
>>
>>
>>
>