login
Login
/
Register
Search
Search this site:
Forums
News
Blogs
Features
Site
Home
»
Mailing list archives
»
linux-kernel
»
2008
»
September
»
15
Re: [PATCH] cgroups: don't depend on CONFIG_MM_OWNER
view
thread
Previous message: [
thread
] [
date
] [
author
]
Next message: [thread] [
date
] [
author
]
[view in full thread]
From: Pekka Enberg
Subject:
Re: [PATCH] cgroups: don't depend on CONFIG_MM_OWNER
Date: Monday, September 15, 2008 - 10:56 am
Hi Paul, Paul Menage wrote:
quoted text
> On Sun, Sep 14, 2008 at 10:37 AM, Pekka J Enberg <penberg@cs.helsinki.fi> wrote: >> From: Pekka Enberg <penberg@cs.helsinki.fi> >> >> The revoke patches, for example, select CONFIG_MM_OWNER independently of >> cgroups. Therefore, don't depend on CONFIG_MM_OWNER in cgroup specific code. > > Yes, the existing code doesn't seem quite right - if !CONFIG_MM_OWNER > then we don't need to even define a trivial version of > cgroup_mm_owner_callbacks() > > But your patch is too specific - tying the existance of > cgroup_mm_owner_callbacks() to the memory controller would break other > controllers (e.g. the memrlimit or swap controllers, which also want > to use it) > > How about: > > - any cgroup that needs mm-owner callbacks selects an option > CGROUP_MM_OWNER_CALLBACK > > - CGROUP_MM_OWNER_CALLBACK selects MM_OWNER and triggers the > definition of a non-trivial cgroup_mm_owner_callbacks() function
Yeah, sounds good to me. I just want to be able to select CONFIG_MM_OWNER separately for my revoke patches. Pekka --
unsubscribe notice
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to
majordomo@vger.kernel.org
More majordomo info at
http://vger.kernel.org/majordomo-info.html
Please read the FAQ at
http://www.tux.org/lkml/
Previous message: [
thread
] [
date
] [
author
]
Next message: [thread] [
date
] [
author
]
Messages in current thread:
[PATCH] cgroups: don't depend on CONFIG_MM_OWNER
, Pekka J Enberg
, (Sun Sep 14, 10:37 am)
Re: [PATCH] cgroups: don't depend on CONFIG_MM_OWNER
, Paul Menage
, (Mon Sep 15, 10:55 am)
Re: [PATCH] cgroups: don't depend on CONFIG_MM_OWNER
, Pekka Enberg
, (Mon Sep 15, 10:56 am)
Navigation
Mailing list archives
Recent posts
Popular discussions
linux-kernel
:
Paul Turner
[tg_shares_up rewrite v2 11/11] sched: update tg->shares after cpu.shares write
Ingo Molnar
Re: [RFC] Cpu-hotplug: Using the Process Freezer (try2)
Michal Nazarewicz
Re: [PATCH] USB: Gadget: g_multi: added INF file for gadget with multiple configur...
Eric W. Biederman
Re: init's children list is long and slows reaping children.
Jeffrey V. Merkey
Re: Versioning file system
git
:
Matthieu Moy
Re: Bugs in Gitosis
Daniel Barkalow
Re: About git and the use of SHA-1
David Lang
Re: mingw, windows, crlf/lf, and git
Shawn O. Pearce
Re: Bugs in Gitosis
Junio C Hamano
Re: [PATCH 14/21] Convert ce_path_match() to use struct pathspec
linux-netdev
:
David Miller
Re: [2.6.30-rc3] powerpc: compilation error of mace module
David Miller
Re: [PATCH] ipv6: fix display of local and remote sit endpoints
Cong Wang
Re: [PATCH] s2io: add dynamic LRO disable support
Tobacco New Year Promo
Eric Dumazet
Re: [PATCH] net: implement emergency route cache rebulds when gc_elasticity is exc...
git-commits-head
:
Linux Kernel Mailing List
V4L/DVB: tm6000: add special usb request to quit i2c tuner transfer
Linux Kernel Mailing List
of/flattree: merge early_init_dt_scan_memory() common code
Linux Kernel Mailing List
b43: N-PHY: add some registers and structs definitions
Linux Kernel Mailing List
powerpc: Move /proc/ppc64 to /proc/powerpc and add symlink
Linux Kernel Mailing List
drivers/acpi: use kasprintf
openbsd-misc
:
Ted Bullock
Re: Proliant DL380 G3 cannot get on network
Eric Furman
Re: Defending OpenBSD Performance
Damien Miller
Re: Patching a SSH 'Weakness'
Tony Abernethy
Re: The Atheros story in much fewer words
Nick Holland
Re: 1 out of 3 hunks failed--saving rejects to kerberosV/src/lib/krb5/crypto.c.rej
Colocation donated by:
Syndicate