8248130: AArch64: pf() uses mtNone instead of mtInternal

Stefan Karlsson stefan.karlsson at oracle.com
Tue Jun 23 10:02:25 UTC 2020


Thanks, David.

StefanK

On 2020-06-23 11:50, David Holmes wrote:
> Looks good and trivial.
> 
> Thanks,
> David
> 
> On 23/06/2020 5:14 pm, Stefan Karlsson wrote:
>> Hi all,
>>
>> Please review this trivial patch to change the NMT flag used by one of 
>> the debugging functions on AArch64.
>>
>> https://cr.openjdk.java.net/~stefank/8248130/webrev.01/
>> https://bugs.openjdk.java.net/browse/JDK-8248130
>>
>> If pf() is used when NMT is turned on we hit this assert:
>> MallocSite* MallocSiteTable::lookup_or_add(const NativeCallStack& key, 
>> size_t* bucket_idx,
>>    size_t* pos_idx, MEMFLAGS flags) {
>>    assert(flags != mtNone, "Should have a real memory type");
>>
>> The patch simply changes the used flag to mtInternal.
>>
>> Thanks,
>> StefanK


More information about the hotspot-dev mailing list