The diff is handling hyp-mode and making all the necessary operations in order to be able to run bhyve-arm (which will be added at the next diff)
Details
Details
- Reviewers
andrew - Group Reviewers
ARM - Commits
- rS315712: Add the early hypervisor code needed on 32-bit ARMv7. This will be used
Diff Detail
Diff Detail
- Repository
- rS FreeBSD src repository - subversion
- Lint
Lint Skipped - Unit
Tests Skipped
Event Timeline
| sys/arm/arm/hypervisor-stub.S | ||
|---|---|---|
| 2 | You should add a license. | |
| 9 | You should add __FBSDID("$FreeBSD$"); here, and wrap the code with #if __ARM_ARCH >= 7 | |
| 18–21 | Can you add these (and other registers below) to sys/arm/include/sysreg.h then use the macros here. | |
| 25 | You should add a macro to sys/arm/include/armreg.h for this. | |
| sys/arm/arm/locore-v6.S | ||
| 67–69 | I think we should be passing this to initarm via the struct arm_boot_params *. Although I note we are running low on free registers. | |
| sys/conf/files.arm | ||
| 70 | You can make this optional on armv6, we don't need to build it on anything earlier than that. | |
Comment Actions
I've applied all the recomandations from Andrew, besides the "struct arm_boot_params *" one.
Comment Actions
Can you check the diff you uploaded didn't replace tabs with spaces. It's not applying to my svn checkout.