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

new runtime fuzzer pb defs rough poc draft #19

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ibhatt-jumptrading
Copy link
Contributor

No description provided.

Copy link

@topointon-jump topointon-jump left a comment

Choose a reason for hiding this comment

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

🔥 nice one! comments so far

repeated AcctState acct_states = 1;
repeated Feature features = 2;
repeated SlotEnv slots = 3;
EpochSchedule epoch_schedule = 4;

Choose a reason for hiding this comment

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

Leader schedule

int32 todo = 1;
}

message SysvarCache {

Choose a reason for hiding this comment

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

remove this

proto_v2/slot_v2.proto Show resolved Hide resolved
proto_v2/slot_v2.proto Show resolved Hide resolved
proto_v2/slot_v2.proto Show resolved Hide resolved
Comment on lines +63 to +65
repeated AcctState reward_acct_states = 1;
repeated AcctState rent_acct_states = 2;
repeated AcctState sysvar_acct_states = 4;

Choose a reason for hiding this comment

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

collapse into just a single list of accounts

Choose a reason for hiding this comment

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

make it clear that these are accounts that have changed

repeated AcctState reward_acct_states = 1;
repeated AcctState rent_acct_states = 2;
repeated AcctState sysvar_acct_states = 4;
repeated TxnEnv txn_envs = 5; /* Contains all resultant account states */

Choose a reason for hiding this comment

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

txn effects

proto_v2/slot_v2.proto Show resolved Hide resolved
proto_v2/slot_v2.proto Show resolved Hide resolved
proto_v2/txn_v2.proto Show resolved Hide resolved
proto_v2/txn_v2.proto Show resolved Hide resolved
repeated AcctState acct_states = 1;
repeated Feature features = 2;
repeated SlotEnv slots = 3;
EpochSchedule epoch_schedule = 4;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

epoch schedule in acct states


import "slot_v2.proto";

message AcctState {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

keep acct states in epoch effects

Copy link
Contributor Author

Choose a reason for hiding this comment

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

bhq

}

message SlotEffects {
/* The resulting state of the accounts after the slot */
Copy link
Contributor Author

Choose a reason for hiding this comment

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

block valid

/* Account states after instruction execution in the order as specified by
instruction. These states are then updated at fuzzers that operate at
the transaction, slot, and epoch granularity. */
repeated AcctState accounts = 5;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

remvoe

@ibhatt-jumptrading
Copy link
Contributor Author

  1. iron out pbs
  2. dumping
  3. harness(es) (fd, agave) and make sure it works
  4. structured fuzzers -- solfuzz

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 this pull request may close these issues.

3 participants