RFR: 8301149: Parallel: Refactor MutableNUMASpace::update_layout [v3]

Leo Korinth lkorinth at openjdk.org
Wed Feb 1 16:57:54 UTC 2023


On Wed, 1 Feb 2023 14:13:18 GMT, Albert Mingkun Yang <ayang at openjdk.org> wrote:

>> Small cleanup to simplify the structure.
>> 
>> Test: hotspot_gc on boxes with two NUMA nodes.
>
> Albert Mingkun Yang has updated the pull request incrementally with one additional commit since the last revision:
> 
>   review

Marked as reviewed by lkorinth (Reviewer).

src/hotspot/share/gc/parallel/mutableNUMASpace.cpp line 281:

> 279: // Update space layout. Perform adaptation.
> 280: void MutableNUMASpace::update() {
> 281:   bool should_initialize = false;

I think `numa_has_static_binding` should be removed, it is always `true`, correct? If so, the loop can be removed and `should_initialize` can be removed as well. I am unsure if you want to fix these things or you want me to create enhancements for this. It might be possible that if we continue this path and take a better look at `UseAdaptiveNUMAChunkSizing`, the whole `update()` method can be removed and maybe the flag, but I know too little about these flags, and  maybe you want to integrate this as is. If so, I think this looks good, and you can ignore this comment.

-------------

PR: https://git.openjdk.org/jdk/pull/12216


More information about the hotspot-gc-dev mailing list