{"push_id":1683828892,"size":1,"distinct_size":1,"ref":"refs/heads/percpu-for-next","head":"9209bb43c495e66831d69c927fec0ffc742c171d","before":"2a346e246ccce4161bd0f293a210a050cec56424","commits":[{"sha":"9209bb43c495e66831d69c927fec0ffc742c171d","author":{"name":"Zefan Li","email":"5b5e21a558f468d565c29de4a38a97f6bf4eca64@huawei.com"},"message":"cgroup: avoid attaching a cgroup root to two different superblocks\n\nOn 2017/4/15 7:32, Andrei Vagin wrote:\n> On Fri, Apr 14, 2017 at 04:27:37PM -0700, Andrei Vagin wrote:\n>> Hello,\n>>\n>> One of our CRIU tests hangs with this patch.\n>>\n>> Steps to reproduce:\n>> curl -o cgroupns.c https://gist.githubusercontent.com/avagin/f87c8a8bd2a0de9afcc74976327786bc/raw/5843701ef3679f50dd2427cf57a80871082eb28c/gistfile1.txt\n>> gcc cgroupns.c -o cgroupns\n>> ./cgroupns\n>> ./cgroupns\n>\n> I've found a trivial reproducer:\n> mkdir /tmp/xxx\n> mount -t cgroup -o none,name=zdtmtst xxx /tmp/xxx\n> mkdir /tmp/xxx/xxx\n> umount /tmp/xxx\n> mount -t cgroup -o none,name=zdtmtst xxx /tmp/xxx\n>\n\nNow I remember why it didn't check NULL pointer... Could you try the following fix?\nIt also reverts my previous patch. I would appreciate if you run the full test suit,\nto make sure it won't break anything.\n\nPS: Tejun, I found recently I can no longer receive your emails. Don't know why...\n\n=======\n\n[PATCH] cgruop: avoid attaching a cgroup root to two different superblocks, take 2\n\nCommit bfb0b80db5f9 is broken. Now we try to fix the race by delaying\nthe initialization of cgroup root refcnt until a superblock has been\nallocated.\n\nCc: stable@vger.kernel.org # 3.16+\nReported-by: Dmitry Vyukov <dvyukov@google.com>\nReported-by: Andrei Vagin <avagin@virtuozzo.com>\nSigned-off-by: Zefan Li <lizefan@huawei.com>","distinct":true,"url":"https://api.github.com/repos/avagin/linux/commits/9209bb43c495e66831d69c927fec0ffc742c171d"}]}
{"actor":{"display_login":"avagin"}}