Page MenuHomePhabricator
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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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 ↗(On Diff #33503)

Why is that change needed ?

deadalnix added inline comments.May 13 2022, 13:22
src/rcu.cpp
230 ↗(On Diff #33503)

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.

Revision Contents
Changeset List

PathSize
src/
8 lines
31 lines
test/
46 lines

Diff 33506

View Options

src/rcu.h

Loading...
View Options

src/rcu.cpp

Loading...
View Options

src/test/rcu_tests.cpp

Loading...