Re: Race condition between putback_lru_page and mem_cgroup_move_list

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: <balbir@...>
Cc: <kosaki.motohiro@...>, Lee Schermerhorn <Lee.Schermerhorn@...>, MinChan Kim <minchan.kim@...>, KAMEZAWA Hiroyuki <kamezawa.hiroyu@...>, linux-mm <linux-mm@...>, Rik van Riel <riel@...>, LKML <linux-kernel@...>
Date: Tuesday, August 5, 2008 - 7:19 am

> In general yes, but in practice no. We have different paths through which a page

hehe, you said mem_cgroup_per_zone::lru_lock is unnecessary lock.

Also, we can two approach

  1. the pages are allowed to exist in different zone of memcg zone and 
     global zone.
     and recover later (by mem_cgroup_isolate_pages).
     here is current implementation.
  2. the pages aren't allowed to exist in different zone of memcg zone and 
     global zone.
     (you said its mail)

if we select 2, I hope mem_cgroup_move_lists is called by ____pagevec_lru_add
and add_page_to_unevictable_list.
then, page's global lru transition become memcg lru transition automatically.
it increase source code readability.




--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: Race condition between putback_lru_page and mem_cgroup_m..., KOSAKI Motohiro, (Tue Aug 5, 7:19 am)