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

Zero: accessor_entry misses the IRIW case

    XMLWordPrintable

Details

    • b21

    Backports

      Description

        While doing a change in related area, I noticed there is no IRIW handling block in accessor_entry when reading volatile fields. This probably breaks PPC64 Zero.

        There is a block in bytecodeInterpreter.cpp for common code:

                  if (cache->is_volatile()) {
                    if (support_IRIW_for_not_multiple_copy_atomic_cpu) {
                      OrderAccess::fence();
                    }

        AFAIU, this mostly affects 8u that does not have PPC64 port yet.

        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-8255537 Zero: accessor_entry misses the IRIW case

            • P3 - Major loss of function.
            • 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-8256035 Zero: accessor_entry misses the IRIW case

            • P3 - Major loss of function.
            • 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-8256231 Zero: accessor_entry misses the IRIW case

            • P3 - Major loss of function.
            • Resolved
            links to

            Commit Commit openjdk/jdk/bd45191f

            Review Review openjdk/jdk/766

            Activity

              People

                shade Aleksey Shipilev
                shade Aleksey Shipilev
                Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved: