d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Remove direct user access from the arm64 copyinstr

Browse Source 
These already use the load variant that simulates userspace access.
Remove the macros that enable normal loads and stores from userspace
as they are unneeded.

Sponsored by:	Innovate UK
This commit is contained in:
Andrew Turner 2020-10-14 15:31:42 +00:00
parent 6e9127d838
commit 3d839d4bc3
1 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
sys/arm64/arm64/copyinout.S
View File

@ -100,7 +100,6 @@ ENTRY(copyinstr)
adr x6, copyio_fault /* Get the handler address */
SET_FAULT_HANDLER(x6, x7) /* Set the handler */
ENTER_USER_ACCESS(w6, x7)
ldr x7, =VM_MAXUSER_ADDRESS
1: cmp x0, x7
@ -113,8 +112,7 @@ ENTRY(copyinstr)
sub x2, x2, #1 /* len-- */
cbnz x2, 1b
2: EXIT_USER_ACCESS(w6)
SET_FAULT_HANDLER(xzr, x7) /* Clear the handler */
2: SET_FAULT_HANDLER(xzr, x7) /* Clear the handler */
3: cbz x3, 4f /* Check if done != NULL */