Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

atomically read page_bitmaps #810

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

atomically read page_bitmaps #810

wants to merge 1 commit into from

Commits on Dec 25, 2023

  1. atomically read page_bitmaps 

    page_bitmaps does not need to be volatile since it is going to be treated as atomic. In fact, as long as each variable is treated as atomic, we do not need volatile at all, as that means it is modified by outside means, aka hardware, etc.

The atomic macros already ensure optimizations that could cause incorrect values to be used do not happen.

Finally, the first loop is redundant as they are iterated over again in the loop below.
    @AreaZR
    AreaZR committed Dec 25, 2023
    Configuration menu
    Copy the full SHA
    8b53a14 View commit details
    Browse the repository at this point in the history