RFR: 8282355: compiler/arguments/TestCodeEntryAlignment.java failed "guarantee(sect->end() <= tend) failed: sanity" [v2]

[Dean Long]

On Sun, 13 Mar 2022 07:12:57 GMT, Jie Fu <jiefu at openjdk.org> wrote:

>> The only reason is because -XX:CodeCacheSegmentSize=16 gives an error.  If it's important to test for -XX:CodeEntryAlignment=16 then I'll have to rework the logic to be more clever.  I'll probably have to use WhiteBox APIs to read the default CodeCacheSegmentSize and only set it to values that are >= the default.  It's unfortunate that -XX:CodeEntryAlignment doesn't adjust the default value of CodeCacheSegmentSize automatically.
>
> Maybe we can add testing for `-XX:CodeEntryAlignment={512, 1024}` like this.
> 
> diff --git a/test/hotspot/jtreg/compiler/arguments/TestCodeEntryAlignment.java b/test/hotspot/jtreg/compiler/arguments/TestCodeEntryAlignment.java
> index fd6c8ca..f07030e 100644
> --- a/test/hotspot/jtreg/compiler/arguments/TestCodeEntryAlignment.java
> +++ b/test/hotspot/jtreg/compiler/arguments/TestCodeEntryAlignment.java
> @@ -72,6 +72,14 @@ public class TestCodeEntryAlignment {
>                  "-XX:CodeEntryAlignment=" + align
>              );
>          }
> +
> +        for (int align = 512; align < 1024; align *= 2) {
> +            shouldPass(
> +                "-XX:+UnlockExperimentalVMOptions",
> +                "-XX:CodeCacheSegmentSize=" + align,
> +                "-XX:CodeEntryAlignment=" + align
> +            );
> +        }
>      }
>  
>  }
> 
> What do you think?

That sounds good.  Done.

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

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