summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDominik Brodowski <linux@dominikbrodowski.net>2018-03-20 15:06:04 -0400
committerDominik Brodowski <linux@dominikbrodowski.net>2018-04-02 14:15:23 -0400
commit3d65661a494a11266500c2532b4f163537c379db (patch)
treeeb645db5b5ddb58b9be70087d3176a02dff08568
parentd969c6fa7263c8fc1928f528bb68587872350b6c (diff)
ipc: add msgget syscall wrapper
Provide ksys_msgget() wrapper to avoid in-kernel calls to this syscall. The ksys_ prefix denotes that this function is meant as a drop-in replacement for the syscall. In particular, it uses the same calling convention as sys_msgget(). 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: Al Viro <viro@zeniv.linux.org.uk> Cc: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Dominik Brodowski <linux@dominikbrodowski.net>
-rw-r--r--ipc/msg.c7
-rw-r--r--ipc/syscall.c4
-rw-r--r--ipc/util.h1
3 files changed, 9 insertions, 3 deletions
diff --git a/ipc/msg.c b/ipc/msg.c
index 0dcc6699dc53..64e8276be164 100644
--- a/ipc/msg.c
+++ b/ipc/msg.c
@@ -263,7 +263,7 @@ static inline int msg_security(struct kern_ipc_perm *ipcp, int msgflg)
263 return security_msg_queue_associate(msq, msgflg); 263 return security_msg_queue_associate(msq, msgflg);
264} 264}
265 265
266SYSCALL_DEFINE2(msgget, key_t, key, int, msgflg) 266long ksys_msgget(key_t key, int msgflg)
267{ 267{
268 struct ipc_namespace *ns; 268 struct ipc_namespace *ns;
269 static const struct ipc_ops msg_ops = { 269 static const struct ipc_ops msg_ops = {
@@ -280,6 +280,11 @@ SYSCALL_DEFINE2(msgget, key_t, key, int, msgflg)
280 return ipcget(ns, &msg_ids(ns), &msg_ops, &msg_params); 280 return ipcget(ns, &msg_ids(ns), &msg_ops, &msg_params);
281} 281}
282 282
283SYSCALL_DEFINE2(msgget, key_t, key, int, msgflg)
284{
285 return ksys_msgget(key, msgflg);
286}
287
283static inline unsigned long 288static inline unsigned long
284copy_msqid_to_user(void __user *buf, struct msqid64_ds *in, int version) 289copy_msqid_to_user(void __user *buf, struct msqid64_ds *in, int version)
285{ 290{
diff --git a/ipc/syscall.c b/ipc/syscall.c
index a536cca37661..355c4a644bbf 100644
--- a/ipc/syscall.c
+++ b/ipc/syscall.c
@@ -68,7 +68,7 @@ SYSCALL_DEFINE6(ipc, unsigned int, call, int, first, unsigned long, second,
68 second, fifth, third); 68 second, fifth, third);
69 } 69 }
70 case MSGGET: 70 case MSGGET:
71 return sys_msgget((key_t) first, second); 71 return ksys_msgget((key_t) first, second);
72 case MSGCTL: 72 case MSGCTL:
73 return sys_msgctl(first, second, (struct msqid_ds __user *)ptr); 73 return sys_msgctl(first, second, (struct msqid_ds __user *)ptr);
74 74
@@ -161,7 +161,7 @@ COMPAT_SYSCALL_DEFINE6(ipc, u32, call, int, first, int, second,
161 return compat_sys_msgrcv(first, ptr, second, fifth, third); 161 return compat_sys_msgrcv(first, ptr, second, fifth, third);
162 } 162 }
163 case MSGGET: 163 case MSGGET:
164 return sys_msgget(first, second); 164 return ksys_msgget(first, second);
165 case MSGCTL: 165 case MSGCTL:
166 return compat_sys_msgctl(first, second, compat_ptr(ptr)); 166 return compat_sys_msgctl(first, second, compat_ptr(ptr));
167 167
diff --git a/ipc/util.h b/ipc/util.h
index 1f1109b83437..b35c0dfe3bc3 100644
--- a/ipc/util.h
+++ b/ipc/util.h
@@ -242,6 +242,7 @@ long ksys_semtimedop(int semid, struct sembuf __user *tsops,
242 const struct timespec __user *timeout); 242 const struct timespec __user *timeout);
243long ksys_semget(key_t key, int nsems, int semflg); 243long ksys_semget(key_t key, int nsems, int semflg);
244long ksys_semctl(int semid, int semnum, int cmd, unsigned long arg); 244long ksys_semctl(int semid, int semnum, int cmd, unsigned long arg);
245long ksys_msgget(key_t key, int msgflg);
245 246
246/* for CONFIG_ARCH_WANT_OLD_COMPAT_IPC */ 247/* for CONFIG_ARCH_WANT_OLD_COMPAT_IPC */
247#ifdef CONFIG_COMPAT 248#ifdef CONFIG_COMPAT