Michal Hocko <mhocko@xxxxxxxxxx> wrote: > On Fri 28-04-17 14:55:45, Sean Christopherson wrote: > > This patch set contains two bug fixes for mem_cgroup_iter(). The bugs > > were found by code inspection and were confirmed via synthetic testing > > that forcefully setup the failing conditions. > > I assume that you added some artificial sleeps to make those races more > probable, right? Or did you manage to hit those issue solely from the > userspace? I will have a look at those patches. It has been some time > since I've had it cached. It is pretty subtle code so I would like to > understand the urgency before I dive into this further. > -- > Michal Hocko > SUSE Labs The code to prove the bugs is completely artificial, it's basically a unit test for mem_cgroup_iter() that uses a thread barrier to all but guarantee two threads will call mem_cgroup_iter() simultaneously. I haven't even attempted to hit this via the actual userspace flow. -- 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
