diff options
author | John Baldwin <jhb@FreeBSD.org> | 2019-07-15 21:25:16 +0000 |
---|---|---|
committer | John Baldwin <jhb@FreeBSD.org> | 2019-07-15 21:25:16 +0000 |
commit | c18ca749162a8524ea5b3a6ada0b1e94a46fb81a (patch) | |
tree | 59dd610a980058fc04e326e04cf459792d77d915 /sys/kern/subr_syscall.c | |
parent | 1af9474b2608696269c52a7dd77c54c9fae615d9 (diff) | |
download | src-c18ca749162a8524ea5b3a6ada0b1e94a46fb81a.tar.gz src-c18ca749162a8524ea5b3a6ada0b1e94a46fb81a.zip |
Don't pass error from syscallenter() to syscallret().
syscallret() doesn't use error anymore. Fix a few other places to permit
removing the return value from syscallenter() entirely.
- Remove a duplicated assertion from arm's syscall().
- Use td_errno for amd64_syscall_ret_flush_l1d.
Reviewed by: kib
MFC after: 1 month
Sponsored by: DARPA
Differential Revision: https://reviews.freebsd.org/D2090
Notes
Notes:
svn path=/head/; revision=350013
Diffstat (limited to 'sys/kern/subr_syscall.c')
-rw-r--r-- | sys/kern/subr_syscall.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/sys/kern/subr_syscall.c b/sys/kern/subr_syscall.c index b15cc6dea330..951e7b682623 100644 --- a/sys/kern/subr_syscall.c +++ b/sys/kern/subr_syscall.c @@ -54,7 +54,7 @@ __FBSDID("$FreeBSD$"); #endif #include <security/audit/audit.h> -static inline int +static inline void syscallenter(struct thread *td) { struct proc *p; @@ -167,11 +167,10 @@ syscallenter(struct thread *td) PROC_UNLOCK(p); } (p->p_sysent->sv_set_syscall_retval)(td, error); - return (error); } static inline void -syscallret(struct thread *td, int error __unused) +syscallret(struct thread *td) { struct proc *p; struct syscall_args *sa; |