Skip to content

BIP 440/441: adjust varops budget calculation and add missing OP_BOOLAND - #2231

Draft
jmoik wants to merge 5 commits into
bitcoin:masterfrom
jmoik:gsr
Draft

BIP 440/441: adjust varops budget calculation and add missing OP_BOOLAND#2231
jmoik wants to merge 5 commits into
bitcoin:masterfrom
jmoik:gsr

Conversation

@jmoik

@jmoik jmoik commented Aug 1, 2026

Copy link
Copy Markdown

Updates BIP 440 and BIP 441:

  1. document mixed-version resource limits, including the potential 2× validation cost of adversarial input mixing
  2. add the missing OP_BOOLAND entry to BIP 441
  3. clarify postcharging for linear operations, precharging for expensive operations, and conditional signature-check costs
  4. apply the conditional signature-check formulas to BIP 441
  5. update BIP 441’s reference implementation link to the gsr-core branch

@jonatack jonatack added the BIP Update by Owner PR by Author or Deputy to modify their own BIP label Aug 1, 2026
@murchandamus

Copy link
Copy Markdown
Member

I haven’t looked at this yet, because it is marked as a draft pull request. Is this still waiting for someone’s review in particular or was that perhaps an oversight?

Julian Moik added 5 commits August 12, 2026 17:43
Document bounded linear postcharging and require superlinear operations and signature checks to be precharged.

Charge SIGCHECK only for nonempty signature operands.
Define sigcheck_cost from signature emptiness and apply it to CHECKSIG, CHECKSIGVERIFY, and CHECKSIGADD.
@jmoik

jmoik commented Aug 12, 2026

Copy link
Copy Markdown
Author

I haven’t looked at this yet, because it is marked as a draft pull request. Is this still waiting for someone’s review in particular or was that perhaps an oversight?

This was intended, still discussing the approach with Rusty, I have now force pushed over the original commits and only applied a note describing this mixed-version resource limit issue, instead of complicating the budget calculation when mixed-version inputs are present.

I will move it to "Ready for review" when it's clear.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BIP Update by Owner PR by Author or Deputy to modify their own BIP

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants