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.

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

PoC: config write guard #2360

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

Commits on Oct 22, 2024

  1. add and use configuration write guard

    the configuration write guard is now required when the configuration
    struct shall be mutated. the write guards locks multiple writers against
    each other and also, more importantly, makes the writes synchronous to
    the main loop. all code running in the main loop can now be sure that
    (1) reads from the configuration struct are non-preemtive and (2) the
    configuration struct as a whole is in a consistent state when reading
    from it.
    
    NOTE that acquiring a write guard from within the main loop's task will
    immediately cause a deadlock and the watchdog will trigger a reset. if
    writing from inside the main loop should ever become necessary, the
    write guard must be updated to only lock the mutex but not wait for a
    signal.
    schlimmchen committed Oct 22, 2024
    Configuration menu
    Copy the full SHA
    b1edb13 View commit details
    Browse the repository at this point in the history