RFR: 8155524: HotCardCache shouldn't be part of ConcurrentG1Refine

[Thomas Schatzl]

Hi Kim,

On Fri, 2016-04-29 at 02:29 -0400, Kim Barrett wrote:
> Please review this cleanup that moves ownership of the hot card cache
> from the ConcurrentG1Refine class (which doesn't directly use it at
> all) to the G1CollectedHeap, where it is also made accessible to the
> G1RemSet, as both classes make direct use of it.
> 
> This allows the G1RemSet to largely disconnect from the refinement
> objects.  (There's still one lingering reference, to
> ConcurrentG1Refine::thread_num(), which requires additional work to
> clean up.  The G1RemSetSummary also conditionally refers to
> refinement
> objects for logging; see JDK-815525.)
> 
> As a result of this change, ConcurrentG1Refine no longer needs the
> heap object at construction time.
> 
> Also cleaned up includes and forward declarations in
> concurrentG1Refine.hpp.
> 
> CR:
> https://bugs.openjdk.java.net/browse/JDK-8155524
> 
> Webrev:
> http://cr.openjdk.java.net/~kbarrett/8155524/webrev.00/
> 
> Testing:
> Ad-hoc nightly test run using G1.
> 

  looks good. Thanks a lot for this change, I had it on my plans for
some time now...

Thomas