Re: [PATCH 3/3] mm: idle memory tracking

On Thu, Mar 19, 2015 at 01:12:05PM +0300, Cyrill Gorcunov wrote:

On Wed, Mar 18, 2015 at 11:44:36PM +0300, Vladimir Davydov wrote:

quoted

+static void set_mem_idle(void)
+{
+	int nid;
+
+	for_each_online_node(nid)
+		set_mem_idle_node(nid);
+}

Vladimir, might we need get_online_mems/put_online_mems here,
or if node gets offline this wont be a problem? (Asking
because i don't know).

I only need to dereference page structs corresponding to the node here,
and page structs are not freed when the node gets offline AFAICS, so I
guess it must be safe.

Thanks,
Vladimir