Convert an initial set of ordered reads into WT_READ_ONCEs.

XMLWordPrintableJSON

    • Type: Task
    • Resolution: Won't Fix
    • Priority: Major - P3
    • None
    • Affects Version/s: None
    • Component/s: Not Applicable
    • StorEng - 2023-08-08, ASeasonTooMany-2023-08-22, BermudaTriangle- 2023-09-05, TheMoon-StorEng - 2023-09-19, NachoCheese - 2023-10-03, Joker - StorEng - 2023-10-17, Asparagus-StorEng - 2023-10-31, c(3x10^8)-StorEng - 2023-11-14, 2023-11-28 - Anthill Tiger
    • 5

      There are three WT_ORDERED_READs in generations.c that have the comment: "Ensure we only read the value once.". The investigation in WT-11280 highlighted that these WT_ORDERED_READs effectively serve no purpose as they aren't enforcing ordering and the generation arrays are all volatile already.

      The code wants READ_ONCE semantics and already has it but also chose to use ordered reads.

      There are other places in the code where obvious volatile casts exist to achieve READ_ONCE, we should also convert those.

              Assignee:
              [DO NOT USE] Backlog - Storage Engines Team
              Reporter:
              Luke Pearson
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Created:
                Updated:
                Resolved: