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

Unify BPF verifiers #17585

Merged
merged 1 commit into from
May 28, 2021
Merged

Conversation

jackcmay
Copy link
Contributor

Problem

The original rbpf verifier persisted in the rbpf crate and potentially could collect fixes that did not make it into the production verifier in the bpf_loader. Also, was causing confusion as to which one was being used.

Summary of Changes

Unify the two

Fixes #

@jackcmay jackcmay added the CI Pull Request is ready to enter CI label May 28, 2021
@codecov
Copy link

codecov bot commented May 28, 2021

Codecov Report

Merging #17585 (30c9c54) into master (a3240ae) will increase coverage by 0.0%.
The diff coverage is 100.0%.

@@           Coverage Diff            @@
##           master   #17585    +/-   ##
========================================
  Coverage    82.6%    82.7%            
========================================
  Files         427      426     -1     
  Lines      119537   119388   -149     
========================================
- Hits        98851    98782    -69     
+ Misses      20686    20606    -80     

@jackcmay jackcmay merged commit 89ee877 into solana-labs:master May 28, 2021
@jackcmay jackcmay deleted the unified-bpf-verifier branch May 28, 2021 21:24
Lichtso added a commit to Lichtso/solana that referenced this pull request Nov 17, 2021
Lichtso added a commit to Lichtso/solana that referenced this pull request Nov 17, 2021
Lichtso added a commit to Lichtso/solana that referenced this pull request Nov 24, 2021
Lichtso added a commit to Lichtso/solana that referenced this pull request Dec 3, 2021
Lichtso added a commit to Lichtso/solana that referenced this pull request Dec 3, 2021
Lichtso added a commit to Lichtso/solana that referenced this pull request Dec 3, 2021
Lichtso added a commit to Lichtso/solana that referenced this pull request Dec 3, 2021
t-nelson pushed a commit that referenced this pull request Dec 4, 2021
t-nelson added a commit to t-nelson/solana that referenced this pull request Dec 13, 2021
tao-stones pushed a commit that referenced this pull request Dec 13, 2021
Lichtso added a commit to Lichtso/solana that referenced this pull request Dec 13, 2021
Lichtso added a commit to Lichtso/solana that referenced this pull request Dec 13, 2021
mvines pushed a commit that referenced this pull request Dec 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Pull Request is ready to enter CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant