Page MenuHomeFreeBSD

Add to sysreg definition for coproc regs required for virtualization
ClosedPublic

Authored by mihai on Aug 3 2017, 3:00 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Nov 30, 4:05 PM
Unknown Object (File)
Nov 15 2024, 5:11 AM
Unknown Object (File)
Nov 2 2024, 10:39 PM
Unknown Object (File)
Oct 26 2024, 6:38 AM
Unknown Object (File)
Oct 12 2024, 11:50 PM
Unknown Object (File)
Sep 30 2024, 9:58 AM
Unknown Object (File)
Sep 27 2024, 5:53 AM
Unknown Object (File)
Sep 26 2024, 8:08 PM

Details

Summary

Add to sysreg definition for coproc regs required for virtualization

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

In the future can you please include full context (details on https://wiki.freebsd.org/Phabricator) e.g.

git diff -U999999 other-branch
svn diff --diff-cmd=diff -x -U999999

This looks good. I'm assuming the registers are correct.

This revision is now accepted and ready to land.Aug 3 2017, 5:17 PM

I checked the numbers and they look good.

mmel added inline comments.
sys/arm/include/sysreg.h
86

Please, keep the list sorted - here and below.
And yes, the whole hyp. related block is misordered - It should be at the end of C0 section.

323

I'm only curious here - why a '2' suffix and do you really need LPAE version of TTBR registers?

sys/arm/include/sysreg.h
323

On hyp-mode LPAE is the only supported page table format.

sys/arm/include/sysreg.h
323

It's 2 because is the 64bit version (2 registers). CP15_TTBR0/1 already exists for 32bits. I would have the same name. Please provide another naming convention if this is not ok.

mihai edited edge metadata.

Ordered entries. If they are not order, please provide the methodology of ordering them.

This revision now requires review to proceed.Aug 4 2017, 7:54 PM
mmel added inline comments.
sys/arm/include/sysreg.h
320–326

Rather, I would prefer <foo>_LPAE or something like. But this is only my "soft" personal preference.

This revision is now accepted and ready to land.Aug 11 2017, 11:23 AM
This revision was automatically updated to reflect the committed changes.