RFR(M): 8220311: Implementation: NUMA-Aware Memory Allocation for G1, Survivor (2/3)
Kim Barrett
kim.barrett at oracle.com
Mon Oct 21 23:24:33 UTC 2019
> On Oct 13, 2019, at 2:00 AM, sangheon.kim at oracle.com wrote:
>
> Hi all,
>
> Previous patch conflicts, so I'm posting rebased one.
>
> Webrev:
> http://cr.openjdk.java.net/~sangheki/8220311/webrev.2
> Testing: hs-tier 1 ~ 5, with/without UseNUMA
>
> Thanks,
> Sangheon
------------------------------------------------------------------------------
src/hotspot/share/gc/g1/g1ParScanThreadState.hpp
Removed:
190 // ... State is the original (source) cset state for the object
191 // that is allocated for. ...
That simple removal doesn't seem right. Now "state" in the next
sentence has no explanation. Maybe some better rewrite?
------------------------------------------------------------------------------
Looks good, other than that one comment issue.
More information about the hotspot-gc-dev
mailing list