Skip to content

Commit

Permalink
v2.1: Accounting migrating builtin programs default Compute Unit Limi…
Browse files Browse the repository at this point in the history
…t with feature status (backport of #3975) (#4091)

* Accounting migrating builtin programs default Compute Unit Limit with feature status (#3975)

* Accounting migrating builtin programs default Compute Unit Limit with its feature gate status

* Declare Non/migrating buiiltins in const array, eleminates heap allocation (Vec<>) per transaction

* updates for review commients

add explicit positional information to migrating builtin feature obj

update developer notes, added static_assertion to validate no new items are added, add enum type

* use enum to separately define migrating and not-migrating builtins

* rename for clarity

(cherry picked from commit 9379fbc)

* Fix conflicts manually

---------

Co-authored-by: Tao Zhu <[email protected]>
Co-authored-by: Tao Zhu <[email protected]>
  • Loading branch information
3 people authored Jan 16, 2025
1 parent 8b8024f commit 33ab693
Show file tree
Hide file tree
Showing 6 changed files with 451 additions and 86 deletions.
1 change: 1 addition & 0 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 2 additions & 0 deletions builtins-default-costs/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ name = "solana_builtins_default_costs"

[dev-dependencies]
rand = "0.8.5"
static_assertions = { workspace = true }

[package.metadata.docs.rs]
targets = ["x86_64-unknown-linux-gnu"]
Expand All @@ -42,6 +43,7 @@ frozen-abi = [
"dep:solana-frozen-abi",
"solana-vote-program/frozen-abi",
]
dev-context-only-utils = []

[lints]
workspace = true
Loading

0 comments on commit 33ab693

Please sign in to comment.