Integrated: 8255980: G1 Service thread register_task can be used after shutdown

Stefan Johansson sjohanss at openjdk.java.net
Mon Nov 9 14:28:58 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.

This pull request has now been integrated.

Changeset: 79b79095
Author:    Stefan Johansson <sjohanss at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/79b79095
Stats:     31 lines in 2 files changed: 24 ins; 4 del; 3 mod

8255980: G1 Service thread register_task can be used after shutdown

Reviewed-by: tschatzl, ayang

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

PR: https://git.openjdk.java.net/jdk/pull/1093


More information about the hotspot-gc-dev mailing list