Skip to content

Conversation

@wking
Copy link
Contributor

@wking wking commented Sep 21, 2017

So don't duplicate those values. Also drop RLIMIT_LOCKS, which the man page documents as an obsolete-by-Linux-2.6 value. A quick way to get a set of strings from the man page is:

$ curl -s 'https://git.kernel.org/pub/scm/docs/man-pages/man-pages.git/plain/man2/getrlimit.2?h=man-pages-4.13' |
>   sed -n 's/^\.BR\? \(RLIMIT_[^ ]*\)/\1/p' | sort | uniq

Also rename the old default*Rlimits, because these are valid-value arrays, not defaults.

Following up on the just-merged #461.

So don't duplicate those values.  Also drop RLIMIT_LOCKS, which the
man page documents as an obsolete-by-Linux-2.6 value [1].  A quick way
to get a set of strings from the man page is:

  $ curl -s 'https://git.kernel.org/pub/scm/docs/man-pages/man-pages.git/plain/man2/getrlimit.2?h=man-pages-4.13' |
  >   sed -n 's/^\.BR\? \(RLIMIT_[^ ]*\)/\1/p' | sort | uniq

Also rename the old default*Rlimits, because these are valid-value
arrays, not defaults.

[1]: https://git.kernel.org/pub/scm/docs/man-pages/man-pages.git/tree/man2/getrlimit.2?h=man-pages-4.13#n209

Signed-off-by: W. Trevor King <wking@tremily.us>
@wking wking force-pushed the linux-rlimits-extend-posix-rlimits branch from aff40e1 to 894cae7 Compare September 21, 2017 18:30
@Mashimiao
Copy link

Mashimiao commented Sep 22, 2017

LGTM

Approved with PullApprove

1 similar comment
@zhouhao3
Copy link

zhouhao3 commented Sep 22, 2017

LGTM

Approved with PullApprove

@zhouhao3 zhouhao3 merged commit c9abefb into opencontainers:master Sep 22, 2017
@wking wking deleted the linux-rlimits-extend-posix-rlimits branch September 22, 2017 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants