RFR: 8317007: Add bulk removal of dead nmethods during class unloading [v4]

Wed Dec 13 12:26:39 UTC 2023

On Wed, 13 Dec 2023 11:30:53 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

>> I left this out because at the time we unlink nmethods we have not moved the objects yet, only marked them. If we moved the class unloading to where `prune_nmethods_not_into_young` is, it might be possible. Going to file a CR to investigate this.
>
> Fwiw, I did not move the class unloading because in one of the last changes you seemed fairly insistent on keeping ordering and location of class unloading consistent across stw collectors too.

> at the time we unlink nmethods we have not moved the objects yet

I was thinking of calling both at mark-end, but I overlooked that objs have not been relocated yet... The current impl looks good.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16767#discussion_r1425282188