[lworld] Integrated: 8252024: [lworld] Lock ordering issue when printing nmethod labels
Tobias Hartmann
thartmann at openjdk.java.net
Mon Dec 14 12:26:14 UTC 2020
On Mon, 14 Dec 2020 11:21:29 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:
> We hit an assert while printing nmethod entry point labels when attempting to create a symbol from the `SigEntry` to iterate over. The problem is that we are holding the tty lock and symbol creation also acquires certain locks. I've refactored the code to completely avoid symbol creation and also fixed an issue where names of reference argument types are not printed anymore (because we don't keep track of that in `SigEntry`). I've also removed lots of dead code leftovers from old reserved entry support in the calling convention.
>
> Thanks,
> Tobias
This pull request has now been integrated.
Changeset: 34232ad1
Author: Tobias Hartmann <thartmann at openjdk.org>
URL: https://git.openjdk.java.net/valhalla/commit/34232ad1
Stats: 186 lines in 13 files changed: 34 ins; 78 del; 74 mod
8252024: [lworld] Lock ordering issue when printing nmethod labels
-------------
PR: https://git.openjdk.java.net/valhalla/pull/302
More information about the valhalla-dev
mailing list