Page MenuHomePhabricator

Guard vRecvGetData (now in net processing) with its own mutex

Authored by PiRK on Wed, Nov 24, 17:19.



This requires slightly reorganizing the logic in GETBLOCKTXN to
maintain locking order.

This is a backport of core#19911 [5/6]

see D10187 for the removal of the !cs_main negative lock

Depends on D10529

Test Plan
ninja && ninja check check-functional

Diff Detail

rABC Bitcoin ABC
Lint Not Applicable
Tests Not Applicable

Event Timeline

PiRK requested review of this revision.Wed, Nov 24, 17:19
Fabien requested changes to this revision.Thu, Nov 25, 11:04
Fabien added a subscriber: Fabien.

Same as D10528, the test plan doesn't test for the changes

This revision now requires changes to proceed.Thu, Nov 25, 11:04
PiRK requested review of this revision.Thu, Nov 25, 13:05
PiRK edited the test plan for this revision. (Show Details)

@bot build-tsan

This revision is now accepted and ready to land.Thu, Nov 25, 15:14