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

Cleanup vGIC fault handling code #111

Merged
merged 2 commits into from
Aug 13, 2024
Merged

Cleanup vGIC fault handling code #111

merged 2 commits into from
Aug 13, 2024

Conversation

Ivan-Velickovic
Copy link
Collaborator

  • Make virq_controller_init for the vGIC register the vGIC region with the appropriate callbacks using the library's API instead of hard-coding it in the fault handling code.
  • Don't use BOARD_ defines, instead just use seL4 based defines since we eventually want the library to be Microkit agnostic.

Still not satisfied with the vGIC/fault handling code, there's still lots of improvements to be made.

* Make virq_controller_init for the vGIC register the vGIC region
  with the appropriate callbacks using the library's API instead of
  hard-coding it in the fault handling code.
* Don't use `BOARD_` defines, instead just use seL4 based defines
  since we eventually want the library to be Microkit agnostic.

Still not satisfied with the vGIC/fault handling code, there's still
lots of improvements to be made.

Signed-off-by: Ivan Velickovic <[email protected]>
@Ivan-Velickovic Ivan-Velickovic merged commit 4941d21 into main Aug 13, 2024
5 of 8 checks passed
@Ivan-Velickovic Ivan-Velickovic deleted the vgic_cleanup branch August 13, 2024 02:09
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.

1 participant