This change has been provided to improve pagezero call performance.
Details
Details
Diff Detail
Diff Detail
- Repository
- rS FreeBSD src repository - subversion
- Lint
Lint Skipped - Unit
Tests Skipped
Event Timeline
Comment Actions
Wouldn't it be (much) cleaner to select one of two implementations (dc/stp) e.g. by function pointer ?
sys/arm64/arm64/support.S | ||
---|---|---|
300 | What is the purpose of this calculation ? pagezero must always be called with the page-aligned address. |
sys/arm64/arm64/machdep.c | ||
---|---|---|
113 | It's ok like that, we only access this directly from this file. If the implementation was in a .c file it should be in a header, however a comment pointing to the .S file will be useful. |
Comment Actions
There seems to be some issues with spaces/tabs between instructions and operands.
Otherwise looks good.
Comment Actions
It seems that they only appear here as in vim the spacing/tabs looks exactly as in implementations of all other functions.