On Tue, 09 Dec 2008 09:13:12 +0800 Li Zefan <lizf@xxxxxxxxxxxxxx> wrote: > CC: Balbir Singh <balbir@xxxxxxxxxxxxxxxxxx> > CC: KAMEZAWA Hiroyuki <kamezawa.hiroyu@xxxxxxxxxxxxxx> > > Tim Blechmann wrote: > > the lru_lock of struct mem_group_per_zone is used to avoid preemption > > during the __mem_cgroup_stat_add_safe function. > > therefore, a raw_spinlock_t should be used. > > > > FYI, this lru_lock no longer exists in -mm tree. The following patch > removes that lock: > > http://marc.info/?l=linux-mm&m=122665814801979&w=2 > > > Signed-off-by: Tim Blechmann <tim@xxxxxxxxxx> > > --- > > mm/memcontrol.c | 4 ++-- > > 1 files changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/mm/memcontrol.c b/mm/memcontrol.c > > index 517f945..8661732 100644 > > --- a/mm/memcontrol.c > > +++ b/mm/memcontrol.c > > @@ -95,9 +95,9 @@ enum mem_cgroup_zstat_index { > > > > struct mem_cgroup_per_zone { > > /* > > - * spin_lock to protect the per cgroup LRU > > + * raw_spin_lock to protect the per cgroup LRU > > */ > > - spinlock_t lru_lock; > > + raw_spinlock_t lru_lock; > > struct list_head active_list; > > struct list_head inactive_list; > > unsigned long count[NR_MEM_CGROUP_ZSTAT]; yes..I remvoed that. Please see mmotm and subscribe linux-mm, if you have interests memory resource controller. Anyway, thanks. -Kame -- To unsubscribe from this list: send the line "unsubscribe linux-rt-users" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
