RFR: 8269537: memset() is called after operator new [v3]
    Leo Korinth 
    lkorinth at openjdk.java.net
       
    Tue Oct 19 13:46:53 UTC 2021
    
    
  
On Thu, 7 Oct 2021 06:21:52 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:
>> Leo Korinth has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Use a thread local buffer so that the compiler might reorder operator new.
>
> src/hotspot/share/memory/allocation.hpp line 414:
> 
>> 412:   // allocation is done in a recursive step. In that case an assert will trigger.
>> 413:   class RecentAllocations {
>> 414:     static const unsigned BufferSize = 5;
> 
> "5" seems like an odd choice.
What would you suggest? I could make it "2", but then a smaller code change (or compiler upgrade) would force us to change the size. Any size is quite arbitrary, I will update this number to your suggestion if you give me a suggestion.
-------------
PR: https://git.openjdk.java.net/jdk/pull/5387
    
    
More information about the hotspot-dev
mailing list