RFR: 8257230: assert(InitialHeapSize >= MinHeapSize) failed: Ergonomics decided on incompatible initial and minimum heap sizes [v3]

[Jie Fu]

On Thu, 3 Dec 2020 13:47:23 GMT, Stefan Johansson <sjohanss at openjdk.org> wrote:

>> Jie Fu has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains five additional commits since the last revision:
>> 
>>  - Merge branch 'master' into JDK-8257230
>>  - Refinement & jtreg test
>>  - Merge branch 'master' into JDK-8257230
>>  - Merge branch 'master' into JDK-8257230
>>  - 8257230: assert(InitialHeapSize >= MinHeapSize) failed: Ergonomics decided on incompatible initial and minimum heap sizes
>
> Took a closer look at the test now, some comment below.

Hi @kstefanj ,

Thanks for your review.

The fix has been updated according to your comments.

I don't check the exit value because I found different platforms may return different values (Zero on MacOS, but non-zero on Linux).
Maybe, It should return 0 on Linux too, which I think is more reasonable.
And I'd like to file another bug to fix it in the future.

Thanks.
Best regards,
Jie

-------------

PR: https://git.openjdk.java.net/jdk/pull/1492