sysvipc: properly name ipc_addid() limit parameter
authorDavidlohr Bueso <dave@stgolabs.net>
Fri, 17 Nov 2017 23:31:15 +0000 (15:31 -0800)
committerLinus Torvalds <torvalds@linux-foundation.org>
Sat, 18 Nov 2017 00:10:04 +0000 (16:10 -0800)
This is better understood as a limit, instead of size; exactly like the
function comment indicates.  Rename it.

Link: http://lkml.kernel.org/r/20170831172049.14576-4-dave@stgolabs.net
Signed-off-by: Davidlohr Bueso <dbueso@suse.de>
Cc: Manfred Spraul <manfred@colorfullife.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
ipc/util.c

index 429c06bdb8ef7936365fd255dccd09cfb84d5ecc..e09bf76610efb347afc36f91d636250ee0b7b2ea 100644 (file)
@@ -262,7 +262,7 @@ static inline int ipc_buildid(int id, struct ipc_ids *ids,
  * ipc_addid - add an ipc identifier
  * @ids: ipc identifier set
  * @new: new ipc permission set
- * @size: limit for the number of used ids
+ * @limit: limit for the number of used ids
  *
  * Add an entry 'new' to the ipc ids idr. The permissions object is
  * initialised and the first free entry is set up and the id assigned
@@ -271,16 +271,16 @@ static inline int ipc_buildid(int id, struct ipc_ids *ids,
  *
  * Called with writer ipc_ids.rwsem held.
  */
-int ipc_addid(struct ipc_ids *ids, struct kern_ipc_perm *new, int size)
+int ipc_addid(struct ipc_ids *ids, struct kern_ipc_perm *new, int limit)
 {
        kuid_t euid;
        kgid_t egid;
        int id, err;
 
-       if (size > IPCMNI)
-               size = IPCMNI;
+       if (limit > IPCMNI)
+               limit = IPCMNI;
 
-       if (!ids->tables_initialized || ids->in_use >= size)
+       if (!ids->tables_initialized || ids->in_use >= limit)
                return -ENOSPC;
 
        idr_preload(GFP_KERNEL);