RFR: 8319747: galoisCounterMode_AESCrypt stack walking broken [v2]

Daniel Jeliński djelinski at openjdk.org
Thu Nov 16 08:58:43 UTC 2023


On Tue, 14 Nov 2023 08:05:56 GMT, Daniel Jeliński <djelinski at openjdk.org> wrote:

>> This PR fixes stack walking by removing incorrect RBP change in the middle of a stub.
>> 
>> I also changed the register assignment to avoid saving RSI on Windows.
>> 
>> Testing:
>> - test/hotspot/jtreg/compiler/codegen/aes/TestAESMain.java still passes, both on Windows and Linux
>> - mach5 tier1-2 (including security and TLS-heavy networking tests) still pass
>> - the generated flamegraphs (attached in JBS) show the correct stack trace with this fix
>
> Daniel Jeliński has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Revert register renaming

Thanks for the reviews!

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

PR Comment: https://git.openjdk.org/jdk/pull/16567#issuecomment-1814026656


More information about the hotspot-compiler-dev mailing list