public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [Patch] doc: remove ns_cgroup from feature-removal-schedule.txt
@ 2011-06-07 12:06 Américo Wang
  2011-06-07 12:19 ` Daniel Lezcano
  0 siblings, 1 reply; 4+ messages in thread
From: Américo Wang @ 2011-06-07 12:06 UTC (permalink / raw)
  To: LKML, Daniel Lezcano; +Cc: Serge E. Hallyn, Andrew Morton, Paul Menage

In the following commit Daniel removed ns_cgroup

	commit a77aea92010acf54ad785047234418d5d68772e2
	Author: Daniel Lezcano <daniel.lezcano@free.fr>
	Date:   Thu May 26 16:25:23 2011 -0700

	    cgroup: remove the ns_cgroup

but Daniel forgot to remove the related doc in feature-removal-schedule.txt.
This patch removes that too.

Cc: Daniel Lezcano <daniel.lezcano@free.fr>
Cc: Serge E. Hallyn <serge.hallyn@canonical.com>
Signed-off-by: WANG Cong <xiyou.wangcong@gmail.com>

---
diff --git a/Documentation/feature-removal-schedule.txt b/Documentation/feature-removal-schedule.txt
index 1a9446b..72e2384 100644
--- a/Documentation/feature-removal-schedule.txt
+++ b/Documentation/feature-removal-schedule.txt
@@ -481,23 +481,6 @@ Who:	FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
 
 ----------------------------
 
-What:   namespace cgroup (ns_cgroup)
-When:   2.6.38
-Why:    The ns_cgroup leads to some problems:
-	* cgroup creation is out-of-control
-	* cgroup name can conflict when pids are looping
-	* it is not possible to have a single process handling
-	a lot of namespaces without falling in a exponential creation time
-	* we may want to create a namespace without creating a cgroup
-
-	The ns_cgroup is replaced by a compatibility flag 'clone_children',
-	where a newly created cgroup will copy the parent cgroup values.
-	The userspace has to manually create a cgroup and add a task to
-	the 'tasks' file.
-Who:    Daniel Lezcano <daniel.lezcano@free.fr>
-
-----------------------------
-
 What:	iwlwifi disable_hw_scan module parameters
 When:	2.6.40
 Why:	Hareware scan is the prefer method for iwlwifi devices for

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [Patch] doc: remove ns_cgroup from feature-removal-schedule.txt
  2011-06-07 12:06 [Patch] doc: remove ns_cgroup from feature-removal-schedule.txt Américo Wang
@ 2011-06-07 12:19 ` Daniel Lezcano
  2011-06-07 12:22   ` Américo Wang
  0 siblings, 1 reply; 4+ messages in thread
From: Daniel Lezcano @ 2011-06-07 12:19 UTC (permalink / raw)
  To: Américo Wang; +Cc: LKML, Serge E. Hallyn, Andrew Morton, Paul Menage

On 06/07/2011 02:06 PM, Américo Wang wrote:
> In the following commit Daniel removed ns_cgroup
>
> 	commit a77aea92010acf54ad785047234418d5d68772e2
> 	Author: Daniel Lezcano<daniel.lezcano@free.fr>
> 	Date:   Thu May 26 16:25:23 2011 -0700
>
> 	    cgroup: remove the ns_cgroup
>
> but Daniel forgot to remove the related doc in feature-removal-schedule.txt.
> This patch removes that too.

Well, I am not sure we should remove this entry from the documentation.
I let this in the doc because people may want to follow what was removed 
and from what kernel version.
Looking at the doc, there are a lot of old entries.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Patch] doc: remove ns_cgroup from feature-removal-schedule.txt
  2011-06-07 12:19 ` Daniel Lezcano
@ 2011-06-07 12:22   ` Américo Wang
  2011-06-07 12:49     ` Daniel Lezcano
  0 siblings, 1 reply; 4+ messages in thread
From: Américo Wang @ 2011-06-07 12:22 UTC (permalink / raw)
  To: Daniel Lezcano
  Cc: Américo Wang, LKML, Serge E. Hallyn, Andrew Morton,
	Paul Menage

On Tue, Jun 07, 2011 at 02:19:31PM +0200, Daniel Lezcano wrote:
>On 06/07/2011 02:06 PM, Américo Wang wrote:
>>In the following commit Daniel removed ns_cgroup
>>
>>	commit a77aea92010acf54ad785047234418d5d68772e2
>>	Author: Daniel Lezcano<daniel.lezcano@free.fr>
>>	Date:   Thu May 26 16:25:23 2011 -0700
>>
>>	    cgroup: remove the ns_cgroup
>>
>>but Daniel forgot to remove the related doc in feature-removal-schedule.txt.
>>This patch removes that too.
>
>Well, I am not sure we should remove this entry from the documentation.

That is a list of features which will be removed,
not for features that was in the kernel.

>I let this in the doc because people may want to follow what was
>removed and from what kernel version.

We have changelogs or git logs, right? :)

>Looking at the doc, there are a lot of old entries.

Such as?

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Patch] doc: remove ns_cgroup from feature-removal-schedule.txt
  2011-06-07 12:22   ` Américo Wang
@ 2011-06-07 12:49     ` Daniel Lezcano
  0 siblings, 0 replies; 4+ messages in thread
From: Daniel Lezcano @ 2011-06-07 12:49 UTC (permalink / raw)
  To: Américo Wang; +Cc: LKML, Serge E. Hallyn, Andrew Morton, Paul Menage

On 06/07/2011 02:22 PM, Américo Wang wrote:
> On Tue, Jun 07, 2011 at 02:19:31PM +0200, Daniel Lezcano wrote:
>> On 06/07/2011 02:06 PM, Américo Wang wrote:
>>> In the following commit Daniel removed ns_cgroup
>>>
>>> 	commit a77aea92010acf54ad785047234418d5d68772e2
>>> 	Author: Daniel Lezcano<daniel.lezcano@free.fr>
>>> 	Date:   Thu May 26 16:25:23 2011 -0700
>>>
>>> 	    cgroup: remove the ns_cgroup
>>>
>>> but Daniel forgot to remove the related doc in feature-removal-schedule.txt.
>>> This patch removes that too.
>> Well, I am not sure we should remove this entry from the documentation.
> That is a list of features which will be removed,
> not for features that was in the kernel.
>
>> I let this in the doc because people may want to follow what was
>> removed and from what kernel version.
> We have changelogs or git logs, right? :)

Never mind.

The introduction in the doc file is clear:

"... When the feature is removed from the kernel, it should also
be removed from this file."

Sorry I missed this when removing the ns_cgroup.

Thanks for fixing it.

   -- Daniel

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2011-06-07 12:50 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-06-07 12:06 [Patch] doc: remove ns_cgroup from feature-removal-schedule.txt Américo Wang
2011-06-07 12:19 ` Daniel Lezcano
2011-06-07 12:22   ` Américo Wang
2011-06-07 12:49     ` Daniel Lezcano

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox