* [PATCH 1/2] cgroups: Documentation: fix wrong cgroupfs paths
@ 2014-12-03 11:53 SeongJae Park
2014-12-03 11:53 ` [PATCH 2/2] cgroups: Documentation: fix trivial typos and wrong paragraph numberings SeongJae Park
0 siblings, 1 reply; 5+ messages in thread
From: SeongJae Park @ 2014-12-03 11:53 UTC (permalink / raw)
To: tj
Cc: lizefan, corbet, trivial, cgroups, linux-doc, linux-kernel,
SeongJae Park
Few paths used as example to describe cgroupfs usage have been wrong
from f6e07d38078e ("Documentation: update cgroupfs mount point") by
mistake. This patch fix those trivial wrong paths.
Signed-off-by: SeongJae Park <sj38.park@gmail.com>
---
Documentation/cgroups/cgroups.txt | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/Documentation/cgroups/cgroups.txt b/Documentation/cgroups/cgroups.txt
index 10c949b..f935fac 100644
--- a/Documentation/cgroups/cgroups.txt
+++ b/Documentation/cgroups/cgroups.txt
@@ -312,10 +312,10 @@ the "cpuset" cgroup subsystem, the steps are something like:
2) mkdir /sys/fs/cgroup/cpuset
3) mount -t cgroup -ocpuset cpuset /sys/fs/cgroup/cpuset
4) Create the new cgroup by doing mkdir's and write's (or echo's) in
- the /sys/fs/cgroup virtual file system.
+ the /sys/fs/cgroup/cpuset virtual file system.
5) Start a task that will be the "founding father" of the new job.
6) Attach that task to the new cgroup by writing its PID to the
- /sys/fs/cgroup/cpuset/tasks file for that cgroup.
+ /sys/fs/cgroup/cpuset tasks file for that cgroup.
7) fork, exec or clone the job tasks from this founding father task.
For example, the following sequence of commands will setup a cgroup
--
1.9.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH 2/2] cgroups: Documentation: fix trivial typos and wrong paragraph numberings
2014-12-03 11:53 [PATCH 1/2] cgroups: Documentation: fix wrong cgroupfs paths SeongJae Park
@ 2014-12-03 11:53 ` SeongJae Park
2014-12-03 14:01 ` Jonathan Corbet
0 siblings, 1 reply; 5+ messages in thread
From: SeongJae Park @ 2014-12-03 11:53 UTC (permalink / raw)
To: tj
Cc: lizefan, corbet, trivial, cgroups, linux-doc, linux-kernel,
SeongJae Park
Signed-off-by: SeongJae Park <sj38.park@gmail.com>
---
Documentation/cgroups/cpusets.txt | 6 +++---
Documentation/cgroups/memory.txt | 8 ++++----
2 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/Documentation/cgroups/cpusets.txt b/Documentation/cgroups/cpusets.txt
index 3c94ff3..f2235a1 100644
--- a/Documentation/cgroups/cpusets.txt
+++ b/Documentation/cgroups/cpusets.txt
@@ -445,7 +445,7 @@ across partially overlapping sets of CPUs would risk unstable dynamics
that would be beyond our understanding. So if each of two partially
overlapping cpusets enables the flag 'cpuset.sched_load_balance', then we
form a single sched domain that is a superset of both. We won't move
-a task to a CPU outside it cpuset, but the scheduler load balancing
+a task to a CPU outside its cpuset, but the scheduler load balancing
code might waste some compute cycles considering that possibility.
This mismatch is why there is not a simple one-to-one relation
@@ -552,8 +552,8 @@ otherwise initial value -1 that indicates the cpuset has no request.
1 : search siblings (hyperthreads in a core).
2 : search cores in a package.
3 : search cpus in a node [= system wide on non-NUMA system]
- ( 4 : search nodes in a chunk of node [on NUMA system] )
- ( 5 : search system wide [on NUMA system] )
+ 4 : search nodes in a chunk of node [on NUMA system]
+ 5 : search system wide [on NUMA system]
The system default is architecture dependent. The system default
can be changed using the relax_domain_level= boot parameter.
diff --git a/Documentation/cgroups/memory.txt b/Documentation/cgroups/memory.txt
index 46b2b50..a22df3a 100644
--- a/Documentation/cgroups/memory.txt
+++ b/Documentation/cgroups/memory.txt
@@ -326,7 +326,7 @@ per cgroup, instead of globally.
* tcp memory pressure: sockets memory pressure for the tcp protocol.
-2.7.3 Common use cases
+2.7.2 Common use cases
Because the "kmem" counter is fed to the main user counter, kernel memory can
never be limited completely independently of user memory. Say "U" is the user
@@ -354,19 +354,19 @@ set:
3. User Interface
-0. Configuration
+3.0. Configuration
a. Enable CONFIG_CGROUPS
b. Enable CONFIG_MEMCG
c. Enable CONFIG_MEMCG_SWAP (to use swap extension)
d. Enable CONFIG_MEMCG_KMEM (to use kmem extension)
-1. Prepare the cgroups (see cgroups.txt, Why are cgroups needed?)
+3.1. Prepare the cgroups (see cgroups.txt, Why are cgroups needed?)
# mount -t tmpfs none /sys/fs/cgroup
# mkdir /sys/fs/cgroup/memory
# mount -t cgroup none /sys/fs/cgroup/memory -o memory
-2. Make the new group and move bash into it
+3.2. Make the new group and move bash into it
# mkdir /sys/fs/cgroup/memory/0
# echo $$ > /sys/fs/cgroup/memory/0/tasks
--
1.9.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 2/2] cgroups: Documentation: fix trivial typos and wrong paragraph numberings
2014-12-03 11:53 ` [PATCH 2/2] cgroups: Documentation: fix trivial typos and wrong paragraph numberings SeongJae Park
@ 2014-12-03 14:01 ` Jonathan Corbet
2014-12-03 15:36 ` SeongJae Park
0 siblings, 1 reply; 5+ messages in thread
From: Jonathan Corbet @ 2014-12-03 14:01 UTC (permalink / raw)
To: SeongJae Park; +Cc: tj, lizefan, trivial, cgroups, linux-doc, linux-kernel
On Wed, 3 Dec 2014 20:53:45 +0900
SeongJae Park <sj38.park@gmail.com> wrote:
> Documentation/cgroups/cpusets.txt | 6 +++---
> Documentation/cgroups/memory.txt | 8 ++++----
> 2 files changed, 7 insertions(+), 7 deletions(-)
I applied patch 1 to my docs tree, but this one doesn't apply. Which
kernel did you base it on?
Thanks,
jon
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 2/2] cgroups: Documentation: fix trivial typos and wrong paragraph numberings
2014-12-03 14:01 ` Jonathan Corbet
@ 2014-12-03 15:36 ` SeongJae Park
2014-12-03 16:43 ` Jonathan Corbet
0 siblings, 1 reply; 5+ messages in thread
From: SeongJae Park @ 2014-12-03 15:36 UTC (permalink / raw)
To: Jonathan Corbet
Cc: SeongJae Park, tj, lizefan, trivial, cgroups, linux-doc, linux-kernel
On Wed, 3 Dec 2014, Jonathan Corbet wrote:
> On Wed, 3 Dec 2014 20:53:45 +0900
> SeongJae Park <sj38.park@gmail.com> wrote:
>
>> Documentation/cgroups/cpusets.txt | 6 +++---
>> Documentation/cgroups/memory.txt | 8 ++++----
>> 2 files changed, 7 insertions(+), 7 deletions(-)
>
> I applied patch 1 to my docs tree, but this one doesn't apply. Which
> kernel did you base it on?
I based it on latest linux-next tree.
Looks like I should based it on linux-docs tree. I will do for my later
patches for documentations.
Below is the patch 2 based on linux-docs tree.
========================= >3 ===================================
>From 6f0abd850dfdc4ca06a29e62a268b5c8da931053 Mon Sep 17 00:00:00 2001
From: SeongJae Park <sj38.park@gmail.com>
Date: Wed, 3 Dec 2014 20:45:48 +0900
Subject: [PATCH] cgroups: Documentation: fix trivial typos and wrong
paragraph
numberings
Signed-off-by: SeongJae Park <sj38.park@gmail.com>
---
Documentation/cgroups/cpusets.txt | 6 +++---
Documentation/cgroups/memory.txt | 8 ++++----
2 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/Documentation/cgroups/cpusets.txt
b/Documentation/cgroups/cpusets.txt
index 7740038..2e921b5 100644
--- a/Documentation/cgroups/cpusets.txt
+++ b/Documentation/cgroups/cpusets.txt
@@ -445,7 +445,7 @@ across partially overlapping sets of CPUs would risk
unstable dynamics
that would be beyond our understanding. So if each of two partially
overlapping cpusets enables the flag 'cpuset.sched_load_balance', then we
form a single sched domain that is a superset of both. We won't move
-a task to a CPU outside it cpuset, but the scheduler load balancing
+a task to a CPU outside its cpuset, but the scheduler load balancing
code might waste some compute cycles considering that possibility.
This mismatch is why there is not a simple one-to-one relation
@@ -552,8 +552,8 @@ otherwise initial value -1 that indicates the cpuset
has no request.
1 : search siblings (hyperthreads in a core).
2 : search cores in a package.
3 : search cpus in a node [= system wide on non-NUMA system]
- ( 4 : search nodes in a chunk of node [on NUMA system] )
- ( 5 : search system wide [on NUMA system] )
+ 4 : search nodes in a chunk of node [on NUMA system]
+ 5 : search system wide [on NUMA system]
The system default is architecture dependent. The system default
can be changed using the relax_domain_level= boot parameter.
diff --git a/Documentation/cgroups/memory.txt
b/Documentation/cgroups/memory.txt
index 02ab997..228c1cc 100644
--- a/Documentation/cgroups/memory.txt
+++ b/Documentation/cgroups/memory.txt
@@ -321,7 +321,7 @@ per cgroup, instead of globally.
* tcp memory pressure: sockets memory pressure for the tcp protocol.
-2.7.3 Common use cases
+2.7.2 Common use cases
Because the "kmem" counter is fed to the main user counter, kernel memory
can
never be limited completely independently of user memory. Say "U" is the
user
@@ -349,7 +349,7 @@ set:
3. User Interface
-0. Configuration
+3.0. Configuration
a. Enable CONFIG_CGROUPS
b. Enable CONFIG_RESOURCE_COUNTERS
@@ -357,12 +357,12 @@ c. Enable CONFIG_MEMCG
d. Enable CONFIG_MEMCG_SWAP (to use swap extension)
d. Enable CONFIG_MEMCG_KMEM (to use kmem extension)
-1. Prepare the cgroups (see cgroups.txt, Why are cgroups needed?)
+3.1. Prepare the cgroups (see cgroups.txt, Why are cgroups needed?)
# mount -t tmpfs none /sys/fs/cgroup
# mkdir /sys/fs/cgroup/memory
# mount -t cgroup none /sys/fs/cgroup/memory -o memory
-2. Make the new group and move bash into it
+3.2. Make the new group and move bash into it
# mkdir /sys/fs/cgroup/memory/0
# echo $$ > /sys/fs/cgroup/memory/0/tasks
--
1.9.1
Thanks,
SeongJae Park
>
> Thanks,
>
> jon
>
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 2/2] cgroups: Documentation: fix trivial typos and wrong paragraph numberings
2014-12-03 15:36 ` SeongJae Park
@ 2014-12-03 16:43 ` Jonathan Corbet
0 siblings, 0 replies; 5+ messages in thread
From: Jonathan Corbet @ 2014-12-03 16:43 UTC (permalink / raw)
To: SeongJae Park
Cc: tj, lizefan, trivial, cgroups, linux-doc, linux-kernel, Andrew Morton
On Thu, 4 Dec 2014 00:36:09 +0900 (KST)
SeongJae Park <sj38.park@gmail.com> wrote:
> I based it on latest linux-next tree.
>
> Looks like I should based it on linux-docs tree. I will do for my later
> patches for documentations.
>
> Below is the patch 2 based on linux-docs tree.
Thanks, but the best thing to do might be to have Andrew Morton take the
original patch. His tree already contains the other changes to that file;
keeping things there will help to avoid conflicts and dependencies during
the merge window.
Thanks,
jon
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2014-12-03 16:43 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-12-03 11:53 [PATCH 1/2] cgroups: Documentation: fix wrong cgroupfs paths SeongJae Park
2014-12-03 11:53 ` [PATCH 2/2] cgroups: Documentation: fix trivial typos and wrong paragraph numberings SeongJae Park
2014-12-03 14:01 ` Jonathan Corbet
2014-12-03 15:36 ` SeongJae Park
2014-12-03 16:43 ` Jonathan Corbet
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).