Add Atomic<T> to package/replace idiom of volatile var plus AtomicAccess:: operations

XMLWordPrintable

    • Type: Enhancement
    • Resolution: Fixed
    • Priority: P4
    • 26
    • Affects Version/s: None
    • Component/s: hotspot
    • b25

      Lock-free code in HotSpot uses an idiom of declaring variables `volatile` and using operations from the `AtomicAccess` class (formerly `Atomic`) to access and modify those variables. It would be better to package that idiom, preventing accidental misuse such as using non-atomic read-modify-write arithmetic operations on such variables.

      The proposal is to add `Atomic<T>`, where `T` is a supported atomic type as before. Then incrementally replace (most) uses of that idiom with uses of this new type. "Most" because it is expected there will be some small residue where use of the `Atomic<T>` class isn't appropriate. One example is BitMap, which provides some atomic operations but where it would not be appropriate to change the underlying data to be an array of `Atomic<bm_word_t>`. Another example is the atomic functions in the `Copy` class.

            Assignee:
            Kim Barrett
            Reporter:
            Kim Barrett
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: