On Wed, 4 Feb 2026 01:14:05 GMT, Guanqiang Han <[email protected]> wrote:

>> Please review this change. Thanks!
>> 
>> **Description:**
>> 
>> VM crashes during startup on x86 when running with -XX:+UseAESCTRIntrinsics 
>> -XX:-UseAES. In this configuration, UseAESCTRIntrinsics may remain enabled 
>> while UseAES is explicitly disabled, and the VM generates AES-CTR stubs, 
>> hitting an assert(UseAES) in generate_counterMode_AESCrypt_Parallel().
>> 
>> **Fix:**
>> 
>> Update x86 flag initialization to enforce the dependency between 
>> UseAESCTRIntrinsics and UseAES. When UseAES is disabled, explicitly disable 
>> UseAESCTRIntrinsics (with a warning when it was set on the command line), 
>> aligning behavior with the existing UseAES/UseAESIntrinsics gating and 
>> avoiding stub generation with inconsistent flag states.
>> 
>> **Test:**
>> 
>> GHA
>
> Guanqiang Han has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Fix improper use of FLAG_SET_DEFAULT() under 
> !FLAG_IS_DEFAULT(UseAESCTRIntrinsics)

Looks good.

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

Marked as reviewed by kvn (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/29338#pullrequestreview-3748734975

Reply via email to