Differential D11458

[RCU] Run a lazy round of cleanup when unlocking the RCULock
ClosedPublic
Actions

Authored by deadalnix on May 13 2022, 12:00.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABC28064258b769: [RCU] Run a lazy round of cleanup when unlocking the RCULock

Summary

This ensures that we will run the cleanup little by little without a blocking operation such as synchronize.

Test Plan

Added test cases.

Diff Detail

Repository

rABC Bitcoin ABC

Branch

lazycleanup

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 19017
Build 37790: Build Diff	build-diff · lint-circular-dependencies · build-debug · build-without-wallet · build-clang-tidy · build-clang
Build 37789: arc lint + arc unit

Event Timeline

deadalnix created this revision.May 13 2022, 12:00

Herald added a reviewer: Restricted Project. · View Herald TranscriptMay 13 2022, 12:00

deadalnix requested review of this revision.May 13 2022, 12:00

Harbormaster completed remote builds in B19017: Diff 33503.May 13 2022, 12:16

Fabien added a subscriber: Fabien.May 13 2022, 13:19

Fabien added inline comments.

src/rcu.cpp
230	Why is that change needed ?

deadalnix added inline comments.May 13 2022, 13:22

src/rcu.cpp
230	So that the cleanup doesn't appears as if it was still in the map while running.

Fabien accepted this revision.May 13 2022, 13:40

This revision is now accepted and ready to land.May 13 2022, 13:40

deadalnix retitled this revision from [radix] Run a lazy round of cleanup when unlocking the RCULock to [RCU] Run a lazy round of cleanup when unlocking the RCULock.May 13 2022, 14:06

Closed by commit rABC28064258b769: [RCU] Run a lazy round of cleanup when unlocking the RCULock (authored by deadalnix). · Explain WhyMay 13 2022, 14:33

This revision was automatically updated to reflect the committed changes.

deadalnix added a commit: rABC28064258b769: [RCU] Run a lazy round of cleanup when unlocking the RCULock.