Re: Re: [RFC][PATCH 11/9] lazy lru free vector for memcg

From: kamezawa . hiroyu
Date: Tue Sep 16 2008 - 09:06:49 EST


----- Original Message -----

>[snip]
>
>> @@ -938,6 +1047,7 @@ static int mem_cgroup_force_empty(struct
>> * So, we have to do loop here until all lists are empty.
>> */
>> while (mem->res.usage > 0) {
>> + drain_page_cgroup_all();
>
>Shouldn't we wait here till the drain process completes?
>
I thought schedule_on_each_cpu() watis for completion of the work.
I'll check it, again.

Thank you for review.

Regards,
-Kame


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/