RFR: 8255980: G1 Service thread register_task can be used after shutdown
sjohanss at openjdk.java.net
Fri Nov 6 12:24:04 UTC 2020
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.
- 8255980: G1 Service thread register_task can be used after shutdown
Stats: 31 lines in 2 files changed: 24 ins; 4 del; 3 mod
Fetch: git fetch https://git.openjdk.java.net/jdk pull/1093/head:pull/1093
More information about the hotspot-gc-dev