Integrated: 8271073: Improve testing with VM option VerifyArchivedFields

Ioi Lam iklam at openjdk.java.net
Thu Sep 16 23:30:53 UTC 2021


On Tue, 14 Sep 2021 22:16:56 GMT, Ioi Lam <iklam at openjdk.org> wrote:

> - Changed the definition of `VerifyArchivedFields` from a whacky use of `bool` to an `int` and properly define its three levels:
>   - 0: No verification 
>   - 1: Basic verification with VM_Verify (no side effects)
>   - 2: Detailed verification by forcing a GC (with side effects)
> - Changed the default value to 0. The functionality checked by this flag has been very stable so there's no need to verify it in every single test case.
> - Enabled `-XX:VerifyArchivedFields=1` for all CDS test cases.
> - Added a new test case for `-XX:VerifyArchivedFields=2` .
> - Also added comments about that this flag is suppose to check for.

This pull request has now been integrated.

Changeset: b9829044
Author:    Ioi Lam <iklam at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/b98290444a4594d0164d6f313c506287282d1929
Stats:     86 lines in 5 files changed: 73 ins; 0 del; 13 mod

8271073: Improve testing with VM option VerifyArchivedFields

Reviewed-by: ccheung, minqi

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

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


More information about the hotspot-dev mailing list