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

backport: Merge bitcoin#21845 #5993

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

vijaydasmp
Copy link

bitcoin backports

@vijaydasmp vijaydasmp changed the title Bp22 35 backport : Merge bitcoin#21775, 21753, 21710, 21845 Apr 23, 2024
@vijaydasmp vijaydasmp changed the title backport : Merge bitcoin#21775, 21753, 21710, 21845 backport: Merge bitcoin#21775, 21753, 21710, 21845 Apr 23, 2024
@vijaydasmp vijaydasmp force-pushed the bp22_35 branch 9 times, most recently from dd5a340 to 866adbb Compare April 26, 2024 01:16
Copy link

This pull request has conflicts, please rebase.

@vijaydasmp vijaydasmp changed the title backport: Merge bitcoin#21775, 21753, 21710, 21845 backport: Merge bitcoin#21845 May 3, 2024
…fore calling RelayTransactions()

39e1971 [net processing] Add internal _RelayTransactions() (John Newbery)

Pull request description:

  As part of the general effort to reduce cs_main usage in net_processing, this removes the need to be holding `cs_main` when calling `RelayTransactions()` from outside net_processing. Internally, we lock `cs_main` and call an internal `_RelayTransactions()` function that _does_ require `cs_main`.

ACKs for top commit:
  MarcoFalke:
    re-unsigned-code-review ACK 39e1971
  promag:
    Code review ACK 39e1971, just included sync.h since last review.
  ajtowns:
    ACK 39e1971

Tree-SHA512: dc08441233adfb8eaac501cf497cb4bad029eb723bd3fa8a3d8b7e49cc984c98859b95780ad15f5701d62ac745a8223beb0df405e3d49d95a8c86c8be17c9543
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants