LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* [PATCH] Fix declaration of cgroup_mm_owner_callbacks
@ 2008-09-15 19:06 Paul Menage
  2008-09-15 19:10 ` Pekka Enberg
  2008-09-17 21:27 ` Andrew Morton
  0 siblings, 2 replies; 7+ messages in thread
From: Paul Menage @ 2008-09-15 19:06 UTC (permalink / raw)
  To: akpm, balbir, penberg; +Cc: linux-kernel

Fix declaration of cgroup_mm_owner_callbacks()

The choice of real/dummy declaration for cgroup_mm_owner_callbacks()
shouldn't be based on CONFIG_MM_OWNER, but on
CONFIG_CGROUPS. Otherwise kernel/exit.c fails to compile when
something other than a cgroups controller selects CONFIG_MM_OWNER

Signed-off-by: Paul Menage <menage@google.com>

---

This is a retry with a mailer that hopefully didn't break the
whitespace formatting

 include/linux/cgroup.h |   15 ++++++---------
 1 file changed, 6 insertions(+), 9 deletions(-)

Index: mmowner-mmotm-2008-09-10/include/linux/cgroup.h
===================================================================
--- mmowner-mmotm-2008-09-10.orig/include/linux/cgroup.h
+++ mmowner-mmotm-2008-09-10/include/linux/cgroup.h
@@ -403,6 +403,9 @@ void cgroup_iter_end(struct cgroup *cgrp
 int cgroup_scan_tasks(struct cgroup_scanner *scan);
 int cgroup_attach_task(struct cgroup *, struct task_struct *);
 
+void cgroup_mm_owner_callbacks(struct task_struct *old,
+			       struct task_struct *new);
+
 #else /* !CONFIG_CGROUPS */
 
 static inline int cgroup_init_early(void) { return 0; }
@@ -421,15 +424,9 @@ static inline int cgroupstats_build(stru
 	return -EINVAL;
 }
 
+static inline void cgroup_mm_owner_callbacks(struct task_struct *old,
+					     struct task_struct *new) {}
+
 #endif /* !CONFIG_CGROUPS */
 
-#ifdef CONFIG_MM_OWNER
-extern void
-cgroup_mm_owner_callbacks(struct task_struct *old, struct task_struct *new);
-#else /* !CONFIG_MM_OWNER */
-static inline void
-cgroup_mm_owner_callbacks(struct task_struct *old, struct task_struct *new)
-{
-}
-#endif /* CONFIG_MM_OWNER */
 #endif /* _LINUX_CGROUP_H */

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

* Re: [PATCH] Fix declaration of cgroup_mm_owner_callbacks
  2008-09-15 19:06 [PATCH] Fix declaration of cgroup_mm_owner_callbacks Paul Menage
@ 2008-09-15 19:10 ` Pekka Enberg
  2008-09-17 21:27 ` Andrew Morton
  1 sibling, 0 replies; 7+ messages in thread
From: Pekka Enberg @ 2008-09-15 19:10 UTC (permalink / raw)
  To: Paul Menage; +Cc: akpm, balbir, linux-kernel

Paul Menage wrote:
> Fix declaration of cgroup_mm_owner_callbacks()
> 
> The choice of real/dummy declaration for cgroup_mm_owner_callbacks()
> shouldn't be based on CONFIG_MM_OWNER, but on
> CONFIG_CGROUPS. Otherwise kernel/exit.c fails to compile when
> something other than a cgroups controller selects CONFIG_MM_OWNER
> 
> Signed-off-by: Paul Menage <menage@google.com>
> 
> ---
> 
> This is a retry with a mailer that hopefully didn't break the
> whitespace formatting

Acked-by: Pekka Enberg <penberg@cs.helsinki.fi>

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

* Re: [PATCH] Fix declaration of cgroup_mm_owner_callbacks
  2008-09-17 21:27 ` Andrew Morton
@ 2008-09-17 21:24   ` Pekka Enberg
  2008-09-17 21:30   ` Balbir Singh
  1 sibling, 0 replies; 7+ messages in thread
From: Pekka Enberg @ 2008-09-17 21:24 UTC (permalink / raw)
  To: Andrew Morton; +Cc: Paul Menage, balbir, linux-kernel

Andrew Morton wrote:
> On Mon, 15 Sep 2008 12:06:38 -0700 (PDT)
> menage@google.com (Paul Menage) wrote:
> 
>> Fix declaration of cgroup_mm_owner_callbacks()
>>
>> The choice of real/dummy declaration for cgroup_mm_owner_callbacks()
>> shouldn't be based on CONFIG_MM_OWNER, but on
>> CONFIG_CGROUPS. Otherwise kernel/exit.c fails to compile when
>> something other than a cgroups controller selects CONFIG_MM_OWNER
>>
> 
> Being too lazy to go back and find out what "something" is, I assumed
> that this isn't needed in 2.6.27.

Something being the revokeat() patches I recently resurrected. So no, 
not for 2.6.27. :-)

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

* Re: [PATCH] Fix declaration of cgroup_mm_owner_callbacks
  2008-09-15 19:06 [PATCH] Fix declaration of cgroup_mm_owner_callbacks Paul Menage
  2008-09-15 19:10 ` Pekka Enberg
@ 2008-09-17 21:27 ` Andrew Morton
  2008-09-17 21:24   ` Pekka Enberg
  2008-09-17 21:30   ` Balbir Singh
  1 sibling, 2 replies; 7+ messages in thread
From: Andrew Morton @ 2008-09-17 21:27 UTC (permalink / raw)
  To: Paul Menage; +Cc: balbir, penberg, linux-kernel

On Mon, 15 Sep 2008 12:06:38 -0700 (PDT)
menage@google.com (Paul Menage) wrote:

> Fix declaration of cgroup_mm_owner_callbacks()
> 
> The choice of real/dummy declaration for cgroup_mm_owner_callbacks()
> shouldn't be based on CONFIG_MM_OWNER, but on
> CONFIG_CGROUPS. Otherwise kernel/exit.c fails to compile when
> something other than a cgroups controller selects CONFIG_MM_OWNER
> 

Being too lazy to go back and find out what "something" is, I assumed
that this isn't needed in 2.6.27.

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

* Re: [PATCH] Fix declaration of cgroup_mm_owner_callbacks
  2008-09-17 21:27 ` Andrew Morton
  2008-09-17 21:24   ` Pekka Enberg
@ 2008-09-17 21:30   ` Balbir Singh
  2008-09-17 21:39     ` Pekka Enberg
  2008-09-17 23:36     ` Paul Menage
  1 sibling, 2 replies; 7+ messages in thread
From: Balbir Singh @ 2008-09-17 21:30 UTC (permalink / raw)
  To: Andrew Morton; +Cc: Paul Menage, penberg, linux-kernel

Andrew Morton wrote:
> On Mon, 15 Sep 2008 12:06:38 -0700 (PDT)
> menage@google.com (Paul Menage) wrote:
> 
>> Fix declaration of cgroup_mm_owner_callbacks()
>>
>> The choice of real/dummy declaration for cgroup_mm_owner_callbacks()
>> shouldn't be based on CONFIG_MM_OWNER, but on
>> CONFIG_CGROUPS. Otherwise kernel/exit.c fails to compile when
>> something other than a cgroups controller selects CONFIG_MM_OWNER
>>
> 
> Being too lazy to go back and find out what "something" is, I assumed
> that this isn't needed in 2.6.27.

We don't have anything using CONFIG_MM_OWNER in 2.6.27, so we should be OK
pushing this to 2.6.28 (Pekka plans to use it for revoke* calls).

Pekka, Paul do you agree?


-- 
	Balbir

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

* Re: [PATCH] Fix declaration of cgroup_mm_owner_callbacks
  2008-09-17 21:30   ` Balbir Singh
@ 2008-09-17 21:39     ` Pekka Enberg
  2008-09-17 23:36     ` Paul Menage
  1 sibling, 0 replies; 7+ messages in thread
From: Pekka Enberg @ 2008-09-17 21:39 UTC (permalink / raw)
  To: balbir; +Cc: Andrew Morton, Paul Menage, linux-kernel

Balbir Singh wrote:
> Andrew Morton wrote:
>> On Mon, 15 Sep 2008 12:06:38 -0700 (PDT)
>> menage@google.com (Paul Menage) wrote:
>>
>>> Fix declaration of cgroup_mm_owner_callbacks()
>>>
>>> The choice of real/dummy declaration for cgroup_mm_owner_callbacks()
>>> shouldn't be based on CONFIG_MM_OWNER, but on
>>> CONFIG_CGROUPS. Otherwise kernel/exit.c fails to compile when
>>> something other than a cgroups controller selects CONFIG_MM_OWNER
>>>
>> Being too lazy to go back and find out what "something" is, I assumed
>> that this isn't needed in 2.6.27.
> 
> We don't have anything using CONFIG_MM_OWNER in 2.6.27, so we should be OK
> pushing this to 2.6.28 (Pekka plans to use it for revoke* calls).

Oh, I am already using it:

http://git.kernel.org/?p=linux/kernel/git/penberg/linux-2.6.git;a=commitdiff;h=3201a05704afafd7c61b204b7f0c2a9ddcf935d0

Still debugging some ext3 corruption problems, though :-)

> Pekka, Paul do you agree?

Yes, agreed.

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

* Re: [PATCH] Fix declaration of cgroup_mm_owner_callbacks
  2008-09-17 21:30   ` Balbir Singh
  2008-09-17 21:39     ` Pekka Enberg
@ 2008-09-17 23:36     ` Paul Menage
  1 sibling, 0 replies; 7+ messages in thread
From: Paul Menage @ 2008-09-17 23:36 UTC (permalink / raw)
  To: balbir; +Cc: Andrew Morton, penberg, linux-kernel

On Wed, Sep 17, 2008 at 2:30 PM, Balbir Singh <balbir@linux.vnet.ibm.com> wrote:
>>
>> Being too lazy to go back and find out what "something" is, I assumed
>> that this isn't needed in 2.6.27.
>
> We don't have anything using CONFIG_MM_OWNER in 2.6.27, so we should be OK
> pushing this to 2.6.28 (Pekka plans to use it for revoke* calls).
>
> Pekka, Paul do you agree?
>

Yes, there's no rush for this.

Paul

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

end of thread, other threads:[~2008-09-17 23:36 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-09-15 19:06 [PATCH] Fix declaration of cgroup_mm_owner_callbacks Paul Menage
2008-09-15 19:10 ` Pekka Enberg
2008-09-17 21:27 ` Andrew Morton
2008-09-17 21:24   ` Pekka Enberg
2008-09-17 21:30   ` Balbir Singh
2008-09-17 21:39     ` Pekka Enberg
2008-09-17 23:36     ` Paul Menage

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).