RFR(XL): 8220310: Implementation: NUMA-Aware Memory Allocation for G1, Mutator (1/3)

Kim Barrett kim.barrett at oracle.com
Mon Oct 14 22:20:04 UTC 2019


> On Oct 14, 2019, at 5:03 PM, Kim Barrett <kim.barrett at oracle.com> wrote:
>> 2. Add a state to the mappers to say if they are NUMA aware or not, and currently only the heap mapper should be NUMA aware. We could either set this state to true using the mtJavaHeap type as we have checked before or add an explicit setter that we only call for the heap mapper.
>> 
>> I know that only doing 2) will fix the current problem, but I think it would be nice to avoid having the base address in G1NUMA, thoughts?
> 
> I don't understand the point about mappers needing to know if they are
> NUMA or not. request_memory_on_node is only called by the two relevant
> region->space mappers, with the memory involved always in the Java
> heap (after fixing the units mismatch mentioned above). That is,
> G1NUMA::request_memory_on_node should only be called for Java heap
> memory. (It might be able to assert is_in_reserved or something like
> that, though initialization order might prevent that.)

I was confused here too.  Sangheon has repaired my confusion, and he’s
got another change in the works to tidy things up here in a way that I think
will make both me and Stefan happy.



More information about the hotspot-runtime-dev mailing list