RFR: 8255980: G1 Service thread register_task can be used after shutdown
tschatzl at openjdk.java.net
Mon Nov 9 10:15:05 UTC 2020
On Fri, 6 Nov 2020 12:18:55 GMT, Stefan Johansson <sjohanss at openjdk.org> wrote:
> Please review this change to improve the service thread task handling during shutdown.
> This problem isn't currently visible, but with upcoming changes there is a race where tasks might be added to the service thread after it has been shut down. This becomes problematic because the task queue at that point consists of invalid objects. They are invalid because the tasks are currently stack-allocated on the service thread. This change both allocates the tasks handled by the service thread dynamically and add a check to avoid adding tasks when the service thread has been shut down. Just doing the dynamic allocation would be enough, but there is no reason to add tasks after the thread is shut down.
> I've tested this fix together with my concurrent uncommit changes which highlight the problem, and also run tier1 and tier2 for sanity.
Since `ServiceThread` is never deleted (as owned by `G1CollectedHeap`) this sufficiently keeps alive the task queue so that the code can query it and the tasks in there.
Marked as reviewed by tschatzl (Reviewer).
More information about the hotspot-gc-dev