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

add upgrade_core_bpf_program module #2

Closed
wants to merge 1 commit into from

Conversation

buffalojoec
Copy link
Owner

@buffalojoec buffalojoec commented Nov 18, 2023

This PR is a draft for now! As talks on SIMD XXXX continue, I'll jot notes here.

One major note: We'll need to roll a CLI change in order to deploy a program in its finalized version using the non-upgradeable loader. This is necessary since Core BPF programs must be owned by the non-upgradeable loader and must not have a program data account.

@buffalojoec buffalojoec force-pushed the migrate-native-program branch from e7156cb to fe3398e Compare November 18, 2023 14:33
@buffalojoec buffalojoec changed the title add migrate_native_program module add upgrade_core_bpf_program module Nov 18, 2023
#[allow(dead_code)]
pub fn upgrade_core_bpf_program(
bank: &Bank,
target: NativeProgram,
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because of this parameter, it might make sense to rename the enum from NativeProgram to something like CoreProgram. Or we can use a type alias, ie: type NativeProgram = CoreProgram;.

@buffalojoec buffalojoec force-pushed the migrate-native-program branch from fe3398e to a7c1f7e Compare November 30, 2023 21:12
@github-actions github-actions bot added the stale label Dec 15, 2023
@github-actions github-actions bot closed this Dec 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant