Need Reviewer: change MAX_VM_MEMORY to 512

[Kelly O'Hair]

I think we have a problem with this, I've looked at this before, but something is wrong here.
The end result could be a negative number if the max memory was say, 256Mb RAM, and
0 if the machine has 512Mb of RAM.

I'm thinking that this:
 443     MAX_VM_MEMORY := $(shell \
 444       if [ $(MB_OF_MEMORY) -le 1024 ] ; then \
 445         expr $(MB_OF_MEMORY) '-' 512 2> $(DEV_NULL) ; \
 446       else \
 447         echo "512"; \
 448       fi)
Should just be:
 443     MAX_VM_MEMORY := 512

To avoid a negative or 0 result. And we should delete all the comments about subtracting.
Any machine doing a build with less than 512Mb is very very questionable.
-kto

On Apr 12, 2011, at 4:29 PM, SUCHEN.CHIEN at ORACLE.COM wrote:

> Need Reviewer: change MAX_VM_MEMORY to 512
> Windows systems may not be able to handle 896 max memory on every java app started up. Depends on what the system is running, even on a 2Gb system.
> 
> Recommend lowering the 896 to 640 or maybe even 512 if possible.
> 
> 6903609: Max memory of 896 may be too large for typical windows developer environment
> http://cr.openjdk.java.net/~schien/CR6903609/webrev/
> 
> 
> Su-Chen

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/build-dev/attachments/20110412/27ef122a/attachment.htm>