Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8277411

C2 fast_unlock intrinsic on AArch64 has unnecessary ownership check

    XMLWordPrintable

Details

    • b26
    • aarch64

    Backports

      Description

        The AArch64 fast_unlock C2 code checks if the current thread owns the lock. This can be surprisingly expensive in workload where locking is contended. The check is however optional (helpful only for finding JNI code bugs), and indeed not emitted for x86_64. We should consider removing it on AArch64.

        Attachments

          Issue Links

            backported by

            Backport - A issue that is required to port a Bug or Feature into another product release. This issue type is generally associated with the main Bug/Feature to represent each individual release of the port. JDK-8293750 C2 fast_unlock intrinsic on AArch64 has unnecessary ownership check

            • P4 - Minor loss of function, or other problem where easy workaround is present.
            • Resolved

            Backport - A issue that is required to port a Bug or Feature into another product release. This issue type is generally associated with the main Bug/Feature to represent each individual release of the port. JDK-8294123 C2 fast_unlock intrinsic on AArch64 has unnecessary ownership check

            • P4 - Minor loss of function, or other problem where easy workaround is present.
            • Resolved
            relates to

            Enhancement - null JDK-8277877 C2 fast_unlock intrinsic on riscv has unnecessary ownership check

            • P4 - Minor loss of function, or other problem where easy workaround is present.
            • Resolved
            links to

            Commit Commit openjdk/jdk17u-dev/b196bc4c

            Commit Commit openjdk/jdk/3d810ad6

            Review Review openjdk/jdk17u-dev/660

            Review Review openjdk/jdk/6498

            Activity

              People

                eosterlund Erik Österlund
                eosterlund Erik Österlund
                Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved: