RFR(S): 8134889: Kitchensink stress test crashes with out of memory error

[Derek White]

On 4/27/16 8:27 AM, Per Liden wrote:
> On 2016-04-27 14:25, Per Liden wrote:
>> Hi Derek,
>>
>> On 2016-04-20 23:48, Derek White wrote:
>>> Still looking for reviews for a small fix for a P2 bug...
>>>
>>> Thanks!
>>>
>>> On 4/7/16 6:07 PM, Derek White wrote:
>>>> Summary:
>>>> The remaining part of this bug deals with running out of address space
>>>> on 32-bit JVMs running on Win64. This fix is simply to have a hard
>>>> limit on GC threads (still larger than necessary) on 32-bit JVMs. The
>>>> new limit is 16 (2 * switch_pt).
>>>>
>>>> Note that this overlaps somewhat with several "# of GC thread" bugs
>>>> including:
>>>>  - 8131932 <https://bugs.openjdk.java.net/browse/JDK-8131932> :
>>>> Ergonomics oversize number of GC threads for small JVMs on large 
>>>> systems
>>>>  - 8153225 <https://bugs.openjdk.java.net/browse/JDK-8153225>: G1
>>>> creates too many concurrent refinement threads by default
>>>>  - 7189147 <https://bugs.openjdk.java.net/browse/JDK-7189147>: GC
>>>> threads should be created as needed not all at once
>>>>
>>>> Doing the correct thing in general will require a lot more performance
>>>> analysis. The fix here is simple, only affects 32-bit JVMs running on
>>>> >= 24 core machines, and will reduce OOM crashes in nightly testing.
>>
>> Looks good.
>
> ... and I don't need to see a new webrev if you just remove the cast.

Thanks Per!