-
Notifications
You must be signed in to change notification settings - Fork 108
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
Update ARM Access for det_ext changes #840
base: det_ext_state
Are you sure you want to change the base?
Conversation
|
|
||
lemma thread_set_time_slice_pas_refined[wp]: | ||
"thread_set_time_slice tptr time \<lbrace>pas_refined aag\<rbrace>" | ||
by (simp add: thread_set_time_slice_def | wp)+ | ||
"\<lbrace>pas_refined aag and pspace_aligned and valid_vspace_objs and valid_arch_state\<rbrace> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This precondition (and those of several similar lemmas) could be weakened using the changes from 357ccc9. I'll do so once that commit is either in master or is able to be cherry-picked into this branch.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
does this also apply to lemma pas_refined
?
9788c54
to
6f4d440
Compare
Is the plan to rebase this over #844? |
I think it already is. This PR has 3 commits, but only one is new, and the second one is the same as in 844 (currently) |
apply (clarsimp simp: etcbs_of'_def split: if_split_asm kernel_object.splits) | ||
apply (clarsimp simp: default_object_def default_tcb_def tyunt split: apiobject_type.splits) | ||
defer | ||
apply (force intro: domtcbs) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this copied from elsewhere? post-defer
indent is wrong, and probably there's no need to split obj and aobj separately anyway, and on top of that force can probably finish it after the erule, or maybe even do the whole lemma if we're lucky
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One minor comment, else good. Nice to see more red than expected.
Signed-off-by: Corey Lewis <[email protected]>
6f4d440
to
b4bd341
Compare
Following on from #824 and #844, this PR additionally updates the Access Control proofs for the changes to det_ext and the scheduler state.