[PATCH] user_namespaces.7: Record new 340 line idmap limit

[Christian Brauner <christian.brauner-GeWIH/nMZzLQT0dZR+AlfA@public.gmane.org>]

This patch documents the following commit:

commit 6397fac4915ab3002dc15aae751455da1a852f25
Author: Christian Brauner <christian.brauner-GeWIH/nMZzLQT0dZR+AlfA@public.gmane.org>
Date:   Wed Oct 25 00:04:41 2017 +0200

    userns: bump idmap limits to 340

Since Linux 4.15 the number of idmap lines has been bumped to 340.
The patch also removes the "(arbitrary)" in "There is an (arbitrary) limit on
the number of lines in the file." since the 340 line limit is well-explained by
the current implementation. The struct recording the idmaps is 12 bytes and
quite some proc files only allow writes the size of a single page size which is
4096kB. This leaves room for 340 idmappings (340 * 12 = 4080 bytes).
The struct layout itself has been chosen very carefully to allow for an
implementation that limits the time-complexity for the idmap codepaths to
O(log n). However, I think it's unnecessary to expose this much implementation
detail to users in the man page. So only mention this in the commit message.
Furthermore, the comment about the page size restriction is misleading. The
kernel sources show that >= page size is considered an error.

Signed-off-by: Christian Brauner <christian.brauner-GeWIH/nMZzLQT0dZR+AlfA@public.gmane.org>
---
 man7/user_namespaces.7 | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/man7/user_namespaces.7 b/man7/user_namespaces.7
index 11ca41162..5968c09cc 100644
--- a/man7/user_namespaces.7
+++ b/man7/user_namespaces.7
@@ -510,13 +510,10 @@ and the last field must be greater than 0.
 .IP *
 Lines are terminated by newline characters.
 .IP *
-There is an (arbitrary) limit on the number of lines in the file.
-As at Linux 3.18, the limit is five lines.
+There is a limit on the number of lines in the file.
+As of Linux 4.15, the limit is 340 lines.
 In addition, the number of bytes written to
 the file must be less than the system page size,
-.\" FIXME(Eric): the restriction "less than" rather than "less than or equal"
-.\" seems strangely arbitrary. Furthermore, the comment does not agree
-.\" with the code in kernel/user_namespace.c. Which is correct?
 and the write must be performed at the start of the file (i.e.,
 .BR lseek (2)
 and
-- 
2.14.1

--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html