RFR: 8014510: Fix sjavac on all platforms in jprt
Magnus Ihse Bursie
magnus.ihse.bursie at oracle.com
Fri Aug 29 11:11:21 UTC 2014
On 2014-08-22 16:20, Erik Joelsson wrote:
> Thanks for looking into this. I took the liberty of tuning this some
> more and finally arrived at this set of changes:
>
> http://cr.openjdk.java.net/~erikj/8014510/webrev.root.01/
>
> First of all I wanted to get rid of the big if-else construct, so I
> changed that into a simple MEMORY_SIZE/2 for sjavac server mx flag.
> Then I did some tests on my machine and with the modules build, sjavac
> really isn't that memory intensive anymore. I set a cap at 2GB (1,5
> for 32 bit jvms) and also set a minimum of 512M, which I have verified
> still works. I also removed the verbosegc flag.
>
> When changing these settings, it struck me that the mx in
> BIG_JAVA_FLAGS is a static configuration and should probably also be
> tuned in the same way. I left the old values as the maximum caps and
> let them go as low as 512. This has also been verified to be enough.
>
> Finally, since the memory requirements of the java instances in the
> build have been reduced, I also chose to tweak the memory based
> limiter on the default number of jobs.
Looks good to me.
/Magnus
More information about the build-dev
mailing list