RFR: 8310874: Runthese30m crashes with klass should be in the placeholders during verification
Coleen Phillimore
coleenp at openjdk.org
Mon Jul 24 12:53:15 UTC 2023
On Fri, 14 Jul 2023 12:39:50 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:
> For non-parallel capable class loaders, the class is not put in the placeholders table, so a verification at the wrong time will hit this assert. This trivial change removes the invalid assert.
> Tested with inserting a LoaderConstraintTable::verify while running jck tests.
is_parallelCapable is something that systemDictionary.cpp knows so exporting it for this somewhat rare case in an assert seems like adding complications for no real benefit to code safety. The whole parallel-capable conditionals makes class loading complicated and it's better to try to isolate it, not spread it.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/14887#issuecomment-1647848509
More information about the hotspot-runtime-dev
mailing list