On 04/20/2012 04:38 AM, KAMEZAWA Hiroyuki wrote: >> mem_cgroup_get(memcg); >> > - sk->sk_cgrp = sk->sk_prot->proto_cgroup(memcg); >> > + sk->sk_cgrp = cg_proto; >> > } > > > Is this correct ? cg_proto->active can be true before all jump_labels are > patched, then we can loose accounting. That will cause underflow of > res_countner. > > cg_proto->active should be set after jump_label modification. > Then, things will work, I guess. > > Thanks, > -Kame > Kame, You are right. The first update needs to be done after the jump label activation as well. I got myself confused with the two flags =( I will repost with this fixed once I get into agreement with Tejun and Li about the lockless ->destroy() -- To unsubscribe from this list: send the line "unsubscribe cgroups" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
