RFR: 8183539: Remove G1RemSet::_into_cset_dirty_card_queue_set

Thomas Schatzl thomas.schatzl at oracle.com
Tue Jul 4 15:24:56 UTC 2017


Hi Erik,

On Tue, 2017-07-04 at 13:40 +0200, Erik Helin wrote:
> Hi all,
> 
> here comes a simple patch (just removing code) with a quite
> complicated justification :) So grab a cup of coffee, take out that
> good old pen and paper (it is almost impossible to convince yourself
> that this is correct without drawing) and enjoy the following little
> text:
> 
> 
[... long explanation...]
> Patch: http://cr.openjdk.java.net/~ehelin/8183539/00/
> Issue: https://bugs.openjdk.java.net/browse/JDK-8183539
> Testing: make test TEST=hotspot_gc on Linux x86-64 fastdebug

  while I think the explanation is good, and actually we discussed this
together, some more testing would be nice ;) Something like gcbasher
with G1EvacuationFailureALot.

Minor nit: g1RemSet.cpp:710 the "return" statement is superfluous.
(although I have already a change in mind that re-adds a return value
;))

Thanks,
  Thomas




More information about the hotspot-gc-dev mailing list