RFR: 8357473: Compilation spike leaves many CompileTasks in free list

[Aleksey Shipilev]

On Fri, 23 May 2025 16:03:55 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>> See bug for more discussion.
>> 
>> This PR implements the "all the way" solution by removing the free list completely. It complements https://github.com/openjdk/jdk/pull/25364, and can go either first, or second. We will remerge the other one once either integrates.
>> 
>> Additional testing:
>>  - [x] Linux x86_64 server fastdebug, `compiler`
>>  - [ ] Linux AArch64 server fastdebug, `all`
>
> src/hotspot/share/compiler/compileTask.cpp line 81:
> 
>> 79: 
>> 80: CompileTask::~CompileTask() {
>> 81:   assert(!lock()->is_locked(), "Should not be locked when freed");
> 
> Do we need to free `_lock` ?

Ah yes, duh! I cherry-picked this hunk from my other PR that moves this lock to global, so it does not require delete. For this PR, we need `delete lock;` as well. Will add.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25409#discussion_r2105079218