Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

config-linux.md: Reword kernelTCP docs and mention bytes #377

Merged
merged 1 commit into from
Apr 11, 2016

Conversation

wking
Copy link
Contributor

@wking wking commented Apr 11, 2016

Avoid the dangling “using” from e9a6d94 (cgroup: Add support for
memory.kmem.tcp.limit_in_bytes, 2015-10-26, #235). I've tried to
echo the kernel docs by mentioning buffer memory. I'd personally
prefer linking to the kernel docs and mentioning
memory.kmem.tcp.limit_in_bytes, but that seemed like too big of a
break from the existing style for this commit.

Avoid the dangling 'using' from e9a6d94 (cgroup: Add support for
memory.kmem.tcp.limit_in_bytes, 2015-10-26, opencontainers#235).  I've tried to echo
the kernel docs by mentioning buffer memory [1].  I'd personally
prefer linking to the kernel docs and mentioning
memory.kmem.tcp.limit_in_bytes, but that seemed like too big of a
break from the existing style for this commit.

[1]: https://kernel.org/doc/Documentation/cgroup-v1/memory.txt

Signed-off-by: W. Trevor King <wking@tremily.us>
@vishh
Copy link
Contributor

vishh commented Apr 11, 2016

LGTM

1 similar comment
@mrunalp
Copy link
Contributor

mrunalp commented Apr 11, 2016

LGTM

@mrunalp mrunalp merged commit d445f0c into opencontainers:master Apr 11, 2016
@wking wking deleted the kernel-tcp-using branch April 12, 2016 03:15
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.

None yet

3 participants