[8u] request for approval: "8170409: CMS: Crash in CardTableModRefBSForCTRS::process_chunk_boundaries"

[Volker Simonis]

Thanks Rob!
I've added 'noreg-hard' to the bug with a corresponding comment.

Regards,
Volker


On Mon, Dec 12, 2016 at 10:58 PM, Rob McKenna <rob.mckenna at oracle.com> wrote:
> Approved. Please add a suitable noreg label to the bug.
>
>     -Rob
>
> On 12/12/16 07:19, Volker Simonis wrote:
>> Hi,
>>
>> could you please approve the backport of the following GC fix to jdk8u-dev:
>>
>> 8170409: CMS: Crash in CardTableModRefBSForCTRS::process_chunk_boundaries
>>
>> This fix is critical for linux/ppc64 where it impacts Hadoop/Terasort [1].
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8170409
>> Webrev: http://cr.openjdk.java.net/~simonis/webrevs/2016/8170409.v3/
>> Review: http://mail.openjdk.java.net/pipermail/hotspot-gc-dev/2016-November/019313.html,
>> http://mail.openjdk.java.net/pipermail/hotspot-gc-dev/2016-December/019332.html
>> URL: http://hg.openjdk.java.net/jdk9/hs/hotspot/rev/fe86ccf9132f
>>
>> The original patch cleanly applies with only two minor, non-functional changes:
>>
>> - we have to revert the file renaming from
>> "share/vm/gc_implementation/parNew/parCardTableModRefBS.cpp" and
>> "share/vm/memory/cardTableModRefBS.hpp" in jdk8 to
>> "share/vm/gc/cms/parCardTableModRefBS.cpp" and
>> "share/vm/gc/shared/cardTableModRefBSForCTRS.hpp" in jdk9.
>>
>> - in jkd9, jdk8's Universe::heap() was moved to
>> GenCollectedHeap::heap(). While the call to this function is not part
>> of the patch, it has to be renamed in the context of the change such
>> that the patch cleanly applies.
>>
>> The updated webrev can be found here:
>>
>> http://cr.openjdk.java.net/~simonis/webrevs/2016/8170409.8u/
>>
>> Thank you and best regards,
>> Volker
>>
>> [1] http://mail.openjdk.java.net/pipermail/hotspot-dev/2016-December/025483.html