diff options
author | Dominik Brodowski <linux@dominikbrodowski.net> | 2018-03-16 11:58:54 -0400 |
---|---|---|
committer | Dominik Brodowski <linux@dominikbrodowski.net> | 2018-04-02 14:15:18 -0400 |
commit | 73ee3eafd5d1037af9762d28d17611bcf64834d6 (patch) | |
tree | e61cae54b3074bcf0f9bdf4501fb49a792b4f1ba /net/compat.c | |
parent | fd4e82f5b8612ce90bf85f062f2b7954044a1db7 (diff) |
net: socket: add __compat_sys_setsockopt() helper; remove in-kernel call to compat syscall
Using the net-internal helper __compat_sys_setsockopt() allows us to avoid
the internal calls to the compat_sys_setsockopt() syscall.
This patch is part of a series which removes in-kernel calls to syscalls.
On this basis, the syscall entry path can be streamlined. For details, see
http://lkml.kernel.org/r/20180325162527.GA17492@light.dominikbrodowski.net
Cc: David S. Miller <davem@davemloft.net>
Cc: netdev@vger.kernel.org
Signed-off-by: Dominik Brodowski <linux@dominikbrodowski.net>
Diffstat (limited to 'net/compat.c')
-rw-r--r-- | net/compat.c | 14 |
1 files changed, 10 insertions, 4 deletions
diff --git a/net/compat.c b/net/compat.c index 513adc8d0e0f..75bfcbbb2e3e 100644 --- a/net/compat.c +++ b/net/compat.c | |||
@@ -383,8 +383,8 @@ static int compat_sock_setsockopt(struct socket *sock, int level, int optname, | |||
383 | return sock_setsockopt(sock, level, optname, optval, optlen); | 383 | return sock_setsockopt(sock, level, optname, optval, optlen); |
384 | } | 384 | } |
385 | 385 | ||
386 | COMPAT_SYSCALL_DEFINE5(setsockopt, int, fd, int, level, int, optname, | 386 | static int __compat_sys_setsockopt(int fd, int level, int optname, |
387 | char __user *, optval, unsigned int, optlen) | 387 | char __user *optval, unsigned int optlen) |
388 | { | 388 | { |
389 | int err; | 389 | int err; |
390 | struct socket *sock = sockfd_lookup(fd, &err); | 390 | struct socket *sock = sockfd_lookup(fd, &err); |
@@ -410,6 +410,12 @@ COMPAT_SYSCALL_DEFINE5(setsockopt, int, fd, int, level, int, optname, | |||
410 | return err; | 410 | return err; |
411 | } | 411 | } |
412 | 412 | ||
413 | COMPAT_SYSCALL_DEFINE5(setsockopt, int, fd, int, level, int, optname, | ||
414 | char __user *, optval, unsigned int, optlen) | ||
415 | { | ||
416 | return __compat_sys_setsockopt(fd, level, optname, optval, optlen); | ||
417 | } | ||
418 | |||
413 | static int do_get_sock_timeout(struct socket *sock, int level, int optname, | 419 | static int do_get_sock_timeout(struct socket *sock, int level, int optname, |
414 | char __user *optval, int __user *optlen) | 420 | char __user *optval, int __user *optlen) |
415 | { | 421 | { |
@@ -864,8 +870,8 @@ COMPAT_SYSCALL_DEFINE2(socketcall, int, call, u32 __user *, args) | |||
864 | ret = __sys_shutdown(a0, a1); | 870 | ret = __sys_shutdown(a0, a1); |
865 | break; | 871 | break; |
866 | case SYS_SETSOCKOPT: | 872 | case SYS_SETSOCKOPT: |
867 | ret = compat_sys_setsockopt(a0, a1, a[2], | 873 | ret = __compat_sys_setsockopt(a0, a1, a[2], |
868 | compat_ptr(a[3]), a[4]); | 874 | compat_ptr(a[3]), a[4]); |
869 | break; | 875 | break; |
870 | case SYS_GETSOCKOPT: | 876 | case SYS_GETSOCKOPT: |
871 | ret = compat_sys_getsockopt(a0, a1, a[2], | 877 | ret = compat_sys_getsockopt(a0, a1, a[2], |