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

BpfLoader211111... programs can't CPI into BpfLoader111111.. programs #11686

Closed
hendrikhofstadt opened this issue Aug 18, 2020 · 3 comments · Fixed by #11695
Closed

BpfLoader211111... programs can't CPI into BpfLoader111111.. programs #11686

hendrikhofstadt opened this issue Aug 18, 2020 · 3 comments · Fixed by #11695
Assignees
Milestone

Comments

@hendrikhofstadt
Copy link
Contributor

Problem

fetch-spl.sh uses the deprecated BpfLoader11111 which breaks the default SPL programs when launched via the runscript.

Proposed Solution

SPL in the runscript should compiled for and use the new BpfLoader211111.

@mvines
Copy link
Member

mvines commented Aug 18, 2020

Hmm, that's strange. fetch-spl.sh downloads the 1.0.0 version of the SPL programs, which require the old loader and will not work with the new loader. Maybe something else is going on in your environment @hendrikhofstadt?

@hendrikhofstadt
Copy link
Contributor Author

@mvines Technially it's "working" but I can't CPI into the old token program. I misworded the issue I think.

Are there plans to support CPI between incompatible BPFLoader versions?

@mvines mvines changed the title Upgrade run script and SPL for new bpf loader BpfLoader211111... programs can't CPI into BpfLoader111111.. programs Aug 18, 2020
@mvines
Copy link
Member

mvines commented Aug 18, 2020

Ah, that makes sense. Yes we need to support that, since we can't force spl-token 1.0.0s to upgrade. I fixed the title of the issue to reflect this.

@mvines mvines added this to the v1.2.25 milestone Aug 18, 2020
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 a pull request may close this issue.

3 participants