Messages in this thread | | | Date | Thu, 14 Jan 2010 14:12:25 +0900 | Subject | Re: [PATCH 2/3][v2] vmstat: add anon_scan_ratio field to zoneinfo | From | Minchan Kim <> |
| |
On Thu, Jan 14, 2010 at 8:50 AM, KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com> wrote: > Hi > >> Hi, Kosaki. >> >> On Wed, Jan 13, 2010 at 5:21 PM, KOSAKI Motohiro >> <kosaki.motohiro@jp.fujitsu.com> wrote: >> > Changelog >> > from v1 >> > - get_anon_scan_ratio don't tak zone->lru_lock anymore >> > because zoneinfo_show_print takes zone->lock. >> >> When I saw this changelog first, I got confused. >> That's because there is no relation between lru_lock and lock in zone. >> You mean zoneinfo is allowed to have a stale data? >> Tend to agree with it. > > Well. zone->lock and zone->lru_lock should be not taked at the same time.
I looked over the code since I am out of office. I can't find any locking problem zone->lock and zone->lru_lock. Do you know any locking order problem? Could you explain it with call graph if you don't mind?
I am out of office by tomorrow so I can't reply quickly. Sorry for late reponse.
-- Kind regards, Minchan Kim -- 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/
| |