RFR: 8319747: galoisCounterMode_AESCrypt stack walking broken

Vladimir Kozlov kvn at openjdk.org
Wed Nov 8 22:41:57 UTC 2023


On Wed, 8 Nov 2023 18:11:58 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

Looks good.  Please run tier3 too which will execute the test as part for `hotspot_vector_2` group with different AVX settings.

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

Marked as reviewed by kvn (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/16567#pullrequestreview-1721395834


More information about the hotspot-compiler-dev mailing list