Integrated: 8261480: MetaspaceShared::preload_and_dump should check exceptions

[Ioi Lam]

On Wed, 10 Mar 2021 20:37:17 GMT, Ioi Lam <iklam at openjdk.org> wrote:

> `MetaspaceShared::preload_and_dump()`, as well as many of the functions that it calls, have extensive use of `THREAD` when calling functions that can potentially throw exceptions. Many of these call sites explicitly check for failure status, or assume that the caller would terminate the VM if an exception were to happen. This makes the code hard to understand and potentially unsafe.
> 
> I have make 3 types of changes:
> 
> 1. In most cases, replace THREAD with CHECK. Moved all the fatal error check to `MetaspaceShared::preload_and_dump()`, where it handles any uncaught exceptions and explicitly terminates the VM.
> 2. If a `TRAPS` function can never throw, replace the `TRAPS` declaration with `Thread* current`
> 3. In cases where an exception is handled and discarded, use `THREAD` immediately followed by  `HAS_PENDING_EXCEPTION` -> `CLEAR_PENDING_EXCEPTION` to make the code easy to understand. HandleMark is used to assert that no exception escapes.
> 
> (See `MetaspaceShared::try_link_class` for an example of 2 and 3).
> 
> Tested with tiers 1-4.

This pull request has now been integrated.

Changeset: 2b93ae00
Author:    Ioi Lam <iklam at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/2b93ae00
Stats:     411 lines in 13 files changed: 152 ins; 151 del; 108 mod

8261480: MetaspaceShared::preload_and_dump should check exceptions

Reviewed-by: dholmes, ccheung

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

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