Details

Reviewers

Group Reviewers

Restricted Project

Commits

rABCe88e4271c0d9: net_processing: move extra transactions to msgproc mutex

Summary

Previously vExtraTxnForCompact and vExtraTxnForCompactIt were protected
by g_cs_orphans; protect them by g_msgproc_mutex instead, as they
are only used during message processing.

Partial backport of core#26295:
https://github.com/bitcoin/bitcoin/pull/26295/commits/89e2e0da0bcd0b0757d7b42907e2d2214da9f68b

Depends on D16226.

Test Plan

With Clang and Debug:

ninja all check

./contrib/teamcity/build-configurations.py build-tsan

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.Tue, May 28, 13:34

Herald added a reviewer: Restricted Project. · View Herald TranscriptTue, May 28, 13:34

Fabien requested review of this revision.Tue, May 28, 13:34

teamcity edited the summary of this revision. (Show Details)Tue, May 28, 13:34

@bot build-tsan

teamcity awarded a token.Tue, May 28, 13:36

Fabien added a child revision: D16229: net_processing: Pass a Peer& to ProcessOrphanTx.Tue, May 28, 13:39

Harbormaster completed remote builds in B29141: Diff 47983.Tue, May 28, 14:11

PiRK accepted this revision.Tue, May 28, 15:40

This revision is now accepted and ready to land.Tue, May 28, 15:40

Closed by commit rABCe88e4271c0d9: net_processing: move extra transactions to msgproc mutex (authored by Anthony Towns <aj@erisian.com.au>, committed by Fabien). · Explain WhyWed, May 29, 07:46

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABCe88e4271c0d9: net_processing: move extra transactions to msgproc mutex.

net_processing: move extra transactions to msgproc mutex
ClosedPublic
Actions

Details

Diff Detail

Event Timeline

Revision Contents
Changeset List

Diff 48000

src/net_processing.cpp

src/txorphanage.h

net_processing: move extra transactions to msgproc mutexClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 48000

src/net_processing.cpp

src/txorphanage.h

net_processing: move extra transactions to msgproc mutex
ClosedPublic
Actions

Revision Contents
Changeset List