RFR(S): 8131054: aix: fix two minor issues: large page size and hs_err printing.

Lindenmaier, Goetz goetz.lindenmaier at sap.com
Thu Jul 16 13:09:38 UTC 2015


Hi Coleen,

is it ok if I push this now as is?  I could push it to -rt as well:
http://cr.openjdk.java.net/~goetz/webrevs/8131054-aixMinor/webrev.01-rt/

Best regards,
  Goetz.

-----Original Message-----
From: Coleen Phillimore [mailto:coleen.phillimore at oracle.com] 
Sent: Montag, 13. Juli 2015 19:07
To: Lindenmaier, Goetz; 'hotspot-dev at openjdk.java.net'
Subject: Re: RFR(S): 8131054: aix: fix two minor issues: large page size and hs_err printing.


Yes, you can push this yourself.   My comment was from my last fix for 
too long cpuinfo, I noticed this duplication and would have fixed it for 
you.   Does AIX have a /proc/cpuinfo file?

Coleen

On 7/13/15 12:26 PM, Lindenmaier, Goetz wrote:
> Hi Coleen,
>
> thanks for the review and thanks for the offer of sponsoring,
> but I may push this one myself, as it's aix-only.
>
> Best regards,
>    Goetz.
>
> -----Original Message-----
> From: hotspot-dev [mailto:hotspot-dev-bounces at openjdk.java.net] On Behalf Of Coleen Phillimore
> Sent: Monday, July 13, 2015 5:54 PM
> To: hotspot-dev at openjdk.java.net
> Subject: Re: RFR(S): 8131054: aix: fix two minor issues: large page size and hs_err printing.
>
>
> This looks good.  I was wondering if you wanted me to take this out for you.
>
> Coleen
>
> On 7/13/15 6:18 AM, Lindenmaier, Goetz wrote:
>> Hi,
>>
>> this small change fixes two issues on aix:
>> - _large_page_size was set to -1 which made TestLargePageUseForAuxMemory fail.
>> - CPU info was printed twice to hs_err file.
>>
>> Please review this change.
>> http://cr.openjdk.java.net/~goetz/webrevs/8131054-aixMinor/webrev/
>>
>> Best regards,
>>     Goetz.
>>
>>



More information about the hotspot-dev mailing list