Salikh Zakirov wrote:
> Alex Blewitt wrote:
>> But the point is, this is a hardcoded default maximum of 256m for no
>> particularly good reason. Why should it not be infinity?
>
> The following patch does what you are asking for (untested).
Sorry, the first version would lead to integer overflow and the to assertion
failure. The updated patch should work. It runs Hello on Windows/ia32 with a
warning
about reducing maximum heap size to the virtual address range it could allocate.
$ ./java Hello
WARNING: final heap size is too large, reduced to 900 Mb
Hello
> Care to try?
--- a/vm/gc/src/gc_for_vm.cpp
+++ b/vm/gc/src/gc_for_vm.cpp
@@ -210,8 +210,8 @@ static void parse_configuration_properti
// set the initial and maximum heap size based on provided options and
defaults
const int Mb = 1048576;
if (0 == final_heap_size_bytes && 0 == initial_heap_size_bytes) {
- // defaults -Xms64m -Xmx256m
- final_heap_size_bytes = 256*Mb;
+ // defaults -Xms64m -Xmx\infty
+ final_heap_size_bytes = (POINTER_SIZE_INT)-Mb;
initial_heap_size_bytes = 64*Mb;
} else if (0 == initial_heap_size_bytes) {
// -Xms64m or same as mx, whichever is less
---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
